Remove “Reload All Tabs” from Tab context menu

VERIFIED FIXED in Firefox 64

Status

()

enhancement
P3
normal
VERIFIED FIXED
Last year
8 months ago

People

(Reporter: amylee, Assigned: jaws)

Tracking

(Depends on 2 bugs, Blocks 1 bug)

63 Branch
Firefox 64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 wontfix, firefox64 verified)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Posted image Tab Context Menu.png
Now that we have multi-select and an updated overflow menu in the works that would have a reload all tabs option, we no longer need a "reload all tabs" option in the context menu. See spec.
Hi, I want to confirmation.
When I choose "Select All Tabs", then, dose tab context menu dinamically change to "Multi-selected Tabs" context menu" without extra right click?

I mean
1. Right click on a tab
2. Choose "Select All Tabs"
   ---- then the context menu will change to "Multi-selected tabs" dynamically. i.e. "Reload Tabs" will appear on the context-menu without any extra operation.
3. Choose "Reload Tabs"


(Anyway, I use the "Reload All Tabs" very often, This bug is so sad)
Priority: -- → P3
(In reply to Alice0775 White from comment #1)
> Hi, I want to confirmation.
> When I choose "Select All Tabs", then, dose tab context menu dinamically
> change to "Multi-selected Tabs" context menu" without extra right click?
> 
> I mean
> 1. Right click on a tab
> 2. Choose "Select All Tabs"
>    ---- then the context menu will change to "Multi-selected tabs"
> dynamically. i.e. "Reload Tabs" will appear on the context-menu without any
> extra operation.
> 3. Choose "Reload Tabs"
> 
> 
> (Anyway, I use the "Reload All Tabs" very often, This bug is so sad)

As it stands, you are correct, there will be another right click. After a button press, the context menu automatically closes. The "Reload Tabs" option is going to be the first option though, so hopefully it's not too much extra effort. However... it might be worth discussing the benefit of leaving the menu open after pressing the "Select All Tabs" button in the context menu. I'll bring it up on bug 1480456, might be worth opening another bug.
Hi, 

I'm proposing we implement a shortcut for "Select All Tabs" and removing it from the context menu which is native to MacOS, Windows, and Linux. That way it removes the need to open the context menu twice and it's one less item in the context menu. We also have a "reload tabs" that reloads all your current tabs in the overflow menu in this bug https://bugzilla.mozilla.org/show_bug.cgi?id=1480542. That means you click once in the overflow menu and click again to select "Reload Tabs"
MozReview-Commit-ID: Ix4oJH1oWvU
As far as I can tell neither the Select All Tabs option nor the updated overflow menu are implemented yet, so it seems we're not yet ready to move forward here.
Depends on: 1480456, 1480542
Hey Ben, would you still like to work on this? I've implemented Select All Tabs now so this bug can be worked on. I have looked at your patch and I saw a couple things we should change. We will need to add a string for "Reload Tabs", and we should add a menuitem to the overflow menu[1]. The "Reload Tabs" string should be used if the contextTab is multiselected.

Also, "Reload All Tabs" is referenced by the toolbar-context-menu which you can see if you right click on the tab toolbar overflow buttons. For this item, we should use the string "Reload Selected Tabs" (which would not change based on multiselection) since there is no tab context in this case.

[1] https://searchfox.org/mozilla-central/source/browser/base/content/browser-allTabsMenu.inc.xul
Flags: needinfo?(ben)
I'll work on this since we need to get this fixed in the next couple days.
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Flags: needinfo?(ben)
Attachment #9001803 - Attachment is obsolete: true
I left off the part about adding Reload Tabs to the All Tabs menu since the specs don't mention how this would appear with the New Container Tab menuitem and the Tab Manager project has been split off from the multiselect tabs project.
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/99483ca5d3c6
Remove 'Reload All Tabs' from tab context menu. r=Felipe
https://hg.mozilla.org/mozilla-central/rev/99483ca5d3c6
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Verified 

Build ID 	20181017101626
User Agent 	Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:64.0) Gecko/20100101 Firefox/64.0
Depends on: 1500515
As a Front End Software Engineer and a UX/UI expert I'd suggest the following context menu:

Reload
Mute
Pin
Duplicate
Bookmark
---
All Tabs ->
            Reload
            Mute
            Pin
            Bookmark
Move Tab ->
            Move to Start
            Move to End
            Move to New Window
Send Tab(18 Tabs) to Device ->
            Firefox on PC-NAME
            ---
            Send to All Devices
---
Close Tab Options ->
            Close Tabs to the Right
            Close Other Tabs
Undo Close Tab
Close Tab

            


This makes the "Select All Tabs" option redundant and saves quite a few clicks.
(In reply to kiril from comment #12)
> As a Front End Software Engineer and a UX/UI expert I'd suggest the
> following context menu:
> 
> Reload
> Mute
> Pin
> Duplicate
> Bookmark
> ---
> All Tabs ->
>             Reload
>             Mute
>             Pin
>             Bookmark
> Move Tab ->
>             Move to Start
>             Move to End
>             Move to New Window
> Send Tab(18 Tabs) to Device ->
>             Firefox on PC-NAME
>             ---
>             Send to All Devices
> ---
> Close Tab Options ->
>             Close Tabs to the Right
>             Close Other Tabs
> Undo Close Tab
> Close Tab
> 
>             
> 
> 
> This makes the "Select All Tabs" option redundant and saves quite a few
> clicks.

Better file a new bug on this proposal, or morph bug 1500515 to evaluate this.
Depends on: 1501352
Verified with 64.0b8 and 65.0a1 (2018-11-11) on Win10, macOS10.13, Ubuntu16.04.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.