ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land.
+1, much better than having a submenu with only two items when the main menu is itself not very big. Works great. One thought: with this patch, there will be two disabled and inapplicable items in the menu when you're not using Short View, which might confuse people. Might be appropriate to only show them for Short View. REPOSITORY R241 KIO BRANCH icon-position-usability (branched from master) REVISION DETAIL https://phabricator.kde.org/D12591 To: rkflx, #frameworks, ngraham Cc: ngraham, elvisangelaccio, michaelh, bruns
