Ricky Rosario [:rrosario, :r1cky] Assignee
	Description • 7 years ago

Instead of guessing if theme is dark or light by textcolor, we'll hard code it to only activate on Dark theme (id="firefox-compact-dark@mozilla.org").

	[github robot]
	Comment 2 • 7 years ago

Commits pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/ee4767547d53c06dfc9dc238f2c44164fe5e9d8e
Fix Bug 1454501 - apply dark theme only to builtin Dark theme

https://github.com/mozilla/activity-stream/commit/2e41605750b3f2132de3610242ee1e0145eb7839
Merge pull request #4095 from rlr/Bug1454501/dark-theme-only

Fix Bug 1454501 - apply dark theme only to builtin Dark theme

	[github robot]
	Comment 4 • 7 years ago

Commits pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/7f97e0f7bc713fe498e7fc61d6130bf030f17ba0
Fix Bug 1454501 - followup to ee47675 to handle default theme being undefined

https://github.com/mozilla/activity-stream/commit/fbc8f8cc9ac8a5666d60c93c7e7eb32463cafbb7
Merge pull request #4099 from rlr/followup-dark-switch-fix

Fix Bug 1454501 - followup to ee47675 to handle default theme being undefined

	Ed Lee :Mardak
	Comment 5 • 7 years ago

https://hg.mozilla.org/mozilla-central/rev/4183020e7017

	Ciprian Muresan [:cmuresan], Ecosystem QA
	Comment 6 • 7 years ago

Build ID 	20180422223305
User Agent 	Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0

I have verified that the issue is no longer reproducible on the latest Nightly build on Windows 10 x64, Mac 10.13.3 and Arch Linux x64. The New Tab page dark theme is only triggered when the browser's dark theme is enabled.