This is a pretty longstanding feature request and all the HIG and VDG people polled
were in favor. Since those were the people requested to sign off on it, it looks like
we can make the necessary code change now.
BUG: 106807
FIXED-IN: 5.63
cfeck | |
ndavis |
VDG | |
Frameworks |
This is a pretty longstanding feature request and all the HIG and VDG people polled
were in favor. Since those were the people requested to sign off on it, it looks like
we can make the necessary code change now.
BUG: 106807
FIXED-IN: 5.63
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
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?
@cfeck can clarify, but I believe it specifies the location of locally-added actions that come from the app itself. It would be empty if the app has a View menu but doesn't add any of its own actions (which seems unlikely, but is theoretically possible).
Alternatively we could remove the separator below redisplay, which would allow us to always have a separator above Full Screen Mode with no danger of ever having double separators.