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)

All
macOS
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: alqahira, Assigned: alqahira)

References

()

Details

(Whiteboard: [camino-2.0.4])

Attachments

(1 file)

Attached patch FixSplinter 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 on attachment 464705 [details] [diff] [review] Fix Oops! sr=smorgan
Attachment #464705 - Flags: superreview?(stuart.morgan+bugzilla) → superreview+
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.

Attachment

General

Created:
Updated:
Size: