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

Refactor - Configure colours through prefix config variables #21

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

itslukeDev
Copy link

Description

I really liked this snippet and wanted to experiment with different colors to find something that would match my theme a little better. I found that it was kind of tedious to find and replace 11 references to a color variable for each prefix group.

This PR is the result of me reworking the CSS to only define the prefix's color in one (technically two) variables to make color swapping a breeze.

Changes made

  • Created prefix variables under the light and dark themes that are used instead of the color variables for each prefix group.
  • Added a VSCode snippet to quickly add new prefix groups.
  • Heavily refactored the README to shorten paragraphs into step-by-step bullets and to reflect the changes made above. (I completely understand if this last change is pushed back.)

@itslukeDev
Copy link
Author

itslukeDev commented Jan 4, 2025

Note to self: Actually look at the issues and pr's on a repo before accidentally re inventing the wheel
#4
#18
#14

🙃🙃🙃🙃🙃🙃🙃🙃🙃🙃🙃🙃🙃

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.

1 participant