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

[BLOCKED] tutor: Tweak Restore old ASCII art for Helix name #5796

Conversation

goyalyashpal
Copy link
Contributor

@goyalyashpal goyalyashpal commented Feb 2, 2023

This restores the old ASCII art for Helix name (with one slight tweak) due to following reasons

old one was:

  • more aesthetically pleasing to the eye
  • ... as it followed a wave like pattern:
    large small medium small large
    capital small small small capital
    H eli X
  • the eli were of comparable weights
  • being small, and consistent, the eli looked cute
  • The H is now widened by one character, so, now both H and X are of same width: 8, so, they too are now of comparable weights

The new ascii art for name:

  • is confusing eye a lot
  • though the elix are intended to be small, there doesn't feel any significant distinction from the capital H
  • gives too much weight to the e
  • and e looks like having gaps in the inner curve side

the new design just feels lacking any character or charm of its own,
and totally confuses the eye due to lack of any pattern.

I don't think that forcing the graphic proportion on it is a great idea....

But again, feel free to discard if you disagree


Overlapping Comparison video:

helix.tutor.logo.PicasaPhotoViewer_UJuTftF2nz.mp4

@goyalyashpal
Copy link
Contributor Author

goyalyashpal commented Feb 3, 2023

vivaldi_JEWY7GopOv.mp4
Notepad3_6GPrPXrYc1.mp4

@archseer
Copy link
Member

archseer commented Feb 3, 2023

If I wanted to keep the original text I would have indicated that in the original PR. The new design is a lot more suitable since the name is Helix and not HeliX.

@archseer archseer closed this Feb 3, 2023
@goyalyashpal
Copy link
Contributor Author

goyalyashpal commented Feb 3, 2023

since the name is Helix and not HeliX.

is that the only concern?
if yes, then i have fixed it in the next commits - which aren't showing up here due to the closure

only one 'character' remains to be changed.

The previous version was just quickly generated using an online tool.
- @ archseer at #5681 (comment)

link?

@goyalyashpal goyalyashpal changed the title tutor: Tweak Restore old ASCII art for HeliX name tutor: Tweak Restore old ASCII art for Helix name Feb 3, 2023
@goyalyashpal
Copy link
Contributor Author

hey @archseer

  • can u please open this pull temporarily so that
  • at least the latest final commits get fetched and show up here for the history ...
  • even if you don't want to merge, at least the final result of this pull will be preserved here?
  • feel free to close it after the latest commits show up here...

@goyalyashpal
Copy link
Contributor Author

the new x is:

  • small case,
  • kinda matches the logo's style with right pointing arrow being slightly up, and left pointing arrow being slightly down

@goyalyashpal goyalyashpal changed the title tutor: Tweak Restore old ASCII art for Helix name [BLOCKED] tutor: Tweak Restore old ASCII art for Helix name Jul 9, 2023
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