Closed
Bug 964217
Opened 11 years ago
Closed 11 years ago
Search item is broken in customization mode
Categories
(Firefox :: Toolbars and Customization, defect)
Firefox
Toolbars and Customization
Tracking
()
VERIFIED
FIXED
Firefox 29
People
(Reporter: u428464, Assigned: Gijs)
References
(Blocks 1 open bug)
Details
(Keywords: regression, Whiteboard: [Australis:P3])
Attachments
(2 files)
(deleted),
patch
|
mikedeboer
:
review+
Gijs
:
checkin+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Unfocused
:
review+
|
Details | Diff | Splinter Review |
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.
Blocks: australis-cust
Whiteboard: [Australis:P3]
Assignee | ||
Comment 1•11 years ago
|
||
At least the cut-off-ness is a regression...
Assignee | ||
Comment 2•11 years ago
|
||
I tried to bisect this using mozregression and got nowhere. :-\
Assignee | ||
Comment 3•11 years ago
|
||
This was regressed by the flex code I added in bug 922834.
Assignee | ||
Comment 4•11 years ago
|
||
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 5•11 years ago
|
||
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+
Assignee | ||
Comment 6•11 years ago
|
||
remote: https://hg.mozilla.org/integration/fx-team/rev/849724d4081e
Leaving open for point 3 from comment 0.
Whiteboard: [Australis:P3] → [Australis:P3][leave open]
Assignee | ||
Updated•11 years ago
|
Attachment #8366667 -
Flags: checkin+
Comment 7•11 years ago
|
||
Assignee | ||
Comment 9•11 years ago
|
||
3. is because of the search splitter state, which is getting confused. :-\
Assignee | ||
Comment 10•11 years ago
|
||
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)
Updated•11 years ago
|
Attachment #8369346 -
Flags: review?(bmcbride) → review+
Assignee | ||
Comment 11•11 years ago
|
||
Whiteboard: [Australis:P3][leave open] → [Australis:P3][fixed-in-fx-team]
Comment 12•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P3][fixed-in-fx-team] → [Australis:P3]
Target Milestone: --- → Firefox 29
Updated•11 years ago
|
QA Contact: cornel.ionce
Comment 13•11 years ago
|
||
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.
Description
•