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 Embark theme #533

Merged
merged 6 commits into from
Feb 24, 2025
Merged

Add Embark theme #533

merged 6 commits into from
Feb 24, 2025

Conversation

stephenfin
Copy link
Contributor

Description

Add the Embark theme from https://embark-theme.github.io/.

While adding this, I noted that we were missing a few screenshots and some variants of themes for various editors. These are all addressed in separate commits. I suggest looking at the commits tab where reviewing these.

I have no updated CREDITS.md since I did not author this theme.

Theme Submission Checklist

  • Included theme in iTerm2 format
  • Included 600x300 screenshot, 13pt Monaco font, no transparency
  • Updated README.md with new theme and screenshot
  • Updated CREDITS.md with new theme
  • Ran tools/gen.py to generate themes in all formats
  • Updated screenshots/README.md with new theme

  SyntaxWarning: invalid escape sequence '\.'

Signed-off-by: Stephen Finucane <[email protected]>
This was done in mbadolato#526, however, we should update the root iTerm2 theme,
not a specific terminal theme.

Signed-off-by: Stephen Finucane <[email protected]>
Signed-off-by: Stephen Finucane <[email protected]>
I don't know how these have been missed. They're generated simply by
running:

  python tools/gen.py

Signed-off-by: Stephen Finucane <[email protected]>
Signed-off-by: Stephen Finucane <[email protected]>
@mbadolato mbadolato merged commit 2b408b5 into mbadolato:master Feb 24, 2025
@mbadolato
Copy link
Owner

Thanks!

@stephenfin stephenfin deleted the add-embark branch February 24, 2025 23:32
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