Closed Bug 1178193 Opened 5 years ago Closed 5 years ago

[Rule View] Tweak add rule button to only have a hover opacity when button is not disabled

Categories

(DevTools :: Inspector, defect)

defect
Not set

Tracking

(firefox41 fixed)

RESOLVED FIXED
Firefox 41
Tracking Status
firefox41 --- fixed

People

(Reporter: gl, Assigned: gl)

Details

Attachments

(1 file)

From https://bugzilla.mozilla.org/show_bug.cgi?id=1177564#c8:
> ::: browser/themes/shared/devtools/ruleview.css
> @@ +291,5 @@
> > +#pseudo-class-panel-toggle {
> > +  opacity: 0.8;
> > +}
> > +
> > +#ruleview-add-rule-button:hover,

#ruleview-add-rule-button needs a check for :not([disabled]), otherwise the disabled state will have an hover effect which is unwanted.
Assignee: nobody → gabriel.luong
Status: NEW → ASSIGNED
Attached patch 1178193.patchSplinter Review
Attachment #8627112 - Flags: review?(pbrosset)
Attachment #8627112 - Flags: review?(pbrosset) → review+
https://hg.mozilla.org/mozilla-central/rev/5eb1de369f21
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.