ndavis added a comment.
In D24091#534786 <https://phabricator.kde.org/D24091#534786>, @ngraham wrote: > Do you think I should always have a separator above the Full Screen menu item? This should generally work fine, but it means that in case the `MergeLocal` content is empty, there will be two separators in a row. I don't know how likely this is to ever happen, but it could. Thoughts? I'm not sure what `MergeLocal` does or when it would be empty. REPOSITORY R263 KXmlGui BRANCH move_fullscreen_to_view_menu (branched from master) REVISION DETAIL https://phabricator.kde.org/D24091 To: ngraham, #vdg, #frameworks, cfeck, ndavis Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
