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

New "theme selector" options page, updating build system and project structure #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

scottalguire
Copy link

First off, I've been using this plugin for years and I have really enjoyed not being blinded by the default white devtools, so thank you!

Recently I decided to adapt my current favorite syntax theme "Panda" for devtools.

After adding the new theme, I realized it would be cool to switch themes without having to:

  • git clone the repo
  • update config
  • npm install
  • run grunt
  • reload the unpacked extension
  • etc.. it's a whole lot of ceremony

For that reason I've updated the plugin so that themes can be switched from an options panel, with the current option being stored in chrome storage.

Other changes are listed below and (in the changelog)

  • Added Panda Syntax theme by @siamak
  • Added new extension options popup for switching themes without leaving Chrome
  • Remove hardcoded stable version in favour of API call
  • Updated project structure and build setup to accomodate theme switching
  • Added JSON file for controlling options panel
  • Added missing variables to older themes that were failing to compile

Here is what the options panel looks like:

image

I'm happy to address any questions/suggestions

Cheers

…ly, adjusting project structure and build tasks for precompiling css, dynamically fetching stable chrome, defining themes using JSON
@mauricecruz
Copy link
Owner

Wow, thanks for doing this! Really impressive work! Always planned on creating a UI loader, but kept kicking that can down the road.

I'll give it a lookover this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants