Closed Bug 1632394 Opened 4 years ago Closed 2 years ago

Saved logins lack hover on focus effects when trying to pick on desired page with a dark theme

Categories

(Toolkit :: Password Manager, defect, P3)

defect

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox-esr68 --- wontfix
firefox75 --- wontfix
firefox76 --- wontfix
firefox77 --- affected

People

(Reporter: cfogel, Assigned: serg)

Details

(Whiteboard: [passwords:fill-ui])

Attachments

(1 file, 1 obsolete file)

Affected versions

  • 75.0, 76.0b7, 77.0a1 (2020-04-21);

Affected platforms

  • macOS 10.15.3, Windows 10, Ubuntu 18.04;

Steps to reproduce

  1. Launch Firefox, save/set up at least 2 accounts for any webpage;
  2. Access that webpage, click to open the saved login options;
  3. Hover over the logins;

Expected result

  • hover effect applied over the on-hover element;

Actual result

  • login data not marked as hovered over, as per Saved Logins button;

Regression range

  • First bad: 2019-07-18;
  • Last good: 2019-07-17;
  • Pushlog: URL
  • not a regression, with introduction of the new about:logins page; checked after that and considered builds prior to it as good. With introduction of it, the hover was not implemented on it at all.

Additional notes

  • attaching screenshot with the "affected area".
Flags: qe-verify+
Priority: -- → P3
Whiteboard: [passwords:fill-ui]

Is this with the default theme or a dark theme? It sounds like this is specific to dark mode.

Summary: Saved logins lack hover on focus effects when trying to pick on desired page → Saved logins lack hover on focus effects when trying to pick on desired page with a dark theme

Indeed, the hover effect appears on the Light Theme setup.

Component: about:logins → Password Manager
Product: Firefox → Toolkit

Because this bug's Severity has not been changed from the default since it was filed, and it's Priority is P3 (Backlog,) indicating it has been triaged, the bug's Severity is being updated to S3 (normal.)

Severity: normal → S3

Hey Matt, It's me again, could I be assigned for this one as well?

Flags: needinfo?(MattN+bmo)

Depends on D76468

Assignee: nobody → f94olivera
Status: NEW → ASSIGNED

I think I messed up, need your help please... this commit bug fix does NOT depend on D76468, tried to solve it myself but i couldn't

Flags: needinfo?(MattN+bmo)

(In reply to f94olivera from comment #4)

Hey Matt, It's me again, could I be assigned for this one as well?

Attaching a patch assigns you automatically

(In reply to f94olivera from comment #6)

I think I messed up, need your help please... this commit bug fix does NOT depend on D76468, tried to solve it myself but i couldn't

Fixed.

I commented on the bug that I'm not sure this is the right solution as it also changes outside of the dark theme but doesn't cleanup the existing styles.

The bug assignee didn't login in Bugzilla in the last 7 months.
:sgalich, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: f94olivera → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(sgalich)

Hey all, thank you for filing and working on this bug.
I just tested on Firefox 100 and the issue does not exists anymore, closing this bug and patch as WORKSFORME.

Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(sgalich)
Resolution: --- → WORKSFORME
Assignee: nobody → sgalich
Attachment #9151244 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: