Some URLs in autocomplete items are sometimes unnecessarily getting a fading effect, if Quantumbar is on
Categories
(Firefox :: Address Bar, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox67 | --- | unaffected |
firefox68 | --- | fixed |
firefox69 | --- | verified |
People
(Reporter: itiel_yn8, Assigned: dao)
References
(Regression)
Details
(Keywords: regression)
Attachments
(6 files)
Windows 10, latest Nightly, browser.urlbar.quantumbar = true
I'm pretty sure this is something new (from the past week or so).
I'm unable to reproduce this on a new profile, but am able to reproduce consistently on my main profile.
Sometimes some URLs for autocomplete items will get the fading effect even when not unnecessary.
For this to reproduce I must type slowly in the URL bar (i.e. wait for results to appear after each key stroke)
See attached.
Comment 3•6 years ago
|
||
I cannot reproduce this on my Win10... do you have any userchrome.css or chrome/ modifications in your profile?
Does adding/removing the search bar make any difference?
Updated•6 years ago
|
(In reply to Marco Bonardo [::mak] from comment #3)
do you have any userchrome.css or chrome/ modifications in your profile?
No userchrome.css or similar. What kind of modifications are you looking for? Just the normal stuff you can do from the Customize page.
Does adding/removing the search bar make any difference?
No :(
This reproduces here for both RTL and LTR btw.
Comment 6•6 years ago
|
||
There must be some difference between your profile and the clean profile where you say this doesn't happen... Could you try copying places.sqlite from your profile to the clean one and try the same identical search? Could you please try to customize the new profile toolbars identical to the default profile and see if you can cause this bug in that profile?
I'm trying to identify a cause that would allow me to reproduce.
(In reply to Marco Bonardo [::mak] from comment #6)
There must be some difference between your profile and the clean profile where you say this doesn't happen... Could you try copying places.sqlite from your profile to the clean one and try the same identical search? Could you please try to customize the new profile toolbars identical to the default profile and see if you can cause this bug in that profile?
All done (plus unticked the "Show search suggestions ahead of browsing history in address bar results" preference to match my main profile), still this works in the new profile.
What files should be related to the URL bar functionality that I can copy from the old profile over to the new profile and re-test?
I've already tried copying prefs.js (resulted in a somewhat corrupted profile) and favicons.sqlite (not really related but I'm out of ideas).
Comment 8•6 years ago
|
||
maybe try attaching here a log from about:support, I may get ideas. In general this kind of things may be related to chrome style changes, or graphics/layout settings. It is strange anyway, because it looks like we are miscalculating the size of the flexbox, that would be a layout bug...
Updated•6 years ago
|
Comment 10•6 years ago
|
||
The only relevant thing I didn't try yet, is that this is an "he" nightly build, though I don't know how that should make a difference compared to the new profile... did you change some text direction of font related prefs?
I guess I should try this localized build.
Updated•6 years ago
|
Reporter | ||
Comment 11•6 years ago
|
||
(In reply to Marco Bonardo [::mak] from comment #10)
The only relevant thing I didn't try yet, is that this is an "he" nightly build, though I don't know how that should make a difference compared to the new profile...
Both profiles (the one being my main profile and the second is the new one where I can't reproduce the issue) are Hebrew build, using the same installation of Nightly. So I can't see it being the cause here.
did you change some text direction of font related prefs?
None that I can remember.
Comment 12•6 years ago
|
||
I tried the he build, and still cannot reproduce, I am running out of ideas :(
Do you have any prefs not at default value under layout.*?
what's value for layout.css.devPixelsPerPx and layout.css.dpi?
Reporter | ||
Comment 13•6 years ago
|
||
layout.spellcheckDefault = 0
layout.css.devPixelsPerPx = -1.0
layout.css.dpi = -1
Windows DPI is set to 150% btw.
Comment 14•6 years ago
|
||
Please, let us know if the next nightly including bug 1550036 solves your problem.
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Reporter | ||
Comment 15•6 years ago
|
||
Not sure if the fix for bug 1550036 is included in the current Nightly (about:build config leads me to https://hg.mozilla.org/mozilla-central/rev/34a824c75b7b5618a06ba8987c418d6363da5038), but if it is- I'm still seeing the issue here.
Assignee | ||
Comment 16•6 years ago
|
||
(In reply to Itiel from comment #15)
Not sure if the fix for bug 1550036 is included in the current Nightly (about:build config leads me to https://hg.mozilla.org/mozilla-central/rev/34a824c75b7b5618a06ba8987c418d6363da5038),
The fix is included there.
Since we can't reproduce this, could you help us find the regression range?
Reporter | ||
Comment 17•6 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #16)
(In reply to Itiel from comment #15)
Not sure if the fix for bug 1550036 is included in the current Nightly (about:build config leads me to https://hg.mozilla.org/mozilla-central/rev/34a824c75b7b5618a06ba8987c418d6363da5038),
The fix is included there.
Since we can't reproduce this, could you help us find the regression range?
I tried at first (by reusing my main profile to bisect with Mozregression) but the profile kept on becoming corrupted.
Anyway, I've tried bisecting again but this time a bit farther back in time and I think I got it right:
2019-05-09T19:57:04: DEBUG : Starting merge handling...
2019-05-09T19:57:04: DEBUG : Using url: https://hg.mozilla.org/integration/autoland/json-pushes?changeset=29c131c2fd402f750064800c41037f534cea5cfd&full=1
2019-05-09T19:57:06: DEBUG : Found commit message:
Bug 1545394 - Keep stale rows in the view while receiving new results. r=mak
Differential Revision: https://phabricator.services.mozilla.com/D28049
2019-05-09T19:57:06: DEBUG : Did not find a branch, checking all integration branches
2019-05-09T19:57:06: INFO : The bisection is done.
Assignee | ||
Comment 18•6 years ago
|
||
Thanks!
Assignee | ||
Comment 19•6 years ago
|
||
(In reply to Marco Bonardo [::mak] from comment #6)
All done (plus unticked the "Show search suggestions ahead of browsing history in address bar results" preference to match my main profile), still this works in the new profile.
How about the reverse? Can you reproduce this in your main profile if you reset that pref?
Reporter | ||
Comment 20•6 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #19)
(In reply to Marco Bonardo [::mak] from comment #6)
All done (plus unticked the "Show search suggestions ahead of browsing history in address bar results" preference to match my main profile), still this works in the new profile.How about the reverse? Can you reproduce this in your main profile if you reset that pref?
Yes.
Reporter | ||
Comment 21•6 years ago
|
||
Assignee | ||
Comment 22•5 years ago
|
||
I've finally seen this happen with my profile, with search suggestions disabled. It's quite intermittent and I can't easily reproduce it by just doing the same query again.
Reporter | ||
Comment 23•5 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #22)
I've finally seen this happen with my profile, with search suggestions disabled. It's quite intermittent and I can't easily reproduce it by just doing the same query again.
This happens to me sometimes as well (though with search suggestions on), but I can bump the chances to see this by opening many tabs (with different urls in each), in a new tab typing "%" followed by any character and hitting Ctrl (to see the urls). In one of the next tries the issue reproduces.
Assignee | ||
Comment 24•5 years ago
|
||
(In reply to Itiel from comment #23)
(In reply to Dão Gottwald [::dao] from comment #22)
I've finally seen this happen with my profile, with search suggestions disabled. It's quite intermittent and I can't easily reproduce it by just doing the same query again.
This happens to me sometimes as well (though with search suggestions on), but I can bump the chances to see this by opening many tabs (with different urls in each), in a new tab typing "%" followed by any character and hitting Ctrl (to see the urls). In one of the next tries the issue reproduces.
That doesn't seem to do the trick over here.
Assignee | ||
Comment 25•5 years ago
|
||
Cristian, could you help us trying to reproduce this reliably?
Reporter | ||
Comment 26•5 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #24)
(In reply to Itiel from comment #23)
(In reply to Dão Gottwald [::dao] from comment #22)
I've finally seen this happen with my profile, with search suggestions disabled. It's quite intermittent and I can't easily reproduce it by just doing the same query again.
This happens to me sometimes as well (though with search suggestions on), but I can bump the chances to see this by opening many tabs (with different urls in each), in a new tab typing "%" followed by any character and hitting Ctrl (to see the urls). In one of the next tries the issue reproduces.
That doesn't seem to do the trick over here.
If needed, I'm willing to upload my profile for you to test.
Assignee | ||
Comment 27•5 years ago
|
||
(In reply to Itiel from comment #26)
If needed, I'm willing to upload my profile for you to test.
Feel free to send it to my email address (without +bmo) and I'll give it a try, but I suspect we have a race condition here and it not only depends on your profile but also on your system.
Assignee | ||
Updated•5 years ago
|
Reporter | ||
Comment 28•5 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #27)
(In reply to Itiel from comment #26)
If needed, I'm willing to upload my profile for you to test.
Feel free to send it to my email address (without +bmo) and I'll give it a try, but I suspect we have a race condition here and it not only depends on your profile but also on your system.
Email sent.
Assignee | ||
Comment 29•5 years ago
|
||
(In reply to Itiel from comment #28)
(In reply to Dão Gottwald [::dao] from comment #27)
(In reply to Itiel from comment #26)
If needed, I'm willing to upload my profile for you to test.
Feel free to send it to my email address (without +bmo) and I'll give it a try, but I suspect we have a race condition here and it not only depends on your profile but also on your system.
Email sent.
Got it, thanks! Unfortunately, I still can't seem to reproduce this with your steps from comment 23. Tried different window sizes too. This is on Ubuntu, I can try Windows 10 next.
Reporter | ||
Comment 30•5 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #29)
Got it, thanks! Unfortunately, I still can't seem to reproduce this with your steps from comment 23. Tried different window sizes too. This is on Ubuntu, I can try Windows 10 next.
Looking back at comment 23, I guess I omitted a few steps:
- Start Nightly using my profile, with all the 370~ tabs, in full screen
- Open a new tab
- Type "%" followed by any character
- Hit Ctrl and see that one of the entries URL are unnecessarily faded out
- If none of them are (happens to me sometimes), repeat step 3 with a different character
- Usually when you can reproduce step 4, you can then clear the url bar text and start typing "9gag" or "google" and then you'll see the autocomplete for some 9gag.com entries and for translate.google.co.il being unnecessarily faded out.
If you're still unable to reproduce, can I do something over here to debug this for you?
Assignee | ||
Comment 31•5 years ago
|
||
Alright, I can reproduce now with your profile and with those steps on Windows 10.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 32•5 years ago
|
||
I have a patch that seems to fix this. Itiel, could you please check if you can reproduce the bug with this build? https://queue.taskcluster.net/v1/task/Dl099TIEQYGrIppHVbkPYQ/runs/0/artifacts/public/build/target.zip
Reporter | ||
Comment 33•5 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #32)
I have a patch that seems to fix this. Itiel, could you please check if you can reproduce the bug with this build? https://queue.taskcluster.net/v1/task/Dl099TIEQYGrIppHVbkPYQ/runs/0/artifacts/public/build/target.zip
Not reproducing on this build running with the same profile, also in RTL :)
Assignee | ||
Comment 34•5 years ago
|
||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 35•5 years ago
|
||
Comment 36•5 years ago
|
||
bugherder |
Comment 38•5 years ago
|
||
Thank you, Itiel!
Assignee | ||
Comment 39•5 years ago
|
||
Comment on attachment 9068354 [details]
Bug 1549787 - Hide row elements using visibility:collapse instead of display:none since the latter can confuse the overflow state of title and url elements.
Beta/Release Uplift Approval Request
- User impact if declined: see comment 0
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Medium
- Why is the change risky/not risky? (and alternatives if risky): reasonably straightforward patch, contains some refactoring making this code easier to maintain
- String changes made/needed:
Updated•5 years ago
|
Comment 40•5 years ago
|
||
Comment on attachment 9068354 [details]
Bug 1549787 - Hide row elements using visibility:collapse instead of display:none since the latter can confuse the overflow state of title and url elements.
quantum bar fix for 68.0b8
Comment 41•5 years ago
|
||
bugherder uplift |
Updated•3 years ago
|
Description
•