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

[ICON REQUEST]: (Oh My) ZSH #1264

Closed
1 task done
SIMULATAN opened this issue Jul 11, 2022 · 27 comments
Closed
1 task done

[ICON REQUEST]: (Oh My) ZSH #1264

SIMULATAN opened this issue Jul 11, 2022 · 27 comments
Labels
in-develop The fix/feature is present in `develop` branch but not merged into `master` yet request:icon When requesting a new icon to be added to the collection or a refactor use this label in your issue

Comments

@SIMULATAN
Copy link

I have searched through the issues and didn't find my problem.

  • Confirm

About the icon

ZSH is quite a popular shell, used by many developers, often in conjunction with Oh My ZSH (147k GitHub stars)

Links and sources

ZSH (shell itself): https://commons.wikimedia.org/wiki/File:Z_Shell_Logo_Color_Vertical.svg
Oh My ZSH (configuration framework): https://ohmyz.sh/img/ohmyzsh-original-logo.svg

Additional information

No response

@SIMULATAN SIMULATAN added the request:icon When requesting a new icon to be added to the collection or a refactor use this label in your issue label Jul 11, 2022
@kilianpaquier
Copy link
Contributor

Hello @SIMULATAN, so your want the ZSH icon or Oh My ZSH icon ?

@SIMULATAN
Copy link
Author

It would be great to have both, but I think that OMZ is definitely more important, since it is the one actually doing the "magic"

@jeremyxwing
Copy link

I wonder if more people would be familiar with ZSH as a common default vs Oh My ZSH as a plugin for enthusiasts. I would assume anyone using a ZSH icon is also indicating Oh My ZSH. The Oh My ZSH icon might also not look great when it's small with all the people in it.

@SIMULATAN
Copy link
Author

OMZ isn't really a plugin for enthusiastics, but rather, it's mostly the reason for using ZSH in the first place.

@jeremyxwing
Copy link

@SIMULATAN You're not wrong! 🤘

@BenSouchet
Copy link
Contributor

BenSouchet commented Jul 25, 2022

Hi guys, I just created a PR for the Oh My Zsh logo/icon #1297 :)

@Snailedlt
Copy link
Collaborator

@SIMULATAN Nice one. Could you make a separate request for ZSH too?

Like it's explained here, every icon should have it's own separate issue -> https://github.com/devicons/devicon/wiki/Requesting-an-Icon

@BenSouchet
Copy link
Contributor

And for info the ZSH official logo : https://github.com/Zsh-art/logo
I get the link from the news page of the ZSH org: https://zsh.sourceforge.io/News/

@Snailedlt
Copy link
Collaborator

I wonder which icon they would prefer us to use for the original icon, since they have two different ones that both seem pretty official. I'd assume the one with color is the one to use, but can't find a SVG version of it.

logo with color
logo without color

@BenSouchet
Copy link
Contributor

@Snailedlt In my PR I use the one provided by the creator of Oh My Zsh here : ohmyzsh/ohmyzsh#11019 (comment)

The retro one with color isn't the main one, it's a retro variant they created after.

@Snailedlt
Copy link
Collaborator

@BenSouchet Hmm okey, both seem to be official, and the new retro one is being called their logo in some of their commit messages. I guess we can ask them directly which one they prefer :)

@BenSouchet
Copy link
Contributor

BenSouchet commented Jul 25, 2022

@Snailedlt Like I said, the creator of the tool give the the SVG logo file, why do you want to ask, or use the "retro" styled one? Since it's more complicated and also not possible to use in the devicon font.

You can still ask but no one will ever vectorize the retro one, it's terminal "ascii" art.

@Snailedlt
Copy link
Collaborator

Snailedlt commented Jul 25, 2022

@mcornella @robbyrussell
Hi! Any chance you can shed some light on what icon Oh My Zsh thinks should be included here?

@Snailedlt
Copy link
Collaborator

a.k.a. which one do you prefer, and which one is the official one?

@BenSouchet
Copy link
Contributor

Nice spam...

@mcornella
Copy link

@Snailedlt please use the vectorized old logo. The ascii art variant is not supposed to be used as the main one.

@Snailedlt
Copy link
Collaborator

Great, thank you for your quick response @mcornella 💯

@BenSouchet
Copy link
Contributor

@Snailedlt I don't want to be rude but you need to stop being so picky, you want to do things good but when you already have an anwser, you can see the logo used in the website, you have an issue answered by the creator of the tool with the SVG file, why in hell do you want to ping people ??

By doing this you devalue the work of people like me creating PRs.

@Snailedlt
Copy link
Collaborator

@BenSouchet
First of all I'm sorry if I was rude. That's never been my intention.

Imo being picky is a good thing though. It prevents bugs, typos, misunderstandings and a bunch of other problems that would otherwise need to be fixed afterwards. Overall it saves time, and improves quality.

I see you answered already, but no offense, your answer is not an official one. There have been times before, where we thought we picked the right icon, only to later figure out it wasn't. I just wanted to make sure we picked the correct one, so we didn't have to redo it.

@BenSouchet
Copy link
Contributor

@Snailedlt
After all the elements that was listed above I can't believe that a doubt remained on the icon to use, in my opinion I think it's more about your ego, you are picky (way too much) to show that you have something to say.

My opinion :
On each PR you look for things to say. Having relevant feedback is good but when it is more on the subjective side it has no place (in my opinion) in a PR thread.

I think you will realize that if you want people to continue to actively participate in open source projects you have to be cooler, things will not be always perfect for you, and it's okay / normal.

I think you have a lot of motivation for Devicon, but you can't always convince people to see things the way you do.

Last message for me in this thread, I think we can agree that we won't find a common ground on the topic and I don't want to spam.
Have a great day!

@Snailedlt
Copy link
Collaborator

@BenSouchet I think you're right about me being too picky sometimes (or maybe often). I do still think it was a good idea to ask though. It took what, like 1min to ask? and we got a concrete answer from the maintainer of the project. Now there's no doubt at all which icon we should use, and there's no need for a discussion.

You say subjective things have no say in the matter. Isn't that another reason to get an answer from the creators of the icon?

I dunno, I feel my decision was right. I do agree it can be considered spam, and picky, but I still think it was the right thing to do, and I think this is an overreaction on your part.

Now as for the more general message of your comment, I agree that I'm a bit too picky often times, and I'll work on that. Thank you for letting me know, even though I think you should have brought it up in a more respectful manner.

I hope this doesn't discourage you from contributing in the future, since I think you're a great asset to the repo :)
Have a great day you too! 💯

PS: if you want to have a chat in dm so as to not spam here, you can contact me on discord Snailed lt#3563

@Snailedlt Snailedlt added bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger in-develop The fix/feature is present in `develop` branch but not merged into `master` yet and removed bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger labels Jul 26, 2022
@BenSouchet
Copy link
Contributor

BenSouchet commented Jul 28, 2022

@SIMULATAN Hi, please don't close the issue until a new version of Devicon as been release. So we can keep track of element added and others people can see the icon is in develop but no yet available on the current version.

@Snailedlt can you re-open it.

@SIMULATAN
Copy link
Author

Implemented in b9023a5, but unreleased right now.

@SIMULATAN SIMULATAN reopened this Jul 28, 2022
@SIMULATAN
Copy link
Author

Sorry for closing it too early - I'm not on my computer and just saw the PR was merged so I figured I'd close it, sorry for not checking before taking action :/

@Snailedlt
Copy link
Collaborator

No worries :)

@mcornella
Copy link

Just for the record, it was not an issue to ask us @Snailedlt. And thanks @SIMULATAN for taking care of this :)

@Snailedlt
Copy link
Collaborator

Thank you @mcornella ❤️

The icon has been merged into develop now, and I think a new release is coming soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-develop The fix/feature is present in `develop` branch but not merged into `master` yet request:icon When requesting a new icon to be added to the collection or a refactor use this label in your issue
Projects
None yet
Development

No branches or pull requests

6 participants