Simplify Recently Closed Tabs and Recently Closed Windows subviews
Categories
(Firefox :: Menus, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox87 | --- | verified |
People
(Reporter: mconley, Assigned: emmamalysz)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-hamburger-menu])
Attachments
(1 file)
This is something that can ship pre-Proton, so it can happen immediately, and doesn't need to be behind a pref.
The Recently Closed Tabs and Recently Closed Windows subviews are dynamically generated. They have their "showing" event handlers set up here:
and the actual work of filling them with items is here:
There are some utilities and abstractions in browser/components/sessionstore/RecentlyClosedTabsAndWindowsMenuUtils.jsm that will likely need some slight modification.
The structures (I'm combining them here since they're so similar, but they remain two separate submenus) should be:
Recently Closed Tabs/Windows <header>
<separator>
<list of recently closed tabs/windows>
<separator>
Reopen All Tabs/Windows
What we need to do:
- Remove the keyboard shortcut indicator on the first item in both subviews. This can be done by removing these lines, and then I think we can also then get rid of this
- Turn the "Restore All Tabs" and "Restore All Windows" footer into a normal toolbarbutton subview item
- Rename "Restore All Tabs" to "Reopen All Tabs"
- Rename "Restore All Windows" to "Reopen All Windows"
One thing we need to figure out with UX is whether or not we're going to change the limit on how many recently closed tabs / windows to show - that list can get quite long. Do we limit the number to show, or do we make the list a fixed maximum height with a scrollbar? Or something else entirely?
Remember when updating strings to ensure that they're not re-used elsewhere - if so, double-check that the propagated change is acceptable with UX, or create a new string just for the menu.
Also remember that if you're updating a string, you still have to update the ID of it so that our localizers know to re-translate.
Reporter | ||
Comment 1•3 years ago
|
||
(Naturally, any tests that rely on the old structure or old behaviours will need to be modified as well)
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 2•3 years ago
|
||
Updated•3 years ago
|
Pushed by emalysz@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/61892c5fe604 update recently closed windows and tabs panels r=mconley,fluent-reviewers,flod
Comment 4•3 years ago
|
||
bugherder |
Comment 5•3 years ago
|
||
(In reply to Mike Conley (:mconley) (:⚙️) (Catching up on needinfos) from comment #0)
- Remove the keyboard shortcut indicator on the first item in both subviews.
I believe this was the only place where we exposed the Undo Close Window shortcut? (Undo Close Tab is exposed in the tab menu, which we currently only show when overflowing, and we might want to replace this item with the subview in the future.) Was this known context for this change? I'm a bit puzzled as to how this removal, while it is technically a simplification, can be seen as a net win.
Reporter | ||
Comment 6•3 years ago
|
||
Checking back with Meridel and amylee. Let's track this in bug 1692522.
Comment 7•3 years ago
•
|
||
Confirm that keyboard shortcut indicator appears again in the latest Nightly 89.0a1(See the fixed bug 1692522).
Verified - Fixed in latest Nightly 89.0a1 (2021-04-19), Fx 87.0 and Fx 88.0, using Windows 10x64 and Mac 11.
Updated•3 years ago
|
Updated•3 years ago
|
Description
•