Conversation
spenserblack
left a comment
There was a problem hiding this comment.
Unless the Just community is familiar enough with this logo to easily recognize it, maybe you want to keep the name?
|
I think not, I wasn't familiar with this logo until I discovered it Maybe add a simple text that says |
I think some text underneath the circle would work. You might need to reduce the overall size of the circle so that it can all fit. |
| {0} @@@@@@@@@@@@@@@@@@@@@ | ||
| {0} @@@@@@@@@@@@@@ | ||
| {0} | ||
| {1} JUST |
There was a problem hiding this comment.
Thanks! I was hoping that the name "Just" would also be ASCII art and not just the 4 letters. Does that make sense?
There was a problem hiding this comment.
I can't imagine making both an ASCII art for Just as a text and the logo
There was a problem hiding this comment.
What do you think of this?
@@@@@@@@@@@@@@
@@@@@@@@@@@@@@@@@@@@@
@@@@@@@@@@ @@@@@@@@@@
@@@@@@@@@ @@@@@@@@@@
@@@@@@@@@@@@ @@@@@@@@@@@@
@@@@@@@@@@@@@@@ @@@@@@@@@@@@@@@
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@@@@@@@@@@@@@@@ @@@@@@@@@@@@@@@
@@@@@@@@@@@@ @@@@@@@@@@@@
@@@@@@@@@ @@@@@@@@@@
@@@@@@@@@@ @@@@@@@@@@
@@@@@@@@@@@@@@@@@@@@@
@@@@@@@@@@@@@@
db ,d
"" MM
MMMMMM MM MM ,pP"Ybd mmMMmm
MM MM MM 8I `" MM
MM MM MM `YMMMa. MM
MM MM MM L. I8 MM,
MM `Mbod"YM M9mmmP' "Mbmo
M. ,MM
"MMMMM"
There was a problem hiding this comment.
Looks nice!
Maybe some adjustments like changing the text ASCII style and the text alignment?
There was a problem hiding this comment.
Yeah, I think the "just" part needs to be moved to the right a few spaces. Would you be okay with doing the rest of the logo cleanup?
There was a problem hiding this comment.
I feel like the current one is cleaner than adding another ASCII art, it will be a bit noisy.
There was a problem hiding this comment.
"Current" as in the current one in the PR, or current one in the main branch?
spenserblack
left a comment
There was a problem hiding this comment.
I'm going to defer any further artistic opinions to @o2sh. From a technical standpoint, this LGTM. Though I think the simple 4-letter JUST looks a little too plain, and the ASCII art would look better without the word if we can't find a more creative way to add the language's name.
Improve the ASCII art for just inspired by the logo
the ASCII:
Resolves #1729