Make the MenuSeparator span the full width of the menu
like in QWidget-based menus
Details
Details
- Reviewers
mart apol ngraham - Commits
- R858:6eb266c3d4fc: Add MenuSeparator
Diff Detail
Diff Detail
- Repository
- R858 Qt Quick Controls 2: Desktop Style
- Branch
- menu_separator
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 13192 Build 13210: arc lint + arc unit
Comment Actions
In the after picture I'm noticing that the separator is 1px outside the frame when the menus is over a dark bg.
org.kde.desktop/MenuSeparator.qml | ||
---|---|---|
38 | It shouldn't need any height at all. |
org.kde.desktop/MenuSeparator.qml | ||
---|---|---|
38 | Oh duh, I was thinking it was a Rectangle, but it's a Kirigami.Separator |
Comment Actions
LGTM
To be honest, I would expect this to be the actual logic and Kirigami.Separator just use MenuSeparator, but maybe it's something we can look into in the future.