Closed
Bug 985951
Opened 11 years ago
Closed 10 years ago
[box model] Start regions with some transparency and remove it when hovering each region
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: bgrins, Unassigned)
References
Details
Attachments
(2 files)
When hovering padding, for instance, the padding region would become the fully opaque #66cc52, while when not hovered it would have a different color based on whether it is on the light theme or dark theme.
To get the starting colors for light and dark theme, we could just measure the computed color for each region when inspecting an element with the same BG color as the sidebar and use that hex value.
To trigger the correct hovered elements we will need JS so that it wouldn't also hover outer nodes (like when you hovered padding then margins and borders would also be :hover in CSS).
Reporter | ||
Comment 1•11 years ago
|
||
jsfiddle for measuring the hex value of an inspected element with sidebar background colors (screenshot incoming): http://jsfiddle.net/bgrins/2Rkun/
Reporter | ||
Comment 2•11 years ago
|
||
Reporter | ||
Comment 3•11 years ago
|
||
Reporter | ||
Comment 4•11 years ago
|
||
Comment on attachment 8394321 [details]
hex-values-dark-sidebar-bg.png
Darrin, I think these seem a bit dark, but if anything that would be an argument to use more opacity than .4 on the page overlay. You can see this by inspecting the dark element at http://jsfiddle.net/bgrins/2Rkun/. If you think this is fine, all we need to do is eyedrop each region for this and the light theme screenshot to get the non-hovered bg colors for each region.
Attachment #8394321 -
Flags: ui-review?(dhenein)
Updated•10 years ago
|
Attachment #8394321 -
Flags: ui-review?(dhenein) → ui-review?(shorlander)
Reporter | ||
Comment 5•10 years ago
|
||
I think we need to actually to only highlight the relevant region when hovering the associated region in the box model panel.
Reporter | ||
Comment 6•10 years ago
|
||
Going to close this in lieu of bug 938188
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Reporter | ||
Updated•10 years ago
|
Attachment #8394321 -
Flags: ui-review?(shorlander)
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•