-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Comments
Hello @SIMULATAN, so your want the ZSH icon or Oh My ZSH icon ? |
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" |
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. |
OMZ isn't really a plugin for enthusiastics, but rather, it's mostly the reason for using ZSH in the first place. |
@SIMULATAN You're not wrong! 🤘 |
Hi guys, I just created a PR for the Oh My Zsh logo/icon #1297 :) |
@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 |
And for info the ZSH official logo : https://github.com/Zsh-art/logo |
@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. |
@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 :) |
@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. |
@mcornella @robbyrussell |
a.k.a. which one do you prefer, and which one is the official one? |
Nice spam... |
@Snailedlt please use the vectorized old logo. The ascii art variant is not supposed to be used as the main one. |
Great, thank you for your quick response @mcornella 💯 |
@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. |
@BenSouchet 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. |
@Snailedlt My opinion : 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. |
@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 :) PS: if you want to have a chat in dm so as to not spam here, you can contact me on discord |
@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. |
Implemented in b9023a5, but unreleased right now. |
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 :/ |
No worries :) |
Just for the record, it was not an issue to ask us @Snailedlt. And thanks @SIMULATAN for taking care of this :) |
Thank you @mcornella ❤️ The icon has been merged into develop now, and I think a new release is coming soon |
I have searched through the issues and didn't find my problem.
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
The text was updated successfully, but these errors were encountered: