Skip to content

Conversation

@dotarjun
Copy link
Contributor

@dotarjun dotarjun commented Jun 3, 2023

Double check these details before you open a PR

  • PR does not match another non-stale PR currently opened
  • PR name matches the format new icon: Icon name (versions separated by comma). More details here
  • PR's base is the develop branch.
  • Your icons are inside a folder as seen here
  • SVG matches the standards laid out here
  • A new object is added in the devicon.json file as seen here

This PR closes NONE

#1776

Link to prove your SVG is correct and up-to-date.

https://framework7.io/

@dotarjun dotarjun changed the title new icon: framework7 (original,plain) new icon: framework7 (original,plain,original-wordmark, plain-wordmark) Jun 3, 2023
@dotarjun dotarjun changed the title new icon: framework7 (original,plain,original-wordmark, plain-wordmark) new icon: framework7 (original, plain, original-wordmark, plain-wordmark) Jun 3, 2023
@Gautam25Raj
Copy link
Contributor

Maybe we don't need this plain version. Both plain and original have single color. What do you think? 🤔

@dotarjun
Copy link
Contributor Author

dotarjun commented Jun 3, 2023

agreed

dotarjun and others added 3 commits June 4, 2023 00:18
- Rename icons from "framework-[version].svg" to "framework7-[version].svg".
- Remove "style" attribute from `original` version.
- Remove trailing comma in `devicon.json`.
@lunatic-fox lunatic-fox changed the title new icon: framework7 (original, plain, original-wordmark, plain-wordmark) new icon: framework7 (original, original-wordmark) Jun 4, 2023
lunatic-fox
lunatic-fox previously approved these changes Jun 4, 2023
Copy link
Contributor

@lunatic-fox lunatic-fox left a comment

Choose a reason for hiding this comment

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

Icons are great! ✔
Thank you for contributing! 🚀

@lunatic-fox lunatic-fox requested a review from Panquesito7 June 4, 2023 09:43
Copy link
Contributor Author

@dotarjun dotarjun left a comment

Choose a reason for hiding this comment

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

I keep on missing these minor intricacies haha

@Gautam25Raj
Copy link
Contributor

I keep on missing these minor intricacies haha

Same for me. 😆

Gautam25Raj
Gautam25Raj previously approved these changes Jun 4, 2023
Copy link
Contributor

@Gautam25Raj Gautam25Raj left a comment

Choose a reason for hiding this comment

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

Great Contribution. 🚀

@Panquesito7 Panquesito7 added the feature:icon PR when a new icon is ready to be added to the collection label Jun 4, 2023
@dotarjun dotarjun dismissed stale reviews from Gautam25Raj and lunatic-fox via 3b063ae June 5, 2023 05:35
@gustavoSutil
Copy link

thank you, guys!

@lunatic-fox lunatic-fox requested a review from Panquesito7 June 8, 2023 06:52
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Amazing. Thanks! 🚀

@Panquesito7 Panquesito7 merged commit c229e7c into devicons:develop Jun 11, 2023
@Snailedlt Snailedlt mentioned this pull request Feb 5, 2024
GCHQDeveloper926 pushed a commit to GCHQDeveloper926/devicon that referenced this pull request Dec 20, 2024
* add framework7 original and plain icons

* update devicons.json with framework7

* remove color from plain svg

* resize to 128

* try to resize again

* remove grid and fix svg

* optimise svgs

* add original and plain wordmark versions

* rename according to the naming convention

* add wordmark versions

* remove width/height and use viewBox instead

* resize and optimise svg

* resize and optimise original wordmark svg

* add plain wordmark to font array

* remove plain versions

* Add aliases and remove plain, plain-wordmark versions

* Add newline and some fixes:
- Rename icons from "framework-[version].svg" to "framework7-[version].svg".
- Remove "style" attribute from `original` version.
- Remove trailing comma in `devicon.json`.

* add open-source under tags

---------

Co-authored-by: Josélio Júnior <[email protected]>
Co-authored-by: David Leal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature:icon PR when a new icon is ready to be added to the collection

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants