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.

Sign up for GitHub

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

Jump to bottom

Add eightsleep icon #3638

Merged
merged 5 commits into from
Jan 4, 2021
Merged

Add eightsleep icon #3638

merged 5 commits into from
Jan 4, 2021

Conversation

kalpeshT101
Copy link
Contributor

Issue:#3599
Alexa rank:76,908

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

runxel
runxel requested changes Oct 2, 2020
View reviewed changes
Copy link
Member

@runxel runxel left a comment

Choose a reason for hiding this comment

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

Hey @kalpeshT101, thanks for the PR.
However your contribution doesn't match our standards, that's why the build is failing. Please have a careful look at the contribution guidelines and follow them.

Also, please don't distort icons when scaling! Always scale uniformly!

@runxel runxel added changes requested new icon Issues or pull requests for adding a new icon labels Oct 2, 2020
@IIAIronWolf
Copy link

Any update on this? My issue was mentioned in this.

@runxel
Copy link
Member

runxel commented Nov 18, 2020

Any update on this? My issue was mentioned in this.

Hey @IIAIronWolf, no, there weren't any updates by @kalpeshT101.
Let's wait a bit more until I will mark this one as abandoned.

@adamrusted
Copy link
Member

adamrusted commented Dec 30, 2020 •
edited

Assuming this PR has been abandoned, which of these variants do we think is better?
image

Have submitted with wordmark as per original PR for now, but happy to change if preferred by @simple-icons/maintainers.

eightsleep

@fbernhart
Copy link
Contributor

@adamrusted I would go for the second one - just the "8".

They as well use it in their favicon and as logo for their social media presence.

@adamrusted
Copy link
Member

adamrusted commented Dec 30, 2020 •
edited

@adamrusted I would go for the second one - just the "8".

They as well use it in their favicon and as logo for their social media presence.

@fbernhart - Consider it done! Rebuilt from press kit, and used hex found in SVG there.

eightsleep

@IIAIronWolf
Copy link

Looks great! Thanks!

@adamrusted adamrusted linked an issue Jan 1, 2021 that may be closed by this pull request
Eight Sleep Icon #3599
Closed
@fbernhart
Copy link
Contributor

@adamrusted LGTM! 8️⃣ Thanks for rescuing this one.

Thanks as well to @kalpeshT101 for the initial try. 👍🏼

@fbernhart fbernhart merged commit 217dc76 into simple-icons:develop Jan 4, 2021
@github-actions github-actions bot mentioned this pull request Jan 10, 2021
Merged
ericcornelissen added a commit that referenced this pull request Jan 10, 2021
@ericcornelissen
Release 103 new icons and 41 updated icons (v4.6.0)
701d605 
# New Icons

- 3M (#3631)
- Accenture (#4396)
- AIOHTTP (#4536)
- AniList (#4609)
- AnyDesk (#4663)
- Apache JMeter (#4594)
- Apache Kylin (#4545)
- ARK Ecosystem (#4699)
- B&R Automation (#4552)
- bigbasket (#4572, #4616)
- Bugcrowd (#4615)
- CheckiO (#4512)
- Chrysler (#4357)
- Cloudways (#3352)
- Cockroach Labs (#4665)
- CodeProject (#4617)
- CodinGame (#4575)
- Cognizant (#4392)
- Contentful (#4537)
- Cookiecutter (#3649)
- DataStax (#4393)
- De'Longhi (#4426)
- Deepnote (#4546)
- Digi-Key Electronics (#4398)
- Eclipse Vert.x (#4631)
- edX (#4452)
- egghead (#4644)
- Eight Sleep (#3638)
- Facepunch (#3671)
- FedEx (#4564)
- Fortran (#4397)
- FreeNAS (#4573)
- Gitee (#4625)
- Google Domains (#4622)
- Google Photos (#4341)
- Grubhub (#4544)
- Homebridge (#4607)
- Hyperledger (#4673)
- IKEA (#4555)
- Ingress (#3389)
- Jitsi (#4540)
- Kitsu (#4520)
- Klook (#4548)
- Lada (#4361)
- Lichess (#3535)
- Linktree (#4662)
- Looker (#4442)
- McLaren (#4538)
- MetaFilter (#4547)
- Motorola (#3425)
- Noun Project (#4608)
- Observable (#4640)
- Open Bug Bounty (#4627)
- Open Nebula (#4551)
- OpenCV (#4580)
- PageSpeed Insights (#4553)
- Paytm (#4643)
- pfSense (#4671)
- Pop!_OS (#3769)
- Progate (#3680)
- Progress (#4611)
- PyPy (#4281)
- Racket (#3483)
- Radar (#4319)
- Rainmeter (#3554)
- Reliance Industries Limited (#4702)
- Revue (#4391)
- ROS (#4658)
- SciPy (#3772)
- Scrimba (#4556)
- SEPA (#3154)
- SoloLearn (#4624)
- Spark AR (#4468)
- Speedtest (#4626)
- STARZ (#4612)
- Streamlit (#4412)
- Swiper (#4562)
- São Paulo Metro (#4570)
- tado° (#4637)
- TaxBuzz (#4619)
- Testin (#4571)
- Testing Library (#4542)
- ThinkPad (#4550)
- Ticketmaster (#4526)
- Tile (#4614)
- tmux (#3512)
- Treyarch (#4690)
- TrueNAS (#4574)
- Unraid (#4613)
- UPS (#4565)
- USPS (#4549)
- Vault (#4606)
- Vimeo Livestream (#4703)
- Webflow (#4568)
- Weights & Biases (#4630)
- Wikiquote (#4687)
- Winmate (#3316)
- Wipro (#4359)
- Xilinx (#3424)
- Zelle (#4605)
- Zoho (#3661)
- Zomato (#4664)
- Zotero (#4561)

# Updated Icons

- Adobe XD (#3386)
- Amazon Lumberyard (#4591)
- Andela (#4592)
- Apache Maven (#4593)
- Daimler (#4618)
- DigitalOcean (#4648)
- Fossa (#4473)
- Gatsby (#4620)
- GIPHY (#4621)
- Google (#4508)
- Google Analytics (#4504)
- Google Cardboard (#4058)
- Google Fit (#4506)
- Harbor (#4598)
- Jira Software (#4657)
- Kahoot! (#4635)
- Kotlin (#4485)
- Kyocera (#4475)
- Lenovo (#4477)
- Material Design Icons (#4642)
- Mozilla (#4656)
- pr.co (#4655)
- SonicWall (#4654)
- StackPath (#4652)
- Stadia (#4503)
- Synology (#4220)
- TED (#4653)
- Trustpilot (#4481)
- Uber (#4650)
- Uber Eats (#4651)
- Udacity (#4596)
- Viber (#4597)
- Waze (#4583)
- Wish (#4482)
- WP Engine (#4639)
- XMPP (#4483)
- YouTube (#4582)
- YouTube Studio (#4581)
- YouTube TV (#4405)
- Zendesk (#4595)
- Zillow (#4636)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fbernhart fbernhart Awaiting requested review from fbernhart

@runxel runxel Awaiting requested review from runxel

Assignees
No one assigned
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Eight Sleep Icon
5 participants
@kalpeshT101 @IIAIronWolf @runxel @adamrusted @fbernhart