Closed
Bug 589427
Opened 14 years ago
Closed 14 years ago
The Style panel should show the .style property value for each parent
Categories
(DevTools :: General, defect)
DevTools
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: msucan, Assigned: jwalker)
References
Details
Currently we only list rules (selectors) for the property the user expanded. We also list element.style ... to show the property value of the element the user has highlighted. This way we take into consideration element.style properties from the highlighted element. Still, we are missing values given to the inspected property (say color) from the parent elements. For example: <body style="color: blue"> <p>hello world!</p> </body> The blue value for the color property does not show in the current Style panel. We need a way to present this information to the user. One way would be to show the short name of each parent element like this: BODY[0] → blue The status we show on the right column is of dispute. We can't show "parent match", because it's not a rule selector that matched one of the parent elements. Nor can we show "inherited" because the color:blue property on BODY is not inherited from some other parent - it's set *there*. (notice the semantical and logical factors coming into play). Suggestions for a better status name? We could use a status like "LOCAL RULE", "LOCAL STYLE", "LOCAL PROPERTY" or simply "LOCAL". This would be then displayed for the row that shows the highlighted element.style, and for the rows that show the property value from each parent element. Additionally, we need to think of the UI matter: we want to remove the right column that displays the status. Limi will prepare some mockups which use better styling for conveying the status.
Updated•14 years ago
|
Assignee: nobody → jwalker
Whiteboard: [kd4b5]
Comment 1•14 years ago
|
||
Just making a note that Joe and Mihai think that this will not require any new strings.
Whiteboard: [kd4b5] → [kd4b6]
Reporter | ||
Comment 2•14 years ago
|
||
Correct. No new strings used. I have implemented this in bug 589849. If we switch to that code, we can mark this issue as fixed.
Updated•14 years ago
|
Severity: normal → blocker
Comment 4•14 years ago
|
||
Reprioritizing bugs. You can filter the mail on the word TEABAGS.
Severity: blocker → normal
Assignee | ||
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•