Closed
Bug 423770
Opened 17 years ago
Closed 17 years ago
sometimes need to click "Restore default set" twice to actually have the default set restored
Categories
(Firefox :: Toolbars and Customization, defect, P2)
Firefox
Toolbars and Customization
Tracking
()
VERIFIED
FIXED
Firefox 3 beta5
People
(Reporter: Gavin, Assigned: dao)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Gavin
:
review+
beltzner
:
approval1.9b5+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
I think this is probably a regression from bug 414864, but I haven't confirmed it.
STR:
1) default profile, move the stop button the the personal toolbar
2) click "Restore default set"
The stop button will disappear. Click "restore defaults" again, and it will be properly replaced in the navigation toolbar.
I think what's happening is that we are restoring the default set of the navigation toolbar first (because we restore toolbars in document order), and when it tries to add the reload button, it hits http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/toolkit/content/widgets/toolbar.xml&rev=1.45&mark=226#195 , because the element is present in the document.
Dao, can we revert the patch for bug 414864 now that we've undone the ID changes (bug 415099)? I don't exactly recall why we needed that patch in the first place.
Updated•17 years ago
|
Flags: blocking-firefox3?
Priority: -- → P2
Assignee | ||
Comment 1•17 years ago
|
||
(In reply to comment #0)
> I don't exactly recall why we needed that patch in the
> first place.
Because as you say we changed the id of the navigation toolbar. If you put an element from that toolbar's defaultset on the menu bar, where the id hasn't been changed, you get the element twice. It still seems wrong that toolbar.xml can do that, but if the fix caused this bug, we should take it out.
Assignee | ||
Comment 2•17 years ago
|
||
I'll check if reverting the patch fixes this.
Assignee: nobody → dao
Blocks: 414864
Assignee | ||
Updated•17 years ago
|
Status: NEW → ASSIGNED
Component: General → Toolbars
QA Contact: general → toolbars
Assignee | ||
Comment 3•17 years ago
|
||
Attachment #310627 -
Flags: review?(gavin.sharp)
Reporter | ||
Comment 4•17 years ago
|
||
Comment on attachment 310627 [details] [diff] [review]
patch
Thanks Dao.
Attachment #310627 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Updated•17 years ago
|
Attachment #310627 -
Flags: approval1.9b5?
Attachment #310627 -
Flags: approval1.9?
Updated•17 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Comment 5•17 years ago
|
||
Comment on attachment 310627 [details] [diff] [review]
patch
a=beltzner
Attachment #310627 -
Flags: approval1.9b5?
Attachment #310627 -
Flags: approval1.9b5+
Attachment #310627 -
Flags: approval1.9?
Attachment #310627 -
Flags: approval1.9+
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Comment 6•17 years ago
|
||
Checking in toolkit/content/widgets/toolbar.xml;
/cvsroot/mozilla/toolkit/content/widgets/toolbar.xml,v <-- toolbar.xml
new revision: 1.46; previous revision: 1.45
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 beta5
Comment 7•17 years ago
|
||
This now works for me with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b5pre) Gecko/2008032204 Minefield/3.0b5pre . Tried moving stop button and home button around, and they now revert with one click as they should.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•