Closed Bug 1678333 Opened 4 years ago Closed 4 years ago

The color of the checkmark is blue and the background of the checkbox is white from the "Remove all <no of logins> logins?" dialog

Categories

(Toolkit :: Themes, defect, P3)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
85 Branch
Tracking Status
firefox83 --- unaffected
firefox84 --- unaffected
firefox85 --- verified

People

(Reporter: srosu, Assigned: tgiles)

References

(Regressed 2 open bugs)

Details

Attachments

(2 files)

Attached image Checkbox_Checkmark_color.PNG (deleted) —

[Affected versions]:

  • Firefox Nightly 85.0a1 (Build ID: 20201119095716)

[Affected Platforms]:

  • Mac 10.15.6
  • Windows 10 x64
  • Ubuntu 20.04 x64.

[Prerequisites]:

  • Have the latest version of Firefox Nightly installed.
  • Have a Firefox profile with at least one login saved.

[Steps to reproduce]:

  1. Open the Firefox browser with the profile from prerequisites.
  2. Navigate to the "about:logins" page.
  3. Click on the Menu button.
  4. Select the “Remove All Logins…” option.
  5. Tick the "Yes, remove these logins." checkbox.
  6. Observe the colors of the checkmark and the background of the checkbox.

[Expected result]:

  • The color of the checkmark is white, and the color of the background is blue.
    [Actual result]:
  • The checkmark has a blue color and the background of the checkbox is white.

[Notes]:

  • This issue is reproducible for all types of dialog.
  • Attached a screenshot with the issue.

I believe this is expected behavior, pulling in Katie to confirm this behavior. It might be divergent from the UX spec listed on the remove all logins ticket, but some changes were made that are not reflected in that UX spec.

Flags: needinfo?(kcaldwell)
Blocks: 1679115

Another good catch - the checkmark box should match other checkboxes components in the ui (like in about:preferences), which they do. However, from what I understand, these components should match the photon design system (as mentioned in the "Expected Results") with the blue background, and they currently don't. I'll have to consult with the design systems team.
[Leaving my NI until I have an update.]

ok - got info. Checkbox should match photon (blue background, please see https://design.firefox.com/photon/components/checkboxes.html). This element can be found in Save Bookmark doorhanger.

Tim, please update. Simona, good catch!

Flags: needinfo?(kcaldwell) → needinfo?(tgiles)
Assignee: nobody → tgiles
Status: NEW → ASSIGNED
Flags: needinfo?(tgiles)
Priority: -- → P3
Attachment #9191374 - Attachment description: Bug 1678333 - Fix checkbox colors to match Photon spec. r=sfoster,jaws → Bug 1678333 - Fix checkbox and radio buttons to match Photon spec. r=sfoster,jaws
Pushed by apavel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cc1e3921893e Fix checkbox and radio buttons to match Photon spec. r=desktop-theme-reviewers,ntim,dao
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch

I have verified this issue and is no longer reproducible using the latest Firefox Nightly 85.0a1 (Build ID: 20201211093444) on Windows 10 x64, Ubuntu 20.04 and Mac 10.15.7.

  • The color of the checkmark is white, and the color of the background is blue.
Status: RESOLVED → VERIFIED
Regressions: 1682094
Component: about:logins → Themes
Product: Firefox → Toolkit
Regressions: 1682128
Regressions: 1685628
No longer regressions: 1685628
Regressions: 1690555
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: