Closed Bug 989479 Opened 7 years ago Closed 7 years ago
Content preferences treats [disabled=false] as disabled state
Screenshot, which also explains the solution to the issue : http://images.devs-on.net/Image/agCctCaihT96Qrd7-Region.png Gonna make a patch for this.
Jared, if you don't know what this is about, check the selected inspector element in the screenshot. It has disabled="false". And the CSS for disabled state applies to it. The [disabled="true"] selector is used in most of Firefox's CSS files, so it'd be nice to stay consistent.
Richard, do you think you can review this ? Or should I leave this to Jared ?
I'm not doing Fx reviews but the patch is looking good.
Attachment #8398759 - Flags: review?(jaws) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
Does this need tests?
I don't think so. Adding reftests for this doesn't seem needed.
Flags: in-testsuite? → in-testsuite-
You need to log in before you can comment on or make changes to this bug.