Open Bug 1707131 Opened 4 years ago Updated 2 years ago

Screenshot tool uses blue button instead of accent colour button

Categories

(Firefox :: Screenshots, defect, P3)

Firefox 89
Desktop
All
defect
Points:
1

Tracking

()

People

(Reporter: pdehaan, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Whiteboard: [screenshots-extension])

Attachments

(1 file)

This comment is copy/pasted from a community issue filed during the Foxfooding program.


Firefox Version: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:90.0) Gecko/20100101 Firefox/90.0
Window Size (inner width and height): 1920x955

Steps to Reproduce

  1. right click on page
  2. click the "take a screenshot" option
  3. select an area to take a screenshot of
  4. look at the colour of the "download" button once the preview comes up

Expected Behavior

Firefox should use the windows accent colour as it does with many other buttons in the interface

Actual Behavior

Firefox uses a blue coloured button no matter the theme or windows accent colour

Screenshot

Attached image screenshot tool download button (deleted) —
Priority: -- → P2
Whiteboard: [foxfooding][community] [proton-foxfooding] → [foxfooding][community] [proton-foxfooding] [priority:2c] [proton-modals]

This isn't a modal so I don't think this is in scope. Can we move to proton-cleanups?

Flags: needinfo?(rtestard)
Blocks: proton-cleanups
No longer blocks: proton-modals
Flags: needinfo?(rtestard)
Priority: P2 → P3
Whiteboard: [foxfooding][community] [proton-foxfooding] [priority:2c] [proton-modals] → [foxfooding][community] [proton-foxfooding] [proton-cleanups]
Points: --- → 1

This is dependent upon converting the screenshots extension into a component because that would allow screenshots to access the browser's colors directly. That's needed because the colors need to match up with whatever the current browser theme is using, whereas in the current setup we would only be able to go off of the OS theme.

Depends on: 1696573
Component: Foxfooding → Screenshots

The severity field is not set for this bug.
:sfoster, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(sfoster)
Severity: -- → S4
Flags: needinfo?(sfoster)
Whiteboard: [foxfooding][community] [proton-foxfooding] [proton-cleanups] → [screenshots-extension]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: