Skip to content

Fixes #46. Submenu parentage should be to the parent menu#59

Merged
markfinal merged 1 commit intomainfrom
mf/46_submenu_parents
Jun 17, 2023
Merged

Fixes #46. Submenu parentage should be to the parent menu#59
markfinal merged 1 commit intomainfrom
mf/46_submenu_parents

Conversation

@markfinal
Copy link
Owner

This was giving a warning on Weyland if the parent was not set correctly.

WARNING:cruiz.entrypoint:setGrabPopup called with a parent, QtWaylandClient::QWaylandXdgSurface(0x55c528e78d30) which does not match the current topmost grabbing popup, QtWaylandClient::QWaylandXdgSurface(0x55c528e91760) According to the xdg-shell protocol, this is not allowed. The wayland QPA plugin is currently handling it by setting the parent to the topmost grabbing popup. Note, however, that this may cause positioning errors and popups closing unxpectedly because xdg-shell mandate that child popups close before parents

This was giving a warning on Weyland if the parent was not set correctly.

WARNING:cruiz.entrypoint:setGrabPopup called with a parent, QtWaylandClient::QWaylandXdgSurface(0x55c528e78d30) which does not match the current topmost grabbing popup, QtWaylandClient::QWaylandXdgSurface(0x55c528e91760) According to the xdg-shell protocol, this is not allowed. The wayland QPA plugin is currently handling it by setting the parent to the topmost grabbing popup. Note, however, that this may cause positioning errors and popups closing unxpectedly because xdg-shell mandate that child popups close before parents
@markfinal markfinal self-assigned this Jun 17, 2023
@markfinal markfinal merged commit f8f78dc into main Jun 17, 2023
@markfinal markfinal deleted the mf/46_submenu_parents branch June 17, 2023 20:03
@markfinal markfinal added this to the v1.4.0 milestone Jun 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant