Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "Properties" section to select dark theme #5

Merged
merged 2 commits into from
Jun 9, 2022

Conversation

FaBjE
Copy link
Contributor

@FaBjE FaBjE commented Jun 6, 2022

I've added the "properties" section to the theme.

This allows for the following:

  • Auto-detects dark theme on websites (like https://www.startpage.com/ and many others)
  • Sets style for "settings pages" and other to dark

Documentation:
https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/theme#properties

@RaphGL RaphGL added the enhancement New feature or request label Jun 8, 2022
@RaphGL
Copy link
Collaborator

RaphGL commented Jun 8, 2022

works fine for me, but you should bump the version number when you change the manifest.json file cause otherwise I won't be able to update the theme on the addons store (tho I could do that myself)

@FaBjE
Copy link
Contributor Author

FaBjE commented Jun 9, 2022

Fixed. I was unsure of the "versioning scheme" used here.

@RaphGL
Copy link
Collaborator

RaphGL commented Jun 9, 2022

All set.

@RaphGL RaphGL closed this Jun 9, 2022
@RaphGL RaphGL reopened this Jun 9, 2022
@RaphGL RaphGL merged commit 486ae40 into dracula:master Jun 9, 2022
@FaBjE
Copy link
Contributor Author

FaBjE commented Jun 14, 2022

@RaphGL
Copy link
Collaborator

RaphGL commented Jul 1, 2022

yes it will, I will be shipping it soon, I just got a little busy lately

@RaphGL
Copy link
Collaborator

RaphGL commented Jul 1, 2022

Version with the changes has been shipped and is waiting for approval by Firefox. An XPI of the theme is also available here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

2 participants