It's possible to accidentally scroll suggestions list down so that blank line appeared

VERIFIED FIXED in Firefox 52

Status

()

Firefox
Search
P3
minor
VERIFIED FIXED
a year ago
4 months ago

People

(Reporter: arni2033, Assigned: standard8)

Tracking

(Depends on: 1 bug, {regression})

Trunk
Firefox 52
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox44 affected, firefox45 affected, firefox52 verified)

Details

(Whiteboard: [fxsearch])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

a year ago
Created attachment 8691589 [details]
screenshot 1 - It's possible to accidentally scroll suggestions list down so that blank line appeared.png

>>>   My Info:   Win7_64, Nightly 45, 32bit, ID 20151120030227
STR:
1.   Focus the searchbar, type "a"   [suggestions list will appear]
2.A) Press Tab key, then Up key
2.B) Move mouse to suggestions list, scroll mousewheel down several times

Result:       
 A blank unselectable line appears in the end of suggestions list.
 If I click it, browser opens a web page with search results for "a"

Expectations: 
 There should be no blank lines in suggestions popup

I'm guessing that it happens because <treechildren> has incorrect height now. It displays 10 suggestions, each tree row has height == 18px, so expected height of <treechildren> in popup is 180px. But it is 179px. Probably that makes it scrollable
(Reporter)

Updated

a year ago
Severity: normal → minor
(Reporter)

Comment 1

a year ago
Oh, sorry.
It was regressed between 2015-10-28 and 2015-10-29 by bug 1173715. Pushlog url:
>   https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=6cf9efd579290fe27af7a964f20727f92578385d&tochange=b82942655a6364a50060fd310d3caae450ac005a
Keywords: regression
status-firefox44: --- → affected
Priority: -- → P3
(Reporter)

Updated

a year ago
See Also: → bug 1238181
Whiteboard: [fxsearch]
Duplicate of this bug: 1308524
(Assignee)

Updated

6 months ago
Assignee: nobody → standard8
Comment hidden (mozreview-request)
(Assignee)

Comment 4

6 months ago
I think the revised changes to border management should work fine - definitely look ok on Mac, I've just requested try builds to confirm on Linux/Windows.
Comment hidden (mozreview-request)
(Assignee)

Comment 6

6 months ago
For some reason Windows/Linux were missing the border at the bottom of the current search engine. I'm experimenting with adding !important to those rules, I've pushed the patch to mozreview & requested another try build.
(Assignee)

Comment 7

6 months ago
I've tested the try build changes and it all looks good to me now, though another pair of eyes on the edge cases would be good.

Comment 8

6 months ago
mozreview-review
Comment on attachment 8805195 [details]
Bug 1227734 - Rework how the top/bottom borders are managed in the search bar popup to avoid a blank search result entry when scrolling.

https://reviewboard.mozilla.org/r/88984/#review89634

Makes senses, thanks for figuring out a solution :-).

One case is not handled: websites offering open search plugins (eg https://bugzilla.mozilla.org/ ) when the suggestions tree is collapsed (eg after typing garbage in the search field so that the engine doesn't return any suggestion).
Attachment #8805195 - Flags: review?(florian) → review-
Comment hidden (mozreview-request)
(Assignee)

Comment 10

6 months ago
mozreview-review-reply
Comment on attachment 8805195 [details]
Bug 1227734 - Rework how the top/bottom borders are managed in the search bar popup to avoid a blank search result entry when scrolling.

https://reviewboard.mozilla.org/r/88984/#review89634

Thanks for catching that, I'd missed looking at the add engines section.

Comment 11

6 months ago
mozreview-review
Comment on attachment 8805195 [details]
Bug 1227734 - Rework how the top/bottom borders are managed in the search bar popup to avoid a blank search result entry when scrolling.

https://reviewboard.mozilla.org/r/88984/#review89706

Thanks! I tested only on Mac, but given comment 6 and comment 7 I'm confident you tested on the other 2 platforms.
Attachment #8805195 - Flags: review?(florian) → review+

Comment 12

6 months ago
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8c941a146208
Rework how the top/bottom borders are managed in the search bar popup to avoid a blank search result entry when scrolling. r=florian

Comment 13

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8c941a146208
Status: NEW → RESOLVED
Last Resolved: 6 months ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
I have reproduced this bug with Nightly 45.0a1 on Windows 8, 64 Bit !

This bug's fix is verified with latest Developer Edition (Aurora)

Build ID   : 20161118004020
User Agent : Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:52.0) Gecko/20100101 Firefox/52.0
[bugday-20161116]

Comment 15

5 months ago
I have reproduced this bug with Nightly 45.0a1 (2015-11-20) on Ubuntu 16.10, 64 Bit !

This bug's fix is verified with latest Developer Edition (Aurora)

Build ID   : 20161125004006
User Agent : Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0


[testday-20161125]
Status: RESOLVED → VERIFIED
status-firefox52: fixed → verified
(Reporter)

Updated

4 months ago
Depends on: 1327603
You need to log in before you can comment on or make changes to this bug.