Closed Bug 407383 Opened 17 years ago Closed 17 years ago

Manage Add-ons in Preferences may open the add-on manager with an irrelevant pane selected

Categories

(Firefox :: Settings UI, enhancement)

enhancement
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 3 beta3

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

FF3b1 & latest nightly. The Manage Add-ons button in the Preferences window does not switch to the Add-ons pane when opening the Add-ons dialog. Steps to reproduce: 1. Open Add-ons manager, and switch to the Plugins tab for example. 2. Close it. 3. Open Preferences. 4. Click Manage Add-ons. Expected Results: the add-ons manager should open with the Add-ons pane selected. Actual Results: The add-ons manager opens with the previously selected pane (in this test, Plugins).
Flags: in-litmus?
Attached patch Patch (v1) (deleted) — Splinter Review
Trivial patch to address this problem. Takes care of both cases when the add-on manager window is already open or a new window should be opened.
Assignee: nobody → ehsan.akhgari
Status: NEW → ASSIGNED
Attachment #292328 - Flags: review?(mano)
Target Milestone: --- → Firefox 3 M11
Why should we always be switching to the Extension's pane when opening from here but not when opening from the Tools menu?
Target Milestone: Firefox 3 M11 → ---
Target Milestone: --- → Firefox 3 M11
Well, bug 384956 implemented this new button in the Preferences window. The aim of that bug was to fulfill the ADD-002a and ADD-002b requirements <http://wiki.mozilla.org/Firefox3/Product_Requirements_Document#P2>. Because these requirements address the visibility of add-on configuration UI, and the only type of add-ons which can have configuration UI are extensions, I guess the "Manage Add-ons" button should be focusing on extensions, hence this bug and this patch. :-) If that impression is not valid, then I guess this bug is INVALID. Comments from Robert and Beltzner are welcome.
Blocks: 384956
Comment on attachment 292328 [details] [diff] [review] Patch (v1) r=mano
Attachment #292328 - Flags: review?(mano) → review+
Attachment #292328 - Flags: ui-review?(beltzner)
I agree that "Manage Add-ons" should always take you to the list of add-ons from where you can get to the properties. I think, though that if bug 404024 gets resolved, Tools > Add-ons should default to that new pane.
(In reply to comment #5) > I agree that "Manage Add-ons" should always take you to the list of add-ons > from where you can get to the properties. So, do you think the patch in attachment 292328 [details] [diff] [review] exposes the correct behavior? > I think, though that if bug 404024 gets resolved, Tools > Add-ons should > default to that new pane. This patch does not change the behavior of Tools > Add-ons. Its behavior is currently the last selected pane being active by default. And even if the new pane would be considered the default after bug 404024 lands, I think the "Manage Add-ons" button should still open the add-ons list.
Whiteboard: [needs ui-r beltzner]
Comment on attachment 292328 [details] [diff] [review] Patch (v1) Perhaps johnath can ui-r this?
Attachment #292328 - Flags: ui-review?(beltzner) → ui-review?(johnath)
Whiteboard: [needs ui-r beltzner] → [has patch][has review+][needs ui-r johnath]
Comment on attachment 292328 [details] [diff] [review] Patch (v1) Yeah, I think this is right.
Attachment #292328 - Flags: ui-review?(johnath) → ui-review+
Whiteboard: [has patch][has review+][needs ui-r johnath]
Comment on attachment 292328 [details] [diff] [review] Patch (v1) This is fairly trivial, asking for approval to land it.
Attachment #292328 - Flags: approval1.9?
Comment on attachment 292328 [details] [diff] [review] Patch (v1) Thanks!
Attachment #292328 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
Checking in browser/components/preferences/main.js; /cvsroot/mozilla/browser/components/preferences/main.js,v <-- main.js new revision: 1.16; previous revision: 1.15 done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
https://litmus.mozilla.org/show_test.cgi?id=5161 in-litmus+ Verified FIXED using: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008012504 Minefield/3.0b3pre Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9b3pre) Gecko/2008012504 Minefield/3.0b3pre -and- Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3pre) Gecko/2008012504 Minefield/3.0b3pre
Status: RESOLVED → VERIFIED
Flags: in-litmus? → in-litmus+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: