Closed
Bug 1104748
Opened 10 years ago
Closed 10 years ago
A long default search provider name can mess up the layout of the one-off buttons when the panel is small
Categories
(Firefox :: Search, defect)
Tracking
()
People
(Reporter: avaida, Assigned: florian)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Felipe
:
review+
lmandel
:
approval-mozilla-beta+
lmandel
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
Reproducible on: Firefox 34.0 (build1 / 20141124205320).
Affected platform(s): Windows 7 64-bit, Windows 8 32-bit, Ubuntu 12.04 LTS 32-bit and Mac OS X 10.9.5.
STR:
1. Launch Firefox with a clean profile.
2. Click the magnifying glass icon from the search bar → choose "Change Search Settings".
3. Click "Add more search providers...".
4. Install two additional search providers from that list and restart the browser if asked.
5. Access https://bugzilla.mozilla.org from the Location Bar.
6. Click the magnifying glass icon from the search bar → choose "Add Bugzilla@Mozilla Search".
7. Click the menu button → go to "Search" → set the "Default Search Engine" to "Bugzilla@Mozilla".
8. Make the width of the browser window smaller and type in the Search Bar a random search term (e.g. example).
Expected result: The search pane's content is displayed properly in case of narrow browser windows.
Actual result: The search pane's content is cutoff by it's margin and there's no ellipsis used for longer strings.
Screenshot: http://i.imgur.com/23VNO8I.png
Comment 1•10 years ago
|
||
Possible duplicate of bug # 1104325
Assignee | ||
Updated•10 years ago
|
Summary: Resizing the window after installing additional search providers results in a cutoff search pane → A long default search provider name can mess up the layout of the one-off buttons when the panel is small
Assignee | ||
Comment 2•10 years ago
|
||
Assignee: nobody → florian
Attachment #8528509 -
Flags: review?(felipc)
Attachment #8528509 -
Flags: approval-mozilla-release?
Attachment #8528509 -
Flags: approval-mozilla-beta?
Updated•10 years ago
|
Attachment #8528509 -
Flags: review?(felipc) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Screenshot without patch: http://i.imgur.com/oGhJeog.png and with the patch: http://i.imgur.com/9rUkMEQ.png
Updated•10 years ago
|
status-firefox35:
--- → affected
status-firefox36:
--- → affected
tracking-firefox34:
--- → +
tracking-firefox35:
--- → +
tracking-firefox36:
--- → +
Flags: firefox-backlog?
Flags: qe-verify+
Comment 4•10 years ago
|
||
Comment on attachment 8528509 [details] [diff] [review]
Fix
Reviewed with Florian. Release+ Beta+
Attachment #8528509 -
Flags: approval-mozilla-release?
Attachment #8528509 -
Flags: approval-mozilla-release+
Attachment #8528509 -
Flags: approval-mozilla-beta?
Attachment #8528509 -
Flags: approval-mozilla-beta+
Comment 5•10 years ago
|
||
Comment 6•10 years ago
|
||
Reporter | ||
Comment 7•10 years ago
|
||
Verified fixed on Firefox 34.0 (build2 / 20141125180439) using Windows 7 64-bit, Mac OS X 10.9.5 and Ubuntu 12.04 LTS 32-bit.
Although most of the pane's elements now react properly in this scenario, the "Change Search Settings" button is still showing this issue:
- the button's label is still being cutoff (on all platforms),
- on Linux, the bottom border of the button is completely missing.
Here's a screenshot that shows both of these side-effects: http://i.imgur.com/IHmoQX8.png.
Perhaps this should be treated in a separate bug. Florian, what do you think?
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(florian)
Resolution: --- → FIXED
Comment 8•10 years ago
|
||
Hi Florian, can you provide a point value.
Iteration: --- → 37.1
Flags: needinfo?(florian)
Flags: firefox-backlog?
Flags: firefox-backlog+
Assignee | ||
Comment 9•10 years ago
|
||
(In reply to Andrei Vaida, QA [:avaida] from comment #7)
> Verified fixed on Firefox 34.0 (build2 / 20141125180439) using Windows 7
> 64-bit, Mac OS X 10.9.5 and Ubuntu 12.04 LTS 32-bit.
>
> Although most of the pane's elements now react properly in this scenario,
> the "Change Search Settings" button is still showing this issue:
> - the button's label is still being cutoff (on all platforms),
> - on Linux, the bottom border of the button is completely missing.
> Here's a screenshot that shows both of these side-effects:
> http://i.imgur.com/IHmoQX8.png.
>
> Perhaps this should be treated in a separate bug. Florian, what do you think?
Yes, separate bug (if we think it needs fixing). I tested this yesterday and discussed it with Felipe. We agreed that the "Change Search Settings" button wasn't worth fixing for beta (I'm not even sure that cropping the text with an ellipsis would actually be better that the current cut-off text) because it doesn't mess up the layout of the rest of the panel like the header did.
Leaving this open because I expect to land the fix for Firefox 36 too.
Status: RESOLVED → REOPENED
Points: --- → 1
Flags: needinfo?(florian)
Resolution: FIXED → ---
Assignee | ||
Comment 10•10 years ago
|
||
Whiteboard: [fixed-in-fx-team]
Comment 11•10 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 36
Assignee | ||
Comment 12•10 years ago
|
||
Reporter | ||
Comment 13•10 years ago
|
||
Verified fixed on Nightly 37.0a1 (2014-12-07) and Aurora 36.0a2 (2014-12-08) as well, using Windows 7 64-bit, Ubuntu 12.04 LTS 32-bit and Mac OS X 10.9.5.
Reporter | ||
Comment 14•10 years ago
|
||
Verified fixed on Firefox 35.0b6 (20141222200458) using Ubuntu 12.04 LTS 32-bit, Windows 8.1 64-bit and Mac OS X 10.9.5.
You need to log in
before you can comment on or make changes to this bug.
Description
•