Closed
Bug 1567335
Opened 5 years ago
Closed 5 years ago
Markup view's DOM mutation highlight is incorrect
Categories
(DevTools :: Inspector: Rules, defect, P3)
DevTools
Inspector: Rules
Tracking
(firefox70 fixed)
RESOLVED
FIXED
Firefox 70
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: mtigley, Assigned: mtigley)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
The DOM mutation highlight in the latest Nightly blinks black -on-yellow then black-on-blue. The text should only be black until the animation is completed (i.e: the yellow background has faded).
A quick fix for this is to explicitly define a transition-duration for both background and color in .flash-out
. See: https://searchfox.org/mozilla-central/source/devtools/client/themes/markup.css#294 . So:
.flash-out {
transition-property: background,color;
transition-duration: .5s .5s;
}
Updated•5 years ago
|
Keywords: regression
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → mtigley
Status: NEW → ASSIGNED
Priority: -- → P3
Assignee | ||
Comment 1•5 years ago
|
||
Pushed by mtigley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/637f32b7e43c Add an explicit CSS transition duration for the markup view's flash-out class. r=gl
Comment 3•5 years ago
|
||
Backed out for dt failures on markup.css
Backout link: https://hg.mozilla.org/integration/autoland/rev/d4e621b5c51bbfb6d27cd9e307360821c447a06b
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=257920557&repo=autoland&lineNumber=1740
Flags: needinfo?(mtigley)
Updated•5 years ago
|
Attachment #9079582 -
Attachment description: Bug 1567335 - Add an explicit CSS transition duration for the markup view's flash-out class. r=gl → Bug 1567335 - Make sure the animation class for the foreground element is removed. r=gl
Assignee | ||
Comment 4•5 years ago
|
||
Flags: needinfo?(mtigley)
Pushed by mtigley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/94afeea9ca07 Make sure the animation class for the foreground element is removed. r=gl
Comment 6•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Updated•2 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•