Closed Bug 1541543 Opened 5 years ago Closed 5 years ago

Incorrect hardcoded icon colors in pocket menu (worse for dark theme)

Categories

(Firefox :: New Tab Page, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 68
Iteration:
68.2 - Apr 1 - 14
Tracking Status
firefox68 --- verified

People

(Reporter: Mardak, Assigned: pdahiya)

References

Details

(Keywords: github-merged)

Attachments

(2 files)

Attached image new icons in dark theme (deleted) —

The svgs have hardcoded colors instead of using context.

Summary: Incorrect icon colors in menu for dark theme → Incorrect hardcoded icon colors in pocket menu (worse for dark theme)
Assignee: nobody → pdahiya
Keywords: github-merged

Hey Punam - is this something you can knock out this week before PTO next? (if not, we'll punt it to next week and someone can grab it if they can)

Iteration: --- → 68.3 - Apr 15 - 28
Flags: needinfo?(pdahiya)
Priority: -- → P1

(In reply to Jessilyn Davis from comment #2)

Hey Punam - is this something you can knock out this week before PTO next? (if not, we'll punt it to next week and someone can grab it if they can)

PR for this bug has landed in GitHub and should get fixed in nightly in next export. Thanks!

Flags: needinfo?(pdahiya)

Amazing! Thank you, Punam!

Iteration: 68.3 - Apr 15 - 28 → 68.2 - Apr 1 - 14
Blocks: 1543756
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68

I have verified that this issue is no longer reproducible with the latest Firefox Nightly (68.0a1 Build ID - 20190414214746) installed, on Windows 10 x64, Arch Linux and Mac 10.14.4. Now, the icons are respecting the browser's "Dark" theme colors.

Status: RESOLVED → VERIFIED
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: