Add 'reverse tabs' menu option to the tab context menu for tabs in a Splitview
Categories
(Firefox :: Tabbed Browser: Split View, enhancement, P2)
Tracking
()
People
(Reporter: flod, Assigned: kcochrane)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fidefe-splitview])
Attachments
(3 files, 2 obsolete files)
It would be very useful to have a quick way to flip the two tabs displayed in Split view (A-B to B-A).
Updated•2 months ago
|
Updated•2 months ago
|
Comment 1•2 months ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #0)
It would be very useful to have a quick way to flip the two tabs displayed in Split view (A-B to B-A).
Are you finding it not convenient to reverse the tabs via the footer in the inactive panel or urlbar indicator menu options?
| Reporter | ||
Comment 2•2 months ago
|
||
They're perfectly usable, but I don't think they're discoverable.
I've been using this feature from the beginning, and yet I discovered the menu via Slack, and the address bar icon literally from your message.
Updated•2 months ago
|
Updated•2 months ago
|
Laura, can we please ask you to weigh in whether you think the string should be:
Reverse tabs
OR
Reverse Tabs in Split View (more verbose, but possibly more helpful?)
This option will only appear in context menu of a tab added to the split view. I also attached the screenshot to show how Reverse tabs string would look like.
| Assignee | ||
Comment 5•2 months ago
|
||
Laura, quick flag - we went for Reverse tabs (since it's past string freeze, and we already have that string in Fluent).
We can change it in release 150 if we feel strongly about the longer string being better.
Comment 9•2 months ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined: Users will not have a way to reverse split view tabs in the tab context menu
- Code covered by automated testing: yes
- Fix verified in Nightly: no
- Needs manual QE test: yes
- Steps to reproduce for manual QE testing: Create a split view. Right click one of the split view tabs and confirm you see a new menu option for Reverse Tabs. Clicking the option should swap the order of the tabs in the split view. This new option should only show for split view tabs and will not show if tabs are multiselected.
- Risk associated with taking this patch: low
- Explanation of risk level: Adding a new split view specific context menu option that is covered by automated tests.
- String changes made/needed: N/A
- Is Android affected?: no
| Assignee | ||
Comment 10•2 months ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D285527
Comment 11•2 months ago
|
||
(In reply to Phabricator Automation from comment #9)
- String changes made/needed: N/A
The patch is adding a string to browser/locales/en-US/browser/tabbrowser.ftl
I believe you need l10n-drivers approval, flod, is that OK to uplift?
| Reporter | ||
Comment 12•2 months ago
|
||
I'm redirecting to Bryan, but also confused by this comment from Ania
Laura, quick flag - we went for Reverse tabs (since it's past string freeze, and we already have that string in Fluent).
I understood that 149 was going to use an existing string?
https://searchfox.org/firefox-main/source/browser/locales/en-US/browser/tabbrowser.ftl#460-461
Comment 13•2 months ago
|
||
The string tab-context-reverse-split-view just landed in tree at the end of the day yesterday which leaves 2 weeks for localizations. There's a greater risk that this string would appear in English for some locales. Instead of uplifting this string, could we temporarily reference the existing string split-view-menuitem-reverse-tabs for 149 instead?
Updated•2 months ago
|
Updated•2 months ago
|
| Assignee | ||
Comment 14•2 months ago
•
|
||
The reason I didn't use the existing string was because there is no access key defined for it. I would add an access key to the existing string, but that would add an access key to that option in existing menus where it didn't exist previously. Ex: The split view popup menu from the Urlbar and split view footer.
I'm reaching out to :asafko on which route we should go here:
Sounds like we can do one of the following:
- Uplift with the existing string but it will have no access key in the tab context menu
- Uplift with the existing string and add an access key which will add an access key to other existing menus unexpectedly
- Not uplift this and let it ride the train for 150
| Assignee | ||
Comment 15•2 months ago
|
||
Update: Ania would like us to go with the existing string with no access key for the uplift. Working on a new patch for that now.
| Assignee | ||
Updated•2 months ago
|
| Assignee | ||
Comment 16•2 months ago
|
||
| Assignee | ||
Comment 17•2 months ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D286073
Updated•2 months ago
|
Comment 18•2 months ago
|
||
| Reporter | ||
Comment 19•2 months ago
|
||
(In reply to Pulsebot from comment #18)
Pushed by kcochrane@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/230e8b941caf
https://hg.mozilla.org/integration/autoland/rev/ba88f2c34305
Use existing 'Reverse Tabs' string for new tab context menu option
r=fluent-reviewers,bolsson
Why did we land this on main? Isn't this just a temporary fix for beta and 149?
Comment 20•2 months ago
|
||
Hello! I have verified with Firefox 150.0a1 (2026-03-05) on Windows 11, macOS 26, and Ubuntu 24 that the Reverse tabs context menu option is displayed and works as expected for both views in horizontal and vertical tabs. The Revese tabs context menu option is not displayed when other tab and Split View are selected.
However, I have two questions:
- I see that pressing the
Rkeyboard key on Windows/ Linux when the context menu is open no longer reloads the tab as before. It cycles between theReverse Tabs optionand theReload Tab. - The
Reverse tabscontext menu option is not displayed when both views are selected.
Are the above expected? Thank you!
Comment 21•2 months ago
|
||
| bugherder | ||
Comment 22•2 months ago
|
||
A patch has been attached on this bug, which was already closed. Filing a separate bug will ensure better tracking. If this was not by mistake and further action is needed, please alert the appropriate party. (Or: if the patch doesn't change behavior -- e.g. landing a test case, or fixing a typo -- then feel free to disregard this message)
| Assignee | ||
Comment 23•2 months ago
•
|
||
(In reply to Francesco Lodolo [:flod] from comment #19)
(In reply to Pulsebot from comment #18)
Pushed by kcochrane@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/230e8b941caf
https://hg.mozilla.org/integration/autoland/rev/ba88f2c34305
Use existing 'Reverse Tabs' string for new tab context menu option
r=fluent-reviewers,bolssonWhy did we land this on
main? Isn't this just a temporary fix forbetaand 149?
I was asked to write a follow-up bug to switch back to the other string: https://bugzilla.mozilla.org/show_bug.cgi?id=2021063
If it's easier, we can backout the change to main and only uplift the change for beta, but I wasn't sure how to uplift something I hadn't also done for main.
| Assignee | ||
Comment 24•2 months ago
•
|
||
(In reply to Alexandru Trif, Desktop Test Engineering [:atrif] from comment #20)
Hello! I have verified with Firefox 150.0a1 (2026-03-05) on Windows 11, macOS 26, and Ubuntu 24 that the
Reverse tabscontext menu option is displayed and works as expected for both views in horizontal and vertical tabs. TheRevese tabscontext menu option is not displayed when other tab and Split View are selected.
However, I have two questions:
- I see that pressing the
Rkeyboard key on Windows/ Linux when the context menu is open no longer reloads the tab as before. It cycles between theReverse Tabs optionand theReload Tab.
We are now using the existing string for this that does not include an access key: https://searchfox.org/firefox-main/source/browser/locales/en-US/browser/tabbrowser.ftl#464
In bug 2021063, we will switch back to using the string with the access key: https://searchfox.org/firefox-main/source/browser/locales/en-US/browser/tabbrowser.ftl#453
- The
Reverse tabscontext menu option is not displayed when both views are selected.
We decided that the 'Reverse Tabs' option will not show when multiselection is being used regardless of if both tabs are in a split view. We wanted to avoid a scenario where a user selects multiple tabs across multiple split views and tries to reverse all the tabs. It will only show when you right click one split view tab.
| Assignee | ||
Updated•2 months ago
|
| Reporter | ||
Comment 25•2 months ago
|
||
Not sure what the question for the NI is? I don't have a problem with landing and then fixing, but I'd be surprised if you can't land a patch directly to beta via RelMan (it's definitely possible for ESR).
We are now using the existing string for this that does not include an access key: https://searchfox.org/firefox-main/source/browser/locales/en-US/browser/tabbrowser.ftl#464
In bug 2021063, we will switch back to using the string with the access key: https://searchfox.org/firefox-main/source/browser/locales/en-US/browser/tabbrowser.ftl#453
Pretty sure the issue is with the new string, not with the one in Beta (that doesn't have an accesskey). So, you might have to pick another accesskey in the followup.
Updated•2 months ago
|
Comment 26•2 months ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #25)
Not sure what the question for the NI is? I don't have a problem with landing and then fixing, but I'd be surprised if you can't land a patch directly to beta via RelMan (it's definitely possible for ESR).
Yes, no problem landing a patch directly to beta to address a release-specific issue.
Updated•2 months ago
|
Updated•2 months ago
|
Comment 27•2 months ago
|
||
Kelly, your patches do not apply cleanly to the beta branch, could you rebase them please? Thanks
Updated•2 months ago
|
Comment 28•2 months ago
|
||
| uplift | ||
| Assignee | ||
Comment 29•2 months ago
|
||
Rebased
| Assignee | ||
Updated•2 months ago
|
Comment 30•2 months ago
|
||
Confirming that the Reverse tabs context menu option is present in Firefox 149.0b5 (20260305221805) for a Split View when no multiselection is used. Also, the R key is not assigned for the Reverse Tabs. Unfortunately, I discovered bug 2021527 when testing this.
| Assignee | ||
Comment 31•2 months ago
|
||
Given the discovery of bug 2021527, we'd like to back out this stack of patches from 149 beta and just have it ride the trains to 150.
Comment 32•2 months ago
|
||
| uplift | ||
Comment 33•2 months ago
|
||
Reverted. Tracking for a potential uplift in our planned dot release.
Updated•1 month ago
|
Updated•1 month ago
|
| Assignee | ||
Updated•1 month ago
|
Comment 34•1 month ago
|
||
:kcochrane did you want to include this in the 150 release notes? Please nominate if so.
| Assignee | ||
Updated•27 days ago
|
Comment 35•26 days ago
|
||
I added a note to release notes here.
It's a joint note with another bug, sharing the text here just in case:
_Split View just got better: right-click any link and choose Open Link in Split View to open it alongside your current tab. You can also quickly reverse tab positions using the new Reverse Tabs option in the tab context menu.
_
Updated•24 days ago
|
Description
•