Closed Bug 1702777 Opened 3 years ago Closed 3 years ago

Urlbar type icons use pre-Proton colours

Categories

(Firefox :: Address Bar, defect, P2)

Firefox 89
defect

Tracking

()

RESOLVED FIXED
Tracking Status
firefox89 --- affected

People

(Reporter: RT, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [priority:2b][proton-address-bar])

Attachments

(1 file)

  • "Search with..." icons look fuzzy in dark theme
  • Dark theme primary blue for bookmark icon: #00DDFF
  • Dark theme icon color for "Switch to tab" icon: #FBFBFE
Component: Search → Address Bar

On today's nightly the star color looks ok in the dark theme, closer to the spec in light theme. As well the tab color. Not sure why they still differ slightly.

The fuzzy icons may just depend on which icon we have and hidpi, in some cases we don't have a better one, or we're waiting to be provided a better one.

NI? harry that recently landed a patch fixing these colors to check those slight differences.

Flags: needinfo?(htwyford)
Priority: -- → P2

Resolving as dupe of bug 1691993 since the newly provided icons will be hidpi

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE

Bug 1691993 won't solve the issue of the search engine icons not being HiDPI. Marco's right that we just use the best icon we have. If a site doesn't provide a HiDPI favicon, there's not much we can do. As for the colours on the star and bookmark, that should have been solved by bug 1699226. iirc it was fixed on Nightly, but it looks like it regressed.

Status: RESOLVED → REOPENED
Flags: needinfo?(htwyford)
Resolution: DUPLICATE → ---
Summary: "Search with..." icons look fuzzy and should use the correct colors in dark theme → Urlbar type icons use pre-Proton colours

Oh, never mind. I was just using an old Nightly build on this machine :) The colours are fixed now, so there's nothing else to address in this bug.

Status: REOPENED → RESOLVED
Closed: 3 years ago3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: