Closed Bug 1397285 Opened 7 years ago Closed 7 years ago

Location bar and Home button jump to left when page start to load

Categories

(Firefox :: Theme, defect, P1)

57 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 57
Iteration:
57.3 - Sep 19
Tracking Status
firefox-esr52 --- unaffected
firefox55 --- unaffected
firefox56 --- unaffected
firefox57 --- verified
firefox58 --- verified

People

(Reporter: alice0775, Assigned: dao)

References

(Blocks 1 open bug)

Details

(Keywords: regression, Whiteboard: [reserve-photon-visual])

Attachments

(2 files)

Attached video screencast
Reproducible: always

Steps To Reproduce:
1. open page

Actual Results
Toolbutton jumps to left
See attached screen cast.
Whiteboard: [photon-structure] → [photon-animation][triage]
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=76a7ead2334c65a6420038919047dce69d040166&tochange=5f580744fe7ae5350f6278ce70749a5696f8d4a5

Regresed by:
5f580744fe7a	Dão Gottwald — Bug 1396624 - Hide URL bar elements and the stop button with display:none instead of visibility:collapse to prevent the icons from being loaded needlessly. r=johannh
Whiteboard: [photon-animation][triage] → [photon-visual][triage]
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Component: Toolbars and Customization → Theme
Flags: qe-verify+
OS: Windows 10 → All
Priority: -- → P1
Hardware: Unspecified → All
Whiteboard: [photon-visual][triage] → [reserve-photon-visual]
(In reply to Dão Gottwald [::dao] from comment #2)
> Created attachment 8905056 [details]
> Bug 1397285 - Set standard toolbarbutton-1 icon size as width rather than
> max-width to reserve space for icons while they're loading.

Is this patch also fixing bug 1394914?
Iteration: --- → 57.3 - Sep 19
QA Contact: ovidiu.boca
(In reply to Florian Quèze [:florian] [:flo] from comment #3)
> (In reply to Dão Gottwald [::dao] from comment #2)
> > Created attachment 8905056 [details]
> > Bug 1397285 - Set standard toolbarbutton-1 icon size as width rather than
> > max-width to reserve space for icons while they're loading.
> 
> Is this patch also fixing bug 1394914?

It seems like it might fix that, but I don't know how exactly the sidebar button is special, i.e. with how much of a delay it's added (and why). If we first paint without the <toolbarbutton> being in the toolbar and then the button gets added (!= the icon gets loaded), then there's no way this will fix bug 1394914.
Blocks: 1394914
(In reply to Dão Gottwald [::dao] from comment #4)
> (In reply to Florian Quèze [:florian] [:flo] from comment #3)
> > (In reply to Dão Gottwald [::dao] from comment #2)
> > > Created attachment 8905056 [details]
> > > Bug 1397285 - Set standard toolbarbutton-1 icon size as width rather than
> > > max-width to reserve space for icons while they're loading.
> > 
> > Is this patch also fixing bug 1394914?
> 
> It seems like it might fix that, but I don't know how exactly the sidebar
> button is special, i.e. with how much of a delay it's added (and why). If we
> first paint without the <toolbarbutton> being in the toolbar and then the
> button gets added (!= the icon gets loaded), then there's no way this will
> fix bug 1394914.

I can only reproduce it on the first window, not on the next ones, so I think it's only the icon's load time that causes the flicker... for me. If others are seeing it on all new windows, there's an additional issue to fix there.
Comment on attachment 8905056 [details]
Bug 1397285 - Set standard toolbarbutton-1 icon size as width rather than max-width to reserve space for icons while they're loading.

https://reviewboard.mozilla.org/r/176862/#review181890

Works for me. :)
Attachment #8905056 - Flags: review?(jhofmann) → review+
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/16290ce48f84
Set standard toolbarbutton-1 icon size as width rather than max-width to reserve space for icons while they're loading. r=johannh
https://hg.mozilla.org/mozilla-central/rev/16290ce48f84
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
User Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:58.0) Gecko/20100101 Firefox/58.0
Build ID: 20170925100307

This issue has been verified on latest Firefox Nightly Build ID: 20170925100307 on Windows 8.1 x64, Mac OS 10.12 and Ubuntu 14.04 and I cannot reproduce it. Now, when opening a bookmarked webpage or any other webpages the URL bar and Home button do not jump to left.
Status: RESOLVED → VERIFIED
User Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20170928180207

This issue has been verified on latest Firefox Beta 57.0b4 on Windows 8.1 x64, Mac OS 10.12 and Ubuntu 14.04 and I cannot reproduce it.
You need to log in before you can comment on or make changes to this bug.