-
Notifications
You must be signed in to change notification settings - Fork 27
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 Aztec #9
Add Aztec #9
Conversation
Add aztec theme
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello 😄 Can you remove the one comment? It doesn't add anything since the information is already contained within the yaml content itself. Once you remove it I'll merge the PR.
base16/aztec.yaml
Outdated
@@ -0,0 +1,22 @@ | |||
# Aztec scheme for the Base16 Builder (https://github.com/chriskempson/base16-builder) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I just copied yaml template from another theme within the repo, thought it is descriptive and appropriate.
base05: "FFDA51" # Foreground -- | ||
base06: "FFE178" # Foreground --- | ||
base07: "FFEBA0" # Foreground ---- | ||
base08: "EE2E00" # Red |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tbh love the comments here. Cheers :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I copied the template from yaml file of 3024 theme. Thank contributor of that theme instead of me.
File is updated. I removed the comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice
It is going to be automatically added to base16 gallery, right? |
Yeah the builds happen every sunday, but I'll manually run it in the gallery now (the rest of the templates will be updated this sunday) |
Add Aztec theme to repository. (https://github.com/TheNeverMan/aztec-base16/)