Don't use menubuttons where we don't want buttons#411
Merged
Conversation
Collaborator
Author
|
From my daily testing this is now in place. BUT the change from button to box/image whatever internals has changed some of the extra padding added by button. The panel is more compact again, especially around menu button. |
trigg
force-pushed
the
less-buttons
branch
2 times, most recently
from
8c8c502 to
99f9338
Compare
Collaborator
Author
|
fixes #418 |
This was referenced May 16, 2026
trigg
force-pushed
the
less-buttons
branch
2 times, most recently
from
e4820e8 to
37f0ffd
Compare
trigg
changed the title
trigg
force-pushed
the
less-buttons
branch
2 times, most recently
from
16f25e4 to
0401c06
Compare
panel: start standardising on popups
panel: fix popup too big for screen
panel: fix menu crash before init panel: made menu-follows-mouse an option
panel: widgets can optionally have fullscreen popover panel: merged popover classes panel: window-list stop using button
background: remove keyboard on_demand
panel: account for smaller widget size now we have removed buttons
panel: reimplement volume scroll as it no longer hangs panel
panel: menuwidget less leakage when rebinding opens_on
the scrolled window was being returned. proper return accounts for scroll window wrapping children in viewports
added configuration options for the now-possible mute on left click cleaned up handling of opening, closing and replacing the popover
…le screen size, if it is too large this is breaking because the way size is calculated is not consistent with before the change
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refactor internal WayfireMenuButton to no longer actually be a menu button
MenuButton was originally chosen as it was the only way I had found to make a popover-menu for panel widgets.
We put a fair bit of extra code to make MenuButton not actually be a menubutton. So this PR will standardise an interface for all Popup-based widgets without each one having to fight to not look like a button
Advantages over not having this PR: