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

New logo for Weapon regeX #197

Merged
merged 3 commits into from
Nov 2, 2022
Merged

New logo for Weapon regeX #197

merged 3 commits into from
Nov 2, 2022

Conversation

nhaajt
Copy link
Contributor

@nhaajt nhaajt commented Oct 31, 2022

In (late) celebration of v1.0 release, I’m proposing a new logo for Weapon regeX. This new design addresses a few issues with the old logo:

  • Simplified design with more balanced proportion
  • Better legibility with the monogram WR and the regex \\?
  • More related symbolism of mutating regex (QuantifierRemoval mutates \\? to \\) in Stryker’s red

New logo:

Weapon regeX new logo

Weapon regeX new logo

Weapon regeX new logo

Old logo:

Weapon regeX old logo

Weapon regeX old logo

Weapon regeX old logo

@nicojs
Copy link
Member

nicojs commented Oct 31, 2022

I like it 🤩!

But I do feel it's a missed opportunity not to put forward slashes in the logo. Do you think you would be able to manage that? Since /forward slashes/ are a regex literal in JS 😎

@nhaajt
Copy link
Contributor Author

nhaajt commented Oct 31, 2022

Very interesting idea. I'll keep /that/ in mind for v3 😉 when wrx hits the next big milestone

@hugo-vrijswijk
Copy link
Member

I also like it! @nhaajt if it's not too much work, could you also try a version with a line connecting the first two backslashes? That might make it a bit more clear that it's a W

@nhaajt
Copy link
Contributor Author

nhaajt commented Oct 31, 2022

Here are some quick edits with a line connecting the first two backslashes. With only 2 colors, the \\ and ? seems a bit lost. A 3rd color is needed to separate things. Overall, I’m not really satisfied with these since it adds unnecessary complexity to the design.

variants

Copy link
Member

@hugo-vrijswijk hugo-vrijswijk left a comment

Choose a reason for hiding this comment

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

Hmm, I agree. The original is best ✨. I like it!

Copy link
Member

@nicojs nicojs left a comment

Choose a reason for hiding this comment

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

🚀

@nhaajt nhaajt merged commit b993215 into main Nov 2, 2022
@nhaajt nhaajt deleted the feat/logo-v2 branch November 2, 2022 09:22
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