Closed Bug 1282168 Opened 8 years ago Closed 8 years ago

Orange loading throbber isn't shown anymore when loading pages

Categories

(Firefox :: Tabbed Browser, defect)

Unspecified
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 50
Tracking Status
firefox50 + fixed

People

(Reporter: marco, Unassigned)

References

Details

(Keywords: regression, Whiteboard: fixed by bug 1280510)

[Tracking Requested - why for this release]: Regression from bug 1280131.
Flags: needinfo?(gijskruitbosch+bugs)
Tracking 50+ for this visible Linux regression.
I couldn't reproduce this earlier today. Are you using a hidpi system/screen?
Flags: needinfo?(gijskruitbosch+bugs) → needinfo?(mcastelluccio)
(In reply to :Gijs Kruitbosch from comment #2) > I couldn't reproduce this earlier today. Are you using a hidpi system/screen? Yes. 2880x1620 with scaling set to 1.75.
Flags: needinfo?(mcastelluccio)
This only happens because the 1.1dppx selector for [busy] is later in the combined browser.css stylesheet than the one for [progress], which is what normally produces the orange throbber, so we always show the grey throbber rather than the orange one. We should fix that, but for nightly I don't think this is life-threatening right now. Rather than increasing the specificity of the lodpi rule, adding a duplicate hidpi rule only to Linux to fix this there, or adding !important to that particular rule (which probably has other side-effects I wouldn't like), I think the best way to fix this is to just fix bug 1280510 instead. I'll nominate that for tracking and do an additional prod of UX for assets later today.
Depends on: 1280510
Marking wfm because bug 1280510 is now fixed.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Resolution: WORKSFORME → FIXED
Whiteboard: fixed by bug 1280510
Target Milestone: --- → Firefox 50
The throbber is blue now instead of orange. Is it expected?
Flags: needinfo?(gijskruitbosch+bugs)
(In reply to Marco Castelluccio [:marco] from comment #6) > The throbber is blue now instead of orange. Is it expected? Yes, see bug 1280510.
Flags: needinfo?(gijskruitbosch+bugs)
You need to log in before you can comment on or make changes to this bug.