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

Correct Accusoft icon #3751

Merged
merged 4 commits into from Jan 12, 2021
Merged

Conversation

lukeocodes
Copy link
Contributor

accusoft

Issue: #3169
Alexa rank:

Checklist

  • I updated the JSON data in _data/simple-icons.json
  • I optimized the icon with SVGO or SVGOMG
  • The SVG viewbox is 0 0 24 24

Description

  • Took existing SVG.
  • Updated the image in Illustrator, resizing to canvas and centering.
  • Ran npm run svgo command line optimizer.

@github-actions github-actions bot added the update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both label Oct 21, 2020
@lukeocodes
Copy link
Contributor Author

Hey @service-paradis, as you've done loads would you help and sanity check this one for me? If it's okay, I can go and do a load more.

Illustrator doesn't seem to enjoy the precision at 24px, so in the raw code everything is at or below 24, but after I optimize it and open it back up in Illustrator, it says the width is 24.0414px 🙄

I'm pretty confident it's wrong...

@ericcornelissen
Copy link
Contributor

ericcornelissen commented Oct 21, 2020

Illustrator doesn't seem to enjoy the precision at 24px, so in the raw code everything is at or below 24, but after I optimize it and open it back up in Illustrator, it says the width is 24.0414px

I'm pretty confident it's wrong...

Actually, it is likely due to optimizing. Part of the optimization is reducing the precision and this may change the width (and height) of the SVG. You can try increasing the precision of the optimization (if you tell us what you're using to do the optimization we can help you with how to do that) to the point where the width no longer changes.

Also, you can try removing Accusoft from the .svglint-ignored.json file and running npm run svglint (without SI_UPDATE_IGNORE=true!) again to check if it worked. If this command does not give any errors it is indeed the case that Illustrator is wrong.

@service-paradis
Copy link
Member

I really had a hard time trying to fix or optimize icon using Illustrator. I finally used Inkscape. Here is a short summary of my process:

  1. Open the current SVG (or the official one if needed) using Inkscape
  2. Use the node editing tool to fix path if needed
    image
  3. While in Node editing mode, I use the transform tool to precisely scale the element and the canvas (sorry for the french capture). The checkboxes are important there (Scale proportionally and Apply to each object separately)
    image
  4. I switch back to the select tool
    image
  5. While in select mode, I use the Align and distribute tool to center the object vertically and horizontally
    image
  6. Finally, I use https://jakearchibald.github.io/svgomg/ to optimize

Copy link
Contributor

@fbernhart fbernhart left a comment

Choose a reason for hiding this comment

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

@lukeocodes Thanks for your work.

Unfortunately it seems like you've optimized the icon with a precision of 1, which leads to a loss of detail and deforms the icon (your version is the red one). Can you instead optimize it with a precision of 3?

image

And please as well remove the entry for Accusoft from the file .svglint-ignored.json.

@fbernhart fbernhart added the awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed label Dec 15, 2020
@lukeocodes
Copy link
Contributor Author

lukeocodes commented Dec 15, 2020 via email

@fbernhart
Copy link
Contributor

@lukeocodes It's easily doable with Inkscape. 😊

  • Open the original version.
  • Resize it to 24x24, center it.
  • File --> Import; Open your new version.
  • Give your new version a coloured fill (for example red) with an opacity. (Object --> Fill and Stroke)
  • Center your new version.

@adamrusted adamrusted removed the awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed label Jan 3, 2021
@fbernhart
Copy link
Contributor

I took the liberty to update this PR with the needed changes, as there has been no activity in the last few weeks.

I've rebuilt it, using the "Accusoft Icon (.SVG)" file from the source. I had to use a precision of 4 to pass the linter.

Copy link
Member

@adamrusted adamrusted left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @lukeocodes and @fbernhart for your work on this one!

@adamrusted adamrusted merged commit 3d6ab06 into simple-icons:develop Jan 12, 2021
ericcornelissen added a commit that referenced this pull request Jan 17, 2021
# New Icons

- Acer (#4680)
- AliExpress (#4712)
- Bank of America (#4722)
- Blazor (#4694)
- Blueprint (#4729)
- CodeceptJS (#3399)
- Codemagic (#4711)
- Cucumber (#4754)
- Deutsche Bahn (#4210)
- Devpost (#4771)
- Espressif (#4720)
- Flipkart (#4734)
- Folium (#4688)
- FutureLearn (#4773)
- Game Jolt (#4742)
- Git LFS (#4737)
- Google Optimize (#4785)
- Graylog (#4675)
- Headspace (#4765)
- Home Assistant Community Store (#4755)
- Hotjar (#4723)
- Jellyfin (#4674)
- Kali Linux (#4768)
- Less (#4693)
- Metro (#3183)
- Momenteo (#4676)
- NBA (#4739)
- New Japan Pro-Wrestling (#2791)
- OWASP (#4783)
- Pioneer DJ (#4709)
- PlayStation 5 (#4641)
- Postmates (#4781)
- Presto (#4623)
- QuickBooks (#4782)
- Rolls-Royce (#4772)
- Rotten Tomatoes (#4770)
- Selenium (#4769)
- Semantic UI React (#4743)
- Southwest Airlines (#2896)
- Taobao (#4632)
- The Irish Times (#4752)
- TP-Link (#4732)
- Unacademy (#4689)
- VOX (#2205)
- WALKMAN (#4670)
- Wikidata (#4738)
- Yale (#4721)

# Updated Icons

- Accusoft (#3751)
- Citrix (#4748)
- Ferrari (#4778)
- General Motors (#4741)
- Google Chat (#4695, #4736)
- Intel (#4751)
- Kahoot! (#4731)
- Kia (#4730)
- Microsoft (#4649)
- Visa (#4719)
- Webflow (#4715)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants