This bug was spun off from the comments attributed to bug #599731c11 and bug #599731c12. Meaning group all restore session menu items together.
Ok, further comment. Bug #599731 moved the Recently Closed menu items from the File menu to the Go Menu. Now basically, the "Restore Previous Session" should be 'grouped' with these menus.
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Created attachment 559084 [details] [diff] [review] Group all session restore menu items (v1) On the first view, the Go menu has an extra disabled "Tab from other computers" menu item. Subsequent views, that item disappears.
Comment on attachment 559084 [details] [diff] [review] Group all session restore menu items (v1) >+ endHistory.hidden = (endHistory == aParent.lastChild) || (syncMenuItem.hidden); Since you didn't ask anyone specifically for review yet, here's a drive-by: Technically, no parentheses is needed here at all. Personally I'd go with one (for everything on the right side), but the reviewer you choose might disagree.
Created attachment 582743 [details] [diff] [review] Group all session restore menu items together (v2)
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/b9f97ed7e628
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.