Closed Bug 1744213 Opened 2 years ago Closed 2 years ago

Import your login icon is not so visible on hover with dark theme

Categories

(Core :: Widget, defect)

Desktop
All
defect

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox-esr91 --- unaffected
firefox94 --- wontfix
firefox95 --- wontfix
firefox96 --- wontfix
firefox97 --- fix-optional

People

(Reporter: atrif, Unassigned)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Attached image import_hover.gif

Affected versions

  • 96.0a1 (20211202215058)
  • 95.0 (20211129150630)
  • 94.0.2 (20211119140621)

Affected platforms

  • Windows10 x64
  • macOS 10.15

Preconditions

  • OS dark theme enabled

Steps to reproduce

  1. Open Firefox with a new profile and go to facebook.com
  2. Click on the email or password field.
  3. Hover over the Import from options.

Expected result

  • Icon from the import form is visible.

Actual result

  • Icon from the import from options is barely visible on hover.

Regression range

Notes

  • Screen recording attached.
  • On 94.0.2 the theme used for the import panel is white so the issue cannot be seen.
  • Linux does not have the import function.
Has Regression Range: --- → yes
Has STR: --- → yes
Flags: needinfo?(emilio)

Set release status flags based on info from the regressing bug 1734115

Do I need to do something else to get the icon to show up? I don't see that popup at all on a local build.

In any case from looking at the code, this seems like it's using color: GrayText, while the other text for the origin and so on is using opacity: .6;. Perhaps we can just use fill-opacity: .6 instead on the images so they match. WDYT Matt?

Flags: needinfo?(mozilla+bmo)
Flags: needinfo?(emilio)
Flags: needinfo?(alexandru.trif)

(In reply to Emilio Cobos Álvarez (:emilio) from comment #2)

Do I need to do something else to get the icon to show up? I don't see that popup at all on a local build.

You can open Firefox with a new profile and then the popup will be displayed a few times. Unfortunately, I don't know another method.

Flags: needinfo?(alexandru.trif)

Mardak would have a better idea about how to enable it but I think you need a login saved for that site in a browser we can import from. You can try to do a manual login import to confirm that the import code is finding the importable login.

Flags: needinfo?(mozilla+bmo) → needinfo?(edilee)
Attached image example dark logins

You should be able to force the import items to appear with the following in Browser Console:

LoginHelper.suggestImportCount = 3;
LoginHelper.showAutoCompleteImport = "import";
ChromeUtils.import("resource:///modules/ChromeMigrationUtils.jsm").ChromeMigrationUtils.getImportableLogins = () => ["chrome", "chromium-edge", "chromium", "hello world"];

(The first 2 lines override the pref values for signon.suggestImportCount and signon.showAutoCompleteImport.)

It seems like the usual "key" icon chrome://browser/skin/login.svg is also affected by this GrayText-ness?

Flags: needinfo?(edilee)

I think this should be working now.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: