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 mdBook icon #7289

Merged
merged 2 commits into from May 9, 2022
Merged

Add mdBook icon #7289

merged 2 commits into from May 9, 2022

Conversation

zyw271828
Copy link
Contributor

image

Issue: closes None

Similarweb rank: ~196k

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

  • mdBook is an alternative to GitBook for generating online books from Markdown
  • Its GitHub repository currently has 9.1k stars
  • I took the SVG from its favicon
  • I used Inkscape to resize viewbox, resize icon and align
  • I used the SVGO command line tool to optimize
  • I compared with the original SVG
  • mdBook's website doesn't have any iconic colors, so I chose #000000

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Mar 31, 2022
Copy link
Contributor

@sisp sisp left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

This is the diff between this icon and the original one referenced as the source:

mdbook-diff

IMO, this PR can be merged.

@sachinraja sachinraja merged commit f988200 into simple-icons:develop May 9, 2022
mondeja added a commit that referenced this pull request May 16, 2022
# New Icons

- EnterpriseDB (#7379)
- GTK (#7340)
- mdBook (#7289)
- RuboCop (#7380)
- Steam Deck (#7411)

# Updated Icons

- Ubuntu (#7407)
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

3 participants