Closed Bug 964217 Opened 11 years ago Closed 11 years ago

Search item is broken in customization mode

Categories

(Firefox :: Toolbars and Customization, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 29

People

(Reporter: u428464, Assigned: Gijs)

References

(Blocks 1 open bug)

Details

(Keywords: regression, Whiteboard: [Australis:P3])

Attachments

(2 files)

I noticed a few bugs with the search bar in customization mode. 1. The item has no text in the palette 2. The box is cut on the right 3. The item can't be placed back to the right of the location bar because the other widgets are placed immediately there after the item is dropped.
Whiteboard: [Australis:P3]
At least the cut-off-ness is a regression...
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression
I tried to bisect this using mozregression and got nowhere. :-\
This was regressed by the flex code I added in bug 922834.
Assignee: nobody → gijskruitbosch+bugs
Blocks: 922834
Status: NEW → ASSIGNED
OS: Windows 7 → All
So this fixes the styling of the search bar and the bookmarks bar. There's still a separate issue that only occurs if moving the bookmarks toolbar items from the palette to the menu panel, but stops as soon as they've been in the toolbar. Don't think that needs to hold up this bug, though.
Attachment #8366667 - Flags: review?(mdeboer)
Comment on attachment 8366667 [details] [diff] [review] search item is broken in Australis' customize code, Review of attachment 8366667 [details] [diff] [review]: ----------------------------------------------------------------- r=me with the following comment addressed: ::: browser/themes/linux/browser.css @@ +118,5 @@ > } > > +#wrapper-personal-bookmarks:not([place="toolbar"]) > #personal-bookmarks { > + -moz-box-pack: center; > +} since you're adding the same block to all platforms' CSS, I'd like you to put this in customizeMode.inc.css
Attachment #8366667 - Flags: review?(mdeboer) → review+
Whiteboard: [Australis:P3] → [Australis:P3][leave open]
Attachment #8366667 - Flags: checkin+
3. is because of the search splitter state, which is getting confused. :-\
So basically, what happens is, customize mode removes the splitter, but then the overflowable toolbar gets disabled, which calls _moveItemsBackToOrigin, which then promptly calls UpdateUrlbarSearchSplitterState, which puts it back again. Let's just teach UpdateUrlbarSearchSplitterState (who picked that function name?) to Not Do That (tm).
Attachment #8369346 - Flags: review?(bmcbride)
Attachment #8369346 - Flags: review?(bmcbride) → review+
Whiteboard: [Australis:P3][leave open] → [Australis:P3][fixed-in-fx-team]
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P3][fixed-in-fx-team] → [Australis:P3]
Target Milestone: --- → Firefox 29
QA Contact: cornel.ionce
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:29.0) Gecko/20100101 Firefox/29.0 Mozilla/5.0 (X11; Linux i686; rv:29.0) Gecko/20100101 Firefox/29.0 All scenarios mentioned in description are now fixed on latest Firefox Aurora (Build ID: 20140317004002).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: