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

Nerves ASCII art! #170

Merged
merged 2 commits into from
Feb 9, 2021
Merged

Nerves ASCII art! #170

merged 2 commits into from
Feb 9, 2021

Conversation

fhunleth
Copy link
Member

Here's the new login:

image

I was playing around with JaVe last night. I'm sure that it could be better, but it seems like we HAVE to have something.

@fhunleth
Copy link
Member Author

Here's another idea:

image

@jjcarstens
Copy link
Member

I also have these I've played with in the past:

image

image

If we're taking votes, I might like the second one you posted the most

@fhunleth
Copy link
Member Author

fhunleth commented Nov 15, 2020

Nice. I was liking the larger ones and then got cold feet since the iex.exs already displays a lot. I came to the conclusion that the "N E R V E S" part doesn't look good for anything 2 or 3 lines high. That led me to make it just normal text and scale the N to something that didn't look too bad.

Except for the size, I do like your versions. And the colors are better.

The block one (the second one of mine) needs to be cleaned up. I did it super quick and ran out of time. Perhaps changing the colors to the ones that you have and cleaning it up so that the logo part doesn't look wonky might be good enough? What do you think?

@jjcarstens
Copy link
Member

Sounds good to me. We can also do a few different versions and pick which one to display based on :io.columns (which is what I've been doing with mine)

I'd be interested to see what the block one looks like a little cleaned up

@fhunleth
Copy link
Member Author

Here's another attempt at a block one:

image

It needs colors. I ended up writing a program to convert png files to Unicode blocks, but I don't have a way of colorizing it except manually. And there are a ton of ways of improving it - slightly frustrated that I couldn't find something to do this that wasn't so tied to another framework that it was hard to use.

If one of yours with the "NERVES" text is < 60 characters wide, I'm thinking that it is "good enough".

@fhunleth
Copy link
Member Author

Here's what it looks like now:

image

At this point, I'm thinking that this is pretty good.

Copy link
Member

@jjcarstens jjcarstens left a comment

Choose a reason for hiding this comment

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

Love it! 😍 :shipit:

@mobileoverlord mobileoverlord merged commit 6e6fb5f into main Feb 9, 2021
@mobileoverlord mobileoverlord deleted the ascii-art branch February 9, 2021 02:50
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.

None yet

3 participants