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

feat: add Twig language support with ASCII art and color scheme #1257

Merged
merged 3 commits into from
Apr 10, 2024

Conversation

arferreira
Copy link
Contributor

Adding support for Twig

#490

Tell me what you think about the proposal

Screenshot 2024-01-20 at 2 16 18 AM

Original logo:

Screenshot 2024-01-20 at 2 17 16 AM

Copy link
Collaborator

@spenserblack spenserblack left a comment

Choose a reason for hiding this comment

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

Overall LGTM! Thanks!

languages.yaml Outdated
@{0}...........:77:...:~!!!~:.............@{0}
@{0}...........:^^:.......................@{0}
@{0}......................................@{0}
########################################
Copy link
Collaborator

Choose a reason for hiding this comment

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

@o2sh what do you think, should these characters be explicitly styled as "white"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what about?
Screenshot 2024-01-20 at 11 42 23 AM

Copy link
Collaborator

Choose a reason for hiding this comment

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

I meant that, instead of just {0} with green, it might be better to have two colors, {0} and {1} for white and green.

@o2sh
Copy link
Owner

o2sh commented Mar 24, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

Copy link
Collaborator

@spenserblack spenserblack left a comment

Choose a reason for hiding this comment

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

I think this change I pushed represents the intended style. @arferreira let me know if this is true.

@spenserblack spenserblack merged commit 41c3ecf into o2sh:main Apr 10, 2024
11 checks passed
@spenserblack
Copy link
Collaborator

BTW 😉 spenserblack/gengo#34 (context: #1305)

o2sh added a commit that referenced this pull request Apr 28, 2024
…me (#1257)"

The ASCII logo needs some rework, I'll revert for now.

This reverts commit 41c3ecf.
@o2sh
Copy link
Owner

o2sh commented Apr 28, 2024

The ASCII logo needs some rework. I'll revert it for now and will try to come up with a better design.

Sorry for not stepping in earlier. 🙏

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.

3 participants