Closed Bug 1049029 Opened 10 years ago Closed 10 years ago

DevTools Themes: Remove hover and active state on active sidebar tab (dark theme)

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(firefox34 verified)

VERIFIED FIXED
Firefox 34
Tracking Status
firefox34 --- verified

People

(Reporter: me, Assigned: me, Mentored)

References

Details

Attachments

(1 file)

When I solved Bug 929127, I added :hover and :active states to the active tab on dark theme. Since in the light theme the active tab color doesn't change on hover, I think in the dark theme it shouldn't either.
Assignee: nobody → aljullu
Blocks: 916766
Status: NEW → ASSIGNED
Agreed, since reselecting on the tab doesn't do anything there is no reason to show hover/active states
Attached patch bug1049029.patchSplinter Review
I think that's all.
Attachment #8470462 - Flags: review?(bgrinstead)
Comment on attachment 8470462 [details] [diff] [review]
bug1049029.patch

Review of attachment 8470462 [details] [diff] [review]:
-----------------------------------------------------------------

Looks great
Attachment #8470462 - Flags: review?(bgrinstead) → review+
https://hg.mozilla.org/mozilla-central/rev/c542ef01339a
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 34
Verified: [bugday-20140813]

Fixed in Windows 7 32-bit OS Firefox Nightly 34. 

Explanation: 
Hover on active sidebar does not change color.
QA Whiteboard: [qa+]
Already verified by Karthikeyan in comment 6.
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa+] → [qa!]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: