Closed
Bug 586182
Opened 15 years ago
Closed 15 years ago
BWC's openFeedPrefPane: still wants to load org.mozilla.camino.preference.navigation
Categories
(Camino Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alqahira, Assigned: alqahira)
References
()
Details
(Whiteboard: [camino-2.0.4])
Attachments
(1 file)
|
923 bytes,
patch
|
stuart.morgan+bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
Somehow this was missed when we cleaned up the Navigation->General stuff æons ago, yet somehow it still works (if a pane can't be found, we fall back to General, maybe?).
We should really fix it, though; that's the only reference to the that pane by that name that's left.
I notice a slight difference in behavior with my patch (with another pane selected before invoking the Feed Preferences… item in the feed icon's menu):
* Before the patch, the current pane completely loads, and then we switch to General
* After the patch, I can barely see a flash of the "selected" state around the current pane's icon before General loads (the current pane never loads; we just see the "selected" state being changed very quickly).
This seems to be an improvement, however slight ;)
Attachment #464705 -
Flags: superreview?(stuart.morgan+bugzilla)
Comment 1•15 years ago
|
||
Comment on attachment 464705 [details] [diff] [review]
Fix
Oops! sr=smorgan
Attachment #464705 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
| Assignee | ||
Comment 2•15 years ago
|
||
http://hg.mozilla.org/camino/rev/f06cab8894ac and CAMINO_2_0_BRANCH.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: camino2.0.4+
Resolution: --- → FIXED
Whiteboard: [camino-2.0.4]
You need to log in
before you can comment on or make changes to this bug.
Description
•