Closed
Bug 414836
Opened 17 years ago
Closed 17 years ago
Need to sync toolbar with icons/text preferences and icon size when new localstore.rdf created
Categories
(Firefox :: Toolbars and Customization, defect, P1)
Firefox
Toolbars and Customization
Tracking
()
RESOLVED
FIXED
Firefox 3 beta3
People
(Reporter: wgianopoulos, Assigned: dao)
References
Details
(Keywords: addon-compat, regression, relnote)
Attachments
(1 file)
(deleted),
patch
|
Gavin
:
review+
beltzner
:
approval1.9b3+
|
Details | Diff | Splinter Review |
In cases when the contents of an existing localstore.rdf file is not being used (with the exception of safe-mode) the toolbar icons/text preferences and icon size preferences need to be applied.
These cases include:
1. localstore.rdf is absent.
2. localstore.rdf is being ignored because it has been determined to be corrupt.
3. The code in bug 404109 is ignoring the localstore.rdf to reset the toolbars to default so that the home button displays on the bookmarks toolbar.
Not doing so results in an inconsistency between the displayed UI and what is selected in the preferences panel.
This problem is exacerbated by the code for bug 404109 which is slated for inclusion in Firefox beta3.
That is why i requested blocker and set target-milestone to beta3. Fell free to re-target as appropriate, I just wanted to make sure the appropriate people would be made aware of the issue before the beta3 release.
Flags: blocking-firefox3?
Assignee | ||
Comment 1•17 years ago
|
||
So, the easiest way to fix this would be to reset the toolbox as well, so that it falls back to the default mode and icon size. Another option is to read these attributes from the toolbox and explicitly apply them to the toolbars on startup.
Comment 2•17 years ago
|
||
Dao, it makes more sense to just reset everything at this point. Can you spin a quick fix for that?
Target Milestone: Firefox 3 beta3 → Firefox 3 beta4
Comment 4•17 years ago
|
||
(In reply to comment #0)
> Not doing so results in an inconsistency between the displayed UI and what is
> selected in the preferences panel.
What are the steps to reproduce the inconsistency? I'm not sure I understand the problem here.
Assignee | ||
Comment 5•17 years ago
|
||
Attachment #300504 -
Flags: review?(gavin.sharp)
Updated•17 years ago
|
Attachment #300504 -
Flags: review?(gavin.sharp)
Attachment #300504 -
Flags: review+
Attachment #300504 -
Flags: approval1.9b3?
Assignee | ||
Updated•17 years ago
|
Keywords: late-compat
Reporter | ||
Comment 6•17 years ago
|
||
(In reply to comment #4)
> (In reply to comment #0)
> > Not doing so results in an inconsistency between the displayed UI and what is
> > selected in the preferences panel.
>
> What are the steps to reproduce the inconsistency? I'm not sure I understand
> the problem here.
>
Well, first I have done more testing and the only way to get the inconsistency is upgrading form a pre bug 4041o9 version to a post 404109 version the corrupt localstore and absent locastore.rdf issues really do not exist.
Steps to reproduce:
1. launch Firefox beta 2 build with a fresh profile.
2. use customize toolbar to select small icons and no text.
3. Launch the 20070130 nightly using the same profile
4. Notice that large icons are displayed, yet when you go to customize toolabars, the use small icons checkbox is checked.
To get back to small icons, it is necessary to toggle the checkbox setting twice.
Comment 7•17 years ago
|
||
Comment on attachment 300504 [details] [diff] [review]
reset the toolbox
a=beltzner for beta 3, fixes issue from toolbar changes
Attachment #300504 -
Flags: approval1.9b3? → approval1.9b3+
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Target Milestone: Firefox 3 beta4 → Firefox 3 beta3
Updated•17 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Priority: -- → P1
Comment 8•17 years ago
|
||
Checking in browser/themes/gnomestripe/browser/browser.css;
/cvsroot/mozilla/browser/themes/gnomestripe/browser/browser.css,v <-- browser.css
new revision: 1.172; previous revision: 1.171
done
Checking in browser/themes/winstripe/browser/browser.css;
/cvsroot/mozilla/browser/themes/winstripe/browser/browser.css,v <-- browser.css
new revision: 1.163; previous revision: 1.162
done
Checking in browser/base/content/browser.css;
/cvsroot/mozilla/browser/base/content/browser.css,v <-- browser.css
new revision: 1.56; previous revision: 1.55
done
Checking in browser/base/content/browser.js;
/cvsroot/mozilla/browser/base/content/browser.js,v <-- browser.js
new revision: 1.956; previous revision: 1.955
done
Checking in browser/base/content/browser.xul;
/cvsroot/mozilla/browser/base/content/browser.xul,v <-- browser.xul
new revision: 1.422; previous revision: 1.421
done
Checking in browser/base/content/customizeToolbarSheet.js;
/cvsroot/mozilla/browser/base/content/customizeToolbarSheet.js,v <-- customizeToolbarSheet.js
new revision: 1.3; previous revision: 1.2
done
Updated•17 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Updated•17 years ago
|
Keywords: dev-doc-needed → relnote
You need to log in
before you can comment on or make changes to this bug.
Description
•