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

added a variant with a few italicized entities #2

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

Conversation

rcrooks
Copy link

@rcrooks rcrooks commented Jan 16, 2021

Hi again @muchirijane - I hope this is not overstepping. I liked your theme so much I forked it to tweak it just a little -- added some italics, mainly to HTML attributes and JS function/method names. Just a personal preference, as I find it makes visual parsing of code easier for me. I didn't want to touch your masterpiece, so I added a variant version.

Offering it back in appreciation. Please reject it if you don't like it or just don't want outside contributions.

Best, Robert
2021-01-16_12-08-26

@muchirijane
Copy link
Owner

@rcrooks, I did try it when I was making it but I didn't like as much. It feels like it's too much at least for my taste.
But I truly appreciate your feedback. Thanks. 🥇

@rcrooks
Copy link
Author

rcrooks commented Jan 19, 2021

@muchirijane I fully understand and that's fine! 👍

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