Skip to content

Conversation

@BURG3R5
Copy link
Contributor

@BURG3R5 BURG3R5 commented Oct 17, 2025

Description

Fixes #10

As I mentioned in #10, support for additional themes will be a really nice QoL upgrade... so I implemented it.

I've changed the theme.toggle shortcut to theme.cycle which steps through an arbitrary list of themes. I've also added four themes: black, red, green and blue. I've updated the relevant test.

Additional context

(Kindly ignore the sliver of grey in the bottom-right corner.)

  • Black:
Screenshot 2025-10-17 at 11 45 20 PM
  • Red:
Screenshot 2025-10-17 at 11 45 00 PM
  • Green:
Screenshot 2025-10-17 at 11 45 07 PM
  • Blue:
Screenshot 2025-10-17 at 11 45 13 PM

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Before submitting the PR, please make sure you do the following

  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

Copy link
Owner

@FPurchess FPurchess left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Thanks for your contribution

@BURG3R5
Copy link
Contributor Author

BURG3R5 commented Oct 18, 2025

@FPurchess If it's alright with you, can you approve the workflows and merge the PR?

@FPurchess FPurchess merged commit c1dac1d into FPurchess:dev Oct 20, 2025
4 checks passed
@BURG3R5 BURG3R5 deleted the feature/themes branch October 21, 2025 07:09
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.

More Themes

2 participants