-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Application menu button #37
Comments
@BobbyWibowo i'm don't know application button ID for aurorae. And i think it's impossible: |
Huh yeah, apparently even in 2017 another theme creator stumbled upon this yet failed to find a solution: andreyorst/Breezemite#5 (comment). I also found this bug report: https://bugs.kde.org/show_bug.cgi?id=375862, however it appears to be pretty much abandoned or only half-solved (possibly due to the fact that no other users said anything more, lol). Previously you couldn't even add application menu icon programmatically, but a fix was added soon after the report to solve that (as can be seen by how Plastik theme is doing it programmatically here, https://phabricator.kde.org/source/kwin/browse/master/plugins/kdecorations/aurorae/themes/plastik/code/plastikbutton.cpp$104). This appears to be the file that will load SVG-based themes, https://phabricator.kde.org/source/kwin/browse/master/plugins/kdecorations/aurorae/src/lib/auroraetheme.cpp$111, and if so, yeah, there is no line to load application menu icon at all. |
Anyway, the bug report I linked was already marked as resolved, so I created a new one, with more details: https://bugs.kde.org/show_bug.cgi?id=404322. |
You see, this not contained application menu icon now:
|
The upstream bug is fixed in https://invent.kde.org/plasma/kwin/-/commit/ca686f651a90f136ef887416fd318cc44624c356 . |
Application menu button in titlebar uses to the app's icon instead of a regular hamburger menu icon.
Here's how it looks with Materia KDE window decoration:
How it looks with Breeze window decoration:
I'm on Plasma 5.15.0.
It's not only limited to Dolphin. All other apps that supports the menu thingy will use their app icons instead.
The text was updated successfully, but these errors were encountered: