Closed
Bug 974258
Opened 10 years ago
Closed 6 years ago
History shown in Location Bar is no longer constrained by urlbar width when using the drop-down arrow
Categories
(SeaMonkey :: Location Bar, defect)
Tracking
(firefox29 fixed, firefox30 fixed, seamonkey2.26 verified, seamonkey2.27 verified)
RESOLVED
FIXED
seamonkey2.27
People
(Reporter: rsx11m.pub, Assigned: neil)
Details
(Keywords: regression)
Attachments
(2 files)
92.01 KB,
image/png
|
Details | |
1.78 KB,
patch
|
philip.chee
:
review+
rsx11m.pub
:
feedback+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Steps to reproduce on aurora or trunk channels: 1. Optional: In the History preferences, click "Clear Location Bar" 2. Visit some page with a long URL (e.g., search something in bugzilla) 3. Click in the Location Bar and hit Enter to add it to its history 4. Now click on the arrow-down button to the left of the Location Bar 5. The list of suggested URLs spans across the full screen width Expected result: 5. The list is constrained by the width of the urlbar element. Screenshot follows for illustration. Note that the list for the regular auto-complete history (when just typing without clicking the arrow) isn't affected and remains properly constrained in width.
SM 2.25a2 (bottom) vs. 2.26a2 (top), also observed on current 2.27a1 build.
status-seamonkey2.26:
--- → affected
status-seamonkey2.27:
--- → affected
Comment 2•10 years ago
|
||
> 4. Now click on the arrow-down button to the left of the Location Bar
You mean to the "right" don't you?
Confirm I see this too.
Neil has something changed in toolkit or gecko?
Flags: needinfo?(neil)
(In reply to Philip Chee from comment #2) > You mean to the "right" don't you? Yes, autocomplete-history-dropmarker.
Assignee | ||
Comment 4•10 years ago
|
||
Well, the (Nothing Available) is supposed to stretch to the full width too...
Assignee | ||
Comment 5•10 years ago
|
||
Regression range: 4da3e21a0e5f::7e79536aca0a - bug 941409 looks suspicious, since it changed things so that IsSizedToPopup is only called for menus, which this isn't. The main autocomplete popup works around the problem by explicitly setting the width on the popup before opening it.
Flags: needinfo?(neil)
Assignee | ||
Comment 6•10 years ago
|
||
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #8378716 -
Flags: review?(philip.chee)
Attachment #8378716 -
Flags: feedback?(rsx11m.pub)
Assignee | ||
Comment 7•10 years ago
|
||
FYI the code is copied from the similar openPopup method.
Keywords: regressionwindow-wanted
Comment on attachment 8378716 [details] [diff] [review] Workaround Solves it for me with a quick test on trunk. :-)
Attachment #8378716 -
Flags: feedback?(rsx11m.pub) → feedback+
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8378716 [details] [diff] [review] Workaround >- <content sizetopopup="pref"> >+ <content> (We should also do this to urlbarBindings.xml)
Comment 10•10 years ago
|
||
Comment on attachment 8378716 [details] [diff] [review] Workaround r=me
Attachment #8378716 -
Flags: review?(philip.chee) → review+
Comment 11•10 years ago
|
||
Something in this push was causing mochitest-other failures. Backed out. https://hg.mozilla.org/integration/mozilla-inbound/rev/8fb083fffb02 https://tbpl.mozilla.org/php/getParsedLog.php?id=35063654&tree=Mozilla-Inbound
Comment 12•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/676016cd743d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.27
Assignee | ||
Comment 13•10 years ago
|
||
Comment on attachment 8378716 [details] [diff] [review] Workaround [Approval Request Comment] Bug caused by (feature/regressing bug #): 941409 Risk to taking this patch (and alternatives if risky): None; code path is only used by SeaMonkey String or IDL/UUID changes made by this patch: None
Attachment #8378716 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•10 years ago
|
Whiteboard: [don't autoland]
Updated•10 years ago
|
Attachment #8378716 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•10 years ago
|
status-firefox29:
--- → affected
status-firefox30:
--- → fixed
Assignee | ||
Comment 14•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/d437f5ed2715
Whiteboard: [don't autoland]
Reporter | ||
Comment 15•10 years ago
|
||
Verified fixed in current 2.27a1 and 2.26a2 Windows nightly builds. Thanks!
Status: RESOLVED → VERIFIED
Comment 16•6 years ago
|
||
Pushed by frgrahl@gmx.net: https://hg.mozilla.org/comm-central/rev/e34ccb9162d1 Manually size autocomplete history popup to the autocomplete's width r=Ratty https://hg.mozilla.org/comm-central/rev/532337d61c38 Manually size autocomplete history popup to the autocomplete's width r=Ratty
Status: VERIFIED → RESOLVED
Closed: 10 years ago → 6 years ago
You need to log in
before you can comment on or make changes to this bug.
Description
•