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)

defect

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;
}
Regressed by: 1541278
Keywords: regression
Assignee: nobody → mtigley
Status: NEW → ASSIGNED
Priority: -- → P3
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
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
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
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: