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

Add OpsLevel icon (#1736) #1890

Merged
merged 2 commits into from Jan 8, 2020
Merged

Conversation

bonez0607
Copy link
Contributor

@bonez0607 bonez0607 commented Nov 7, 2019

Issue: #1736

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

  1. logomark used as referenced in PR OpsLevel Icon added #1736 #1761 by @birjolaxew. Removed "OpsLevel" from svg-grabber
  2. Hex color: #1890FF - Color taken from downloaded svg logo

@phatbhoy67
Copy link
Contributor

Hey @bonez0607, thanks for picking up this issue and your work on this PR. 👍

Can you please confirm the source used for the icon. Looking at the site header logo .svg when compared to your PR version the lines are quite a bit thicker in the official version. Image below - official .svg red, PR version black placed on top.

I believe (https://www.opslevel.com/images/OpsLevelLogoText.svg) is the same .svg you would get with svg grabber so not sure why the lines would be different even after scaling down to 24px from 28.291px (height of the official logo).

stroke-width

@ericcornelissen ericcornelissen added new icon Issues or pull requests for adding a new icon awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed labels Nov 8, 2019
@bonez0607
Copy link
Contributor Author

@ericcornelissen @phatbhoy67 - when I grabbed it and imported to Illustrator everything was a strokes line. I outlined the stroke looking to reduce the number of paths and am wondering if that’s why it ended up being a little larger.

Will take another look when I get a chance, but might be another day or two

@phatbhoy67
Copy link
Contributor

phatbhoy67 commented Nov 10, 2019

it ended up being a little larger

Hey @bonez0607, thanks for replying.

When you have the time to have a look at this again, the version you submitted in the PR has thinner lines than I would have expected, not thicker.

@birjj birjj added changes requested and removed awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed labels Nov 26, 2019
@PeterShaggyNoble
Copy link
Member

Here's my attempt at it, @bonez0607, if it's any help - click the image to download. Your version is overlaid in red for reference.

OpsLevel

@PeterShaggyNoble
Copy link
Member

@bonez0607, are you still working on this one?

@ericcornelissen
Copy link
Contributor

@PeterShaggyNoble I think it is not a problem if you open a new PR for this brand (or, preferably, push changes to this PR if that is possible) 👍

@PeterShaggyNoble
Copy link
Member

Updated the SVG in @bonez0607's branch.

@phatbhoy67
Copy link
Contributor

Thanks for your work on this PR @bonez0607 and thanks to @PeterShaggyNoble for updating the .svg 👏 👍

@phatbhoy67 phatbhoy67 merged commit fff40e0 into simple-icons:develop Jan 8, 2020
@phatbhoy67 phatbhoy67 mentioned this pull request Jan 8, 2020
ericcornelissen added a commit that referenced this pull request Jan 20, 2020
New icons

- Bulma (#2290)
- Bluetooth (#2293)
- Google Assistant (#2296)
- Emby (#2304)
- OpsLevel (#1890)
- ioBroker (#2302)
- Douban (#2303)
- Apache Pulsar (#2314)
- Amazon Lumberyard (#2232)
- Bandsintown (#2319)
- Discogs (#2320)
- ChartMogul (#2308)
- Just Eat (#2109)
- KeePassXC (#2273)
- Parity Substrate (#2324)
- Slackware (#2289)
- Grunt (#2321)
- Transport for Ireland (#2323)
- Quest (#2325)
- Skype for Business (#2337)
- Rhino (#2197)
- Wear OS (#2305)
- NetApp (#2327)
- Kyocera (#2328)
- Fortinet (#2329)
- Citrix (#2330)
- Apache Ant (#2342)
- protocols.io (#2344)
- Google Maps (#2196)
- The Register (#2346)
- Tide (#2307)
- Mocha (#2347)
- Citroën (#2359)
- Opel (#2361)
- Flood (#2349)
- Google Lens (#2354)
- Google Fit (#2355)
- LogMeIn (#2356)
- Das Erste (#2198)
- CD Projekt (#2292)
- HelloFresh (#2295)
- Polymer Project (#2353)
- Jenkins X (#2333)
- Material Design Icons (#2371)
- Apache Maven (#2374)
- LMMS (#2350)
- FedRAMP (#2318)
- Semantic Web (#2372)
- Claris (#2381)
- XAMPP (#2086)
- Pixabay (#2301)
- AngularJS (#2379, b95baf4)
- Aiqfome (#2400)
- Airbus (#2396)

Updated icons

- SourceForge (#2294)
- Xamarin (#2297)
- Fedora (#2300)
- Brave (#2362)
- Microsoft Edge (#2378)
@bonez0607 bonez0607 deleted the OpsLevel branch June 4, 2020 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants