| Summary: | Menu metrics don't match those of the QStyle | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] frameworks-qqc2-desktop-style | Reporter: | Raghavendra kamath <raghu> |
| Component: | general | Assignee: | Marco Martin <notmart> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ahiemstra, kde, kfm-devel, nate, noahadvs |
| Priority: | NOR | ||
| Version: | 5.89.0 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/frameworks/qqc2-desktop-style/commit/bc0ca29bd47ce45b30c66bd8743b6a48e20123d2 | Version Fixed In: | 5.91 |
| Attachments: | screenshot showing inconsistent spacing in hamburger | ||
The problem is actually the opposite here: menus in QtQuick apps aren't respecting the metrics of the QWidgets menus. If we want bigger menu items, that would ideally need to be set in the QStyle, and it would be inherited automatically in QtQuick apps like Elisa. However we did try this once before and a lot of people hated it, especially those who use apps with very large menus that became scrollable as a result of the change. A possibly relevant merge request was started @ https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/112 Git commit bc0ca29bd47ce45b30c66bd8743b6a48e20123d2 by Nate Graham. Committed on 06/01/2022 at 15:14. Pushed by ngraham into branch 'master'. Use metrics from Breeze for menu items We can't read them dynamically right now, unfortunately. So the values are hardcoded. FIXED-IN: 5.91 M +3 -0 org.kde.desktop/Menu.qml M +3 -3 org.kde.desktop/MenuItem.qml https://invent.kde.org/frameworks/qqc2-desktop-style/commit/bc0ca29bd47ce45b30c66bd8743b6a48e20123d2 |
Created attachment 144717 [details] screenshot showing inconsistent spacing in hamburger Comparing the hamburger menu in elisa and dolphins shows that dolphins hamburger menu is tighter in spacing that elisa's. The space in elisa's hamburger menu makes it look clean while tighter menu in dolphin makes it look busy. Regardless of which spacing to use the spacing across application in KDE can be same to maintain consistency. I am guessing this should be filed under Dolphin since I notice the spacing issue in dolphin and I think dolphin's menu need improvement. If this is not the right product please move this to correct location or let me know the correct place so that I file a bug in that product. Attaching a screenshot comparing both hamburger menu left is dolphin and right is elisa