-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Add Homepage icon #10452
Add Homepage icon #10452
Conversation
Honestly would prefer if there was a line in the top right, but the svg that they provided is the official one. So it has to stay this way. |
@21st-centuryman, the source icon was created with Potrace which is a tool for converting raster images to vectors. It often produces very poor results and a lot of people don't seem to bother reviewing them, so we only ever use them as a starting point at the very most. If you feel it needs to be tweaked to make it more faithful to the actual icon or that it would benefit from a cut-out then feel free to do so 👍🏻 |
@PeterShaggyNoble i think it's best I don't mess with it. After all I'm not associated with the project, I'm just a fan. Once they have a better svg I'll update it then on simple-icons. |
I will try to talk to the group on discord so they can update their logo, I don't feel comfortable changing it. For now I think it's fine to merge. |
Did you hear anything back from them yet, @21st-centuryman? |
@PeterShaggyNoble we should just use the one mentioned by @shamoon. I had some discussion with other members and this was the consensus. thank you @shamoon, hope you have a great rest of the week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking the time to respond, @shamoon 👍🏻
Given that feedback, we're good to run with this treatment, @21st-centuryman - the SVG lines up perfectly with the source but it does contain a good few excess points. Could you see if you could remove them before we merge this?
@PeterShaggyNoble I am not sure what you mean by excess points. Do you want me to do something different with svgo? |
See here, @21st-centuryman; you should be able to delete all points along the straight lines as well as one of the overlapping points in the top-left. As the source SVG is generated by Potrace, I wouldn't be too concerned if any minor differences are introduced as a result. |
@PeterShaggyNoble Does this look good? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like work on the bottom of this icon has introduced some weirdness - see the source in red and your PR in black....
Would you mind having another look @21st-centuryman? Might just need to be a higher precision on your PR version :)
@adamrusted I believe these are caused by the excess points I was told to remove earlier, do you want me to revert to commit d47f958 or should I still remove these points but to a satisfying level? I am fine with both. |
Hey @21st-centuryman - I think there's a happy medium somewhere between what's currently in this PR and the source icon. If you keep a local copy of the source icon and compare as you remove points, remove just enough that it's 'minimized' the amount of excess points without affecting the shape (if that's possible!) |
I did my best attempt at this. there will be some small changes I don't think I can avoid that. If you guys are super against any changes I would recommend just using commit d47f958 if that is what you want ill just update the branch. |
I'm confused by the heart reaction. Does that mean my latest commit was unsatisfactory and I should revert to the earlier commit I mentioned? Thank you :) |
Ah, the joys of Potrace - none of the straight lines are actually straight! 🙄 The unoptimised path below is the best I could manage, if you want to run with it. Although, perhaps @adamrusted could throw an eye over it first before you go any further.
|
@PeterShaggyNoble Thank you for your help, i committed the changes. |
Thanks for the assist @PeterShaggyNoble! |
@adamrusted $ npx svgo -p 3 icons/homepage.svg
homepage.svg:
Done in 13 ms!
0.745 KiB - 0% = 0.745 KiB I am not sure what more I could do. do you want me to decrease the precision? here are some outputs $ npx svgo -p 2 icons/homepage.svg
homepage.svg:
Done in 12 ms!
0.745 KiB - 22.8% = 0.575 KiB below is the path for precision of 2 M2 .03C1.21.17.6.84.41 1.77.39 1.9.36 21.86.39 23.9c0 .18 0 .18.92-.6.82-.7 1.65-1.37 2.44-2.09 0-.04.02-1.15.02-2.47l-.01-2.39h1.52c.83 0 1.52.02 1.53.04.02.03.03.54.03 1.15 0 .6 0 1.1.02 1.1.53-.43 1.05-.88 1.57-1.32l1.38-1.15c.28-.23.34-.26.4-.2l4.1 3.44c4.9 4.12 3.1 2.64 5.07 4.27.22.19.42.33.44.32.02-.01.04-3.33.04-7.38 0-6.57 0-7.37.08-7.38.04-.01.9-.02 1.88-.02h1.8l-.02-3.76c0-2.07-.03-3.82-.05-3.88A2.54 2.54 0 0 0 22.67.3C22.2 0 22.83 0 12.1 0 6.66 0 2.11.01 2 .03Z |
@21st-centuryman We usually aim for a path with a precision between 3 and 5, so the linter is likely to fail again on the precision of 2 😅 If a precision of 3 isn't showing any differences compared to Peter's original path, then let's go with that. |
ok i did some other changes. check now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for your work on this both!
…enamed icons (v12.0.0) ## New Icons - Audiobookshelf (#10605) (@21st-centuryman) - Awwwards (#10539) (@stefanobartoletti) - Foundry Virtual Tabletop (#10575) (@adamrusted) - FusionAuth (#10776) (@service-paradis) - Homepage (#10452) (@21st-centuryman) - Hugging Face (#9259) (@PratyushJayachandran) - Kagi (#10976) (@uncenter) - n8n (#10909) (@CodeShakingSheep) - Posit (#10987) (@cwickham) - React Bootstrap (#10921) (@CodeShakingSheep) - Sartorius (#10967) (@CodeShakingSheep) - UiPath (#10773) (@Mvlprem) - unpkg (#10317) (@service-paradis) - Zigbee2MQTT (#10212) (@PeterShaggyNoble) ## Updated Icons - Adidas (#10982) (@jglamp) - FastAPI (#10663) (@uncenter) - RenovateBot (#10996) (@sisp) ## Removed Icons - FITE (#10060) (@PeterShaggyNoble) - Flattr (#10380) (@service-paradis) - Google Bard (#10453) (@PeterShaggyNoble) - Integromat (#10152) (@PeterShaggyNoble) - Niantic (#10979) (@PratyushJayachandran) - Nintendo Network (#10006) (@adamrusted) - Rome (#10113) (@PeterShaggyNoble) - Shotcut (#10043) (@PeterShaggyNoble) - Skynet (#10504) (@kwypchlo) - Twitter (#9748) (@adamrusted) ## Renamed Icons - `Airbrake.io` to `Airbrake` (#10513) (@PeterShaggyNoble) - `Amazon Web Services` to `Amazon Web Services` (#10391) (@readboy8) - `RStudio` to `RStudio IDE` (#10999) (@adamrusted) ## SDK Changes ### URL_REGEX -> urlRegex() The exported const `URL_REGEX` has been removed. Please use `urlRegex()` instead: ```javascript import {urlRegex} from 'simple-icons/sdk'; const regex = await urlRegex(); ``` ### .jsonschema.json Now `.jsonschema.json` file is bundled in our package. For more details, please see: #10276.
Issue: closes #10450
https://github.com/gethomepage/homepage
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description