Closed
Bug 1181868
Opened 9 years ago
Closed 9 years ago
Typing a query that results in no suggestions causes suggestions to break until Firefox is restarted
Categories
(Firefox :: Search, defect)
Firefox
Search
Tracking
()
VERIFIED
FIXED
Firefox 43
People
(Reporter: nhnt11, Assigned: florian)
References
Details
(Keywords: regression)
Attachments
(1 obsolete file)
STR:
1. Type a long query in the search bar until there are no suggestions.
2. Delete it all, and start typing something.
3. No more suggestions!
This regression is reproducible in Nightly, but has not made it to aurora or beta yet.
Assignee | ||
Updated•9 years ago
|
Keywords: regressionwindow-wanted
Updated•9 years ago
|
Attachment #8631460 -
Attachment is obsolete: true
Comment 2•9 years ago
|
||
sorry spam
Assignee | ||
Comment 3•9 years ago
|
||
[Tracking Requested - why for this release]: serious regression in the primary search UI.
status-firefox40:
--- → unaffected
status-firefox41:
--- → unaffected
tracking-firefox42:
--- → ?
Keywords: regression
Comment 4•9 years ago
|
||
I couldn't reproduce with Nightly 42.0a1 - 20150708030204 and 20150721030212 under Win 7 64-bit with Google and Yahoo set as default search engines.
What search engine were you using? Is it still reproducible for you? Thank you!
Flags: needinfo?(nhnt11)
Reporter | ||
Comment 5•9 years ago
|
||
Weird, I can't reproduce it in an official nightly build, but can reproduce in my local build.
Flags: needinfo?(nhnt11)
Reporter | ||
Comment 6•9 years ago
|
||
I can reproduce on an official nightly when trying on a retina display. The issue persists when I move the window back to my external screen.
adw and I were messing with the size of the searchbar and found that increasing its size makes suggestions appear again, but the popup doesn't expand correctly and only 1 and a half suggestions are visible.
Seems like a weird popup issue...
Reporter | ||
Comment 7•9 years ago
|
||
FYI I'm testing on a Macbook Pro (hence the retina display ;)), I can reproduce with Google and Yahoo! (didn't try others).
Comment 8•9 years ago
|
||
Thanks for the info, I could reproduce it too on a Macbook Pro retina.
Here is the mozilla-central pushlog:
Last good revision: 5787d784485e (2015-06-10)
First bad revision: bfd82015df48 (2015-06-11)
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5787d784485e&tochange=bfd82015df48
mozilla-inboud:
Last good revision: 82322f431ac1
First bad revision: bfd82015df48
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=82322f431ac1&tochange=bfd82015df48
There are several bugs here that modified the search bar, maybe Bug 1158295 or Bug 1170500 broke this.
Keywords: regressionwindow-wanted
Comment 10•9 years ago
|
||
Thanks Petruta! Bisected this locally to confirm:
The first bad revision is:
changeset: 247959:612e73e50be6
user: Florian Quèze <florian@queze.net>
date: Tue Jun 02 12:54:21 2015 +0200
summary: Bug 1170500 - [Mac] Broken one-off search items layout. r=Gijs
Firefox 41 is affected as well.
Assignee | ||
Comment 11•9 years ago
|
||
(In reply to :Gavin Sharp [email: gavin@gavinsharp.com] from comment #10)
> Thanks Petruta! Bisected this locally to confirm:
>
> The first bad revision is:
> changeset: 247959:612e73e50be6
> user: Florian Quèze <florian@queze.net>
> date: Tue Jun 02 12:54:21 2015 +0200
> summary: Bug 1170500 - [Mac] Broken one-off search items layout. r=Gijs
Nope, this revision is indeed bad, but it's just fixing a more obvious brokenness, and if I update my local tree to before bug 459563, this bug was already there.
My local bisect points to the first bad revision being https://hg.mozilla.org/mozilla-central/rev/0bb094a070d4 / bug 1108648.
Assignee | ||
Comment 12•9 years ago
|
||
Given that this bug can only be reproduced on retina screens, it could be that the underlying issue is bug 1179234.
Assignee | ||
Comment 13•9 years ago
|
||
I still don't fully understand the reason for this bug, but I suspect a rounding issue somewhere is somehow messing with the sizing of the tree. I have a patch attached in bug 1179234 that fixes this bug by making the margin-bottom of the header at the top of the panel (right above the tree) 2px instead of 1.
Assignee | ||
Comment 15•9 years ago
|
||
Fixed in bug 1179234.
Assignee: nobody → florian
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Comment 16•9 years ago
|
||
Setting flags to make sure this also gets verified along with bug 1179234.
status-firefox43:
--- → fixed
Flags: qe-verify+
Comment 17•9 years ago
|
||
Verified that search suggestions are displayed again after introducing and then deleting a long query using Firefox 41 beta 2, Developer Edition 42.0a2 and Nightly 43.0a1 both 2015-08-19 under MacBook Pro retina 10.9.5.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•