Closed
Bug 1227129
Opened 9 years ago
Closed 9 years ago
'Exit Full Screen (Esc)' text bounce to the right and returns to its initial position
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
VERIFIED
FIXED
Firefox 45
People
(Reporter: adalucinet, Assigned: xidorn)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-review-board-request
|
roc
:
review+
lizzard
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
|
Details |
Reproducible with Firefox 43 beta 5, latest Developer Edition 44.0a2 and latest Nightly 45.0a1
Affected platforms: Windows 7 64-bit (2 different machines) and Mac OS X 10.11.1
Steps to reproduce:
1. Launch Firefox and go to Add-ons Manager (Ctrl/Cmd + Shift + A).
2. Click on 'Watch the Video' button via Get Add-ons tab
3. Enter fullscreen.
4. Mouse over the Fullscreen notification.
Expected results: Fullscreen notification is displayed correctly.
Actual results: 'Exit Full Screen (Esc)' text moves to the right and returns to its initial position (please see the attached gif).
Additional notes:
1. Unable to reproduce under Ubuntu 14.04 32-bit.
2. Not reproducible with Nightly from 2015-08-15 - will investigate further.
Reporter | ||
Comment 1•9 years ago
|
||
Here is the gif - https://goo.gl/wqcRfR (couldn't manage to attach it)
Assignee | ||
Comment 2•9 years ago
|
||
I actually have been seeing this issue since the very beginning.
I guess this is because of minor rendering difference when it is moved across layers.
Assignee | ||
Comment 3•9 years ago
|
||
Bug 1227129 - Apply 'will-change: transform' to fullscreen-warning when it is shown.
Since there is a 3s timeout to hide the fullscreen warning, it makes
sense to always apply will-change transform when it is not hidden. This
could also avoid slight rendering difference when it stops moving.
Attachment #8691164 -
Flags: review?(roc)
Attachment #8691164 -
Flags: review?(roc)
Comment on attachment 8691164 [details]
MozReview Request: Bug 1227129 - Apply 'will-change: transform' to fullscreen-warning when it is shown.
https://reviewboard.mozilla.org/r/26085/#review23455
Attachment #8691164 -
Flags: review+
Comment on attachment 8691164 [details]
MozReview Request: Bug 1227129 - Apply 'will-change: transform' to fullscreen-warning when it is shown.
https://reviewboard.mozilla.org/r/26085/#review23457
Assignee | ||
Comment 6•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/befb8e2645ef77775bbd5ef7eb866fd51b376458
Bug 1227129 - Apply 'will-change: transform' to fullscreen-warning when it is shown. r=roc
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → quanxunzhen
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8691164 [details]
MozReview Request: Bug 1227129 - Apply 'will-change: transform' to fullscreen-warning when it is shown.
Approval Request Comment
[Feature/regressing bug #]: bug 1160023
[User impact if declined]: as described in comment 0 and shown in comment 1
[Describe test coverage new/current, TreeHerder]: n/a
[Risks and why]: very low risk because it is a simple change in style, and the 'will-change' property (the only change here) is generally only a hint for the engine, and isn't meant to cause any major behavior change
[String/UUID change made/needed]: n/a
Attachment #8691164 -
Flags: approval-mozilla-beta?
Attachment #8691164 -
Flags: approval-mozilla-aurora?
Comment 8•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Regression from 43, marking affected and tracking for 43+.
status-firefox43:
--- → affected
status-firefox44:
--- → affected
tracking-firefox43:
--- → +
tracking-firefox44:
--- → +
tracking-firefox45:
--- → +
Keywords: regression
Comment on attachment 8691164 [details]
MozReview Request: Bug 1227129 - Apply 'will-change: transform' to fullscreen-warning when it is shown.
Fix for recent regression in fullscreen style. Please uplift to aurora and beta.
Attachment #8691164 -
Flags: approval-mozilla-beta?
Attachment #8691164 -
Flags: approval-mozilla-beta+
Attachment #8691164 -
Flags: approval-mozilla-aurora?
Attachment #8691164 -
Flags: approval-mozilla-aurora+
Comment 11•9 years ago
|
||
bugherder uplift |
Comment 12•9 years ago
|
||
bugherder uplift |
Updated•9 years ago
|
Flags: qe-verify+
Reporter | ||
Comment 13•9 years ago
|
||
Verified fixed with latest Nightly 45.0a1, across platforms [1].
[1] Mac OS X 10.11.1, Windows 7 64-bit and Ubuntu 14.04 32-bit
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 14•9 years ago
|
||
Also confirming this fix with Firefox 43 beta 7 (Build ID: 20151126120800) and latest Developer Edition 44.0a2 (from 2015-11-26), under Windows 7 64-bit and Mac OS X 10.11.1
You need to log in
before you can comment on or make changes to this bug.
Description
•