Closed Bug 1144377 Opened 10 years ago Closed 10 years ago

hover state on reading list sidebar items remains after the mouse leaves the list

Categories

(Firefox Graveyard :: Reading List, defect)

defect
Not set
normal

Tracking

(firefox38 verified, firefox39 verified)

VERIFIED FIXED
Firefox 39
Tracking Status
firefox38 --- verified
firefox39 --- verified

People

(Reporter: clarkbw, Assigned: florian)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

STR: * have a couple items in the reading list * hover over one of the items * move the mouse away from the sidebar Actually Results - last hovered item keeps its hovered state Expected Results - As the mouse moves off the item (and off the list) its hover state does not remain
Flags: qe-verify+
OS: Mac OS X → All
Attached patch PatchSplinter Review
I discussed this with mmaslaney today, and it turns out he never intended the hovered item to have a different background color. We still need to have this 'selected' color for when the user interacts with the list using the keyboard. As a bonus, I made the [X] of the selected item visible, so removing items from the list is now keyboard accessible. The padding change is so that the focus ring around that X button has a reasonable shape.
Assignee: nobody → florian
Attachment #8579744 - Flags: review?(jaws)
Attachment #8579744 - Flags: review?(jaws) → review+
Hi Florian, can you provide a point value.
Status: NEW → ASSIGNED
Iteration: --- → 39.2 - 23 Mar
Flags: needinfo?(florian)
Flags: firefox-backlog+
Points: --- → 3
Flags: needinfo?(florian)
Blocks: 1132074
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 39
QA Contact: andrei.vaida
Verified fixed on Nightly 39.0a1 (2015-03-23), using Ubuntu 14.04 (x64), Mac OS X 10.9.5 and Windows 7 (x64).
Status: RESOLVED → VERIFIED
Verified fixed on Aurora 38.0a2 (2015-03-29) as well, using Windows 7 (x64), Ubuntu 14.04 (x64) and Mac OS X 10.9.5.
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: