1279526 - Firebug theme: fix CSS for rule view property

Status: RESOLVED FIXED

(DevTools :: Inspector: Rules, defect, P1)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

This is a follow up for bug 1274760

From:
https://bugzilla.mozilla.org/show_bug.cgi?id=1274760#c7

--

One more thing!
The rule that follows at the end of the change:

 .theme-firebug .ruleview-enableproperty:not([checked]) ~ .ruleview-computedlist * {
   color: #CCCCCC;
 }

seems wrong.  The reason for that is that .ruleview-enableproperty and .ruleview-computedlist are not siblings since Firefox 40 (they were siblings in Firefox 39). The rule is intended to make gray the listed (sub) properties from a shorthand property when that shorthand property is disabled.  Given these are not siblings any longer it is hard to fix this with just css.  The best I am able to do is to replace the rule with:

.theme-firebug .ruleview-overridden > * > .ruleview-computed:not(.ruleview-overridden),
.theme-firebug .ruleview-overridden > * > .ruleview-computed:not(.ruleview-overridden) > .ruleview-propertyname,
.theme-firebug .ruleview-overridden > * > .ruleview-computed:not(.ruleview-overridden) > .ruleview-propertyvalue {
  color: #CCCCCC;
}

This works for me but has a problem when disabling a property overridden in another rule.  In such case, it will not apply until inspecting another node.

Also see a screenshot:
https://bugzilla.mozilla.org/attachment.cgi?id=8761782&action=edit

Honza

Comment 1

[Patrick Brosset <:pbro>]

Inspector bug triage (filter on CLIMBING SHOES).
Tagging as P1 like bug 1274760.

Comment 2

[Patrick Brosset <:pbro>]

Honza: do you think P1 is the right priority for this bug. No one has taken a look at this bug in the last month. Should someone urgently investigate the issue here?

Comment 3

[Jan Honza Odvarko [:Honza] (always need-info? me)]

(In reply to Patrick Brosset <:pbro> from comment #2)
> Honza: do you think P1 is the right priority for this bug. No one has taken
> a look at this bug in the last month. Should someone urgently investigate
> the issue here?

Yes, I am planning to have this done by someone from the Taipei office.

Honza

Comment 4

[[e-mail]]

Attachment: Bug 1279526 - Firebug theme: fix CSS for rule view property;

Review commit: https://reviewboard.mozilla.org/r/63828/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/63828/

Comment 5

[[e-mail]]

The patch is took from johngraciliano 's comment from bug 1274760 comment 10

All padding color will turn to grey when the block sign is tapped.

Comment 6

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Does the patch look correct to you?

Honza

Comment 7

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8770362 [details]
Bug 1279526 - Firebug theme: fix CSS for rule view property;

https://reviewboard.mozilla.org/r/63828/#review61862

I tested this and works for me, thanks!

Honza

Comment 8

[[e-mail]]

thanks!

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/fx-team/rev/c96249f34046
Firebug theme: fix CSS for rule view property; r=Honza

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c96249f34046

Comment 11

[Mohammad Maruf Rahman]

I have reproduced this bug with Nightly 50.0a1 (2016-06-10) on Windows 7, 64 Bit !

This bug's fix is verified with latest developer edition (Aurora)

Build ID   20160907004009
User Agent Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:50.0) Gecko/20100101 Firefox/50.0
[bugday-20160907]

Comment 12

[Mohammad Maruf Rahman]

I have reproduced this bug with Nightly 50.0a1 (2016-06-10) on Windows 7, 64 Bit !

This bug's fix is verified with latest developer edition (Aurora)

Build ID   20160907004009
User Agent Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:50.0) Gecko/20100101 Firefox/50.0
[bugday-20160907]