Open Bug 519985 Opened 12 years ago Updated 9 years ago

Define additional jump lists items for Firefox

Categories

(Firefox :: Shell Integration, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

People

(Reporter: jimm, Unassigned)

References

(Depends on 1 open bug)

Details

Attachments

(2 obsolete files)

A number of folks have been making suggestions on tasks for the task list as well as secondary lists to favorites and frequent for win7's jump list for firefox. Filing this meta bug for discussion of possible additions and behavior that the ui guys can approve/disapprove. Items that are approved will have bugs filed off to try and get them added.

Current list of jump lists tasks when browser is open:
Open new tab
Open new window

Current list of jump lists tasks when browser is open:
(empty)

Possible additions:

launch in private mode (must have for parity with ie8, bug 506955)
launch directly to preferred search engine
open last closed tab
linksbar links list
restore last session / open without restore
I should also add a caveat, crazy custom lists can easily be handled by extensions. Features suggested here should be common tasks a majority of users would find helpful.

Extension developers can disable firefox's default jump list handling via the browser.taskbar.lists.enabled pref to prevent conflict.
Tasks:
(Close and) Open in safe mode
Open bookmarks manager
Attached patch privacy mode patch v.1 (obsolete) — Splinter Review
- add a privacy toggle switch to tasks
Assignee: nobody → jmathies
Comment on attachment 446225 [details] [diff] [review]
privacy mode patch v.1

Note, converting over to the new Services.jsm is bug 566856, so I'm still doing this manually here.
Attachment #446225 - Flags: review?(dao)
Comment on attachment 446225 [details] [diff] [review]
privacy mode patch v.1

>+  {
>+    get title() {
>+         if (_privateBrowsingSvc.privateBrowsingEnabled)
>+           return _getString("taskbar.tasks.exitPrivacyMode.label")
>+         else
>+           return _getString("taskbar.tasks.enterPrivacyMode.label")},
>+    get description() {
>+         if (_privateBrowsingSvc.privateBrowsingEnabled)
>+           return _getString("taskbar.tasks.exitPrivacyMode.description")
>+         else
>+           return _getString("taskbar.tasks.enterPrivacyMode.description")},

nits:
- the function bodies are indented too far
- semicolon should be added after the return statements
- line break needed before '}'
Attachment #446225 - Flags: review?(dao) → review+
Comment on attachment 446225 [details] [diff] [review]
privacy mode patch v.1

Is this code resilient to dynamic private browsing mode changes?
Attached patch updated patch (obsolete) — Splinter Review
Attachment #446225 - Attachment is obsolete: true
(In reply to comment #6)
> (From update of attachment 446225 [details] [diff] [review])
> Is this code resilient to dynamic private browsing mode changes?

yes, it updates the list when the state changes via an observer event.
(In reply to comment #8)
> (In reply to comment #6)
> > (From update of attachment 446225 [details] [diff] [review] [details])
> > Is this code resilient to dynamic private browsing mode changes?
> 
> yes, it updates the list when the state changes via an observer event.

Where?  Unless there's something wrong with my eyes, you're actually removing that observer.  :-)
(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #6)
> > > (From update of attachment 446225 [details] [diff] [review] [details] [details])
> > > Is this code resilient to dynamic private browsing mode changes?
> > 
> > yes, it updates the list when the state changes via an observer event.
> 
> Where?  Unless there's something wrong with my eyes, you're actually removing
> that observer.  :-)

it still calls "this.update()" down at the bottom. I just removed the variable as it's not used anymore.
Please add "Downloads" and "Add-ons" menu items. Probably will be good add item "Work Offline".
(In reply to comment #12)
> Please add "Downloads" and "Add-ons" menu items. Probably will be good add item
> "Work Offline".

Just for reference, the work offline menu option is near the bottom of the list of frequently used menu items.  While add-ons is the top entry and downloads is shortly after.

http://blog.mozilla.com/metrics/2010/04/14/menu-item-usage-study-an-update-to-the-initial-analysis/
http://blog.mozilla.com/metrics/2010/04/23/menu-item-usage-study-the-80-20-rule/
There appear to be an issue here.. 

My jumplist item didn't change if I start PB mode using the FF Tools Menu.  If I Start using the tools menu, the tasklist shows Enter Private Browsing Mode and proceeds to Quit my private session.
(In reply to comment #14)
> There appear to be an issue here.. 
> 
> My jumplist item didn't change if I start PB mode using the FF Tools Menu.  If
> I Start using the tools menu, the tasklist shows Enter Private Browsing Mode
> and proceeds to Quit my private session.

Odd, we're hooked up to the observer for this. I'll take a look.
(In reply to comment #15)
> (In reply to comment #14)
> > There appear to be an issue here.. 
> > 
> > My jumplist item didn't change if I start PB mode using the FF Tools Menu.  If
> > I Start using the tools menu, the tasklist shows Enter Private Browsing Mode
> > and proceeds to Quit my private session.
> 
> Odd, we're hooked up to the observer for this. I'll take a look.

Seems to be working here.

1) open browser
2) select start privacy mode
(check jump list title)
3) select quit privacy mode
(check jump list title)

In both cases, the title is correct.

I filed bug 568816 on a related problem, exiting privacy mode via the menu item after entering it via the command line option doesn't appear to work.
(In reply to comment #16)
> (In reply to comment #15)
> > (In reply to comment #14)
> > > There appear to be an issue here.. 
> > > 
> > > My jumplist item didn't change if I start PB mode using the FF Tools Menu.  If
> > > I Start using the tools menu, the tasklist shows Enter Private Browsing Mode
> > > and proceeds to Quit my private session.
> > 
> > Odd, we're hooked up to the observer for this. I'll take a look.
> 
> Seems to be working here.
> 
> 1) open browser
> 2) select start privacy mode
> (check jump list title)
> 3) select quit privacy mode
> (check jump list title)
> 
> In both cases, the title is correct.
> 
> I filed bug 568816 on a related problem, exiting privacy mode via the menu item
> after entering it via the command line option doesn't appear to work.

Thanks.  I see the problem I was having is related to deleting items from the Jumplist Freq/Recent section using "Remove from this list" and switching profiles corrected the issue after doing some investigation (bug 568694).
Depends on: 577212
Would we be able to add startup with profile manager, or startup in safe mode to the list?
Depends on: 607722
Depends on: 612654
Blocks: 515785
No longer depends on: 515785
Attachment #446350 - Attachment is obsolete: true
Assignee: jmathies → nobody
You need to log in before you can comment on or make changes to this bug.