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

update icon: appwrite (original, original-wordmark, plain, plain-wordmark) #2196

Merged
merged 5 commits into from
Dec 23, 2024

Conversation

Yashh56
Copy link
Contributor

@Yashh56 Yashh56 commented May 23, 2024

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 #2181

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

The color #fd366e was selected for the font SVGs as it's the main color in the original logo.

@canaleal canaleal changed the title update icon : appwrite update icon: appwrite (original, original-wordmark, plain, plain-wordmark) Jun 2, 2024
@Yashh56
Copy link
Contributor Author

Yashh56 commented Jun 7, 2024

Hey Maintainer Please Review this PR

@Snailedlt Snailedlt requested review from a team, ConX, weh, Snailedlt, canaleal and lunatic-fox and removed request for a team June 7, 2024 14:54
Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution!

The wordmark versions look very good!

The non-wordmark versions are distorted. Please make sure to keep the aspect ratios the same when resizing the icons, so that they don't become distorted.

icons/appwrite/appwrite-original.svg Show resolved Hide resolved
icons/appwrite/appwrite-plain.svg Outdated Show resolved Hide resolved
@Yashh56
Copy link
Contributor Author

Yashh56 commented Jun 7, 2024

Thanks @Snailedlt for reviewing this PR Can you tell me how to resolve this issue ?

@Snailedlt
Copy link
Collaborator

Snailedlt commented Jun 7, 2024

Thanks @Snailedlt for reviewing this PR Can you tell me how to resolve this issue ?

@Yashh56 I think the easiest way is downloading the official icons again and making sure that the aspect ratio is maintained when you resize it :)

If you're using inkscape you can follow this guide when resizing: #1306 (comment)

PS: there's lots of other great guides in that issue as well!

@Yashh56 Yashh56 requested a review from Snailedlt June 8, 2024 11:42
Copy link
Contributor

github-actions bot commented Jun 8, 2024

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60


Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

@Yashh56
Copy link
Contributor Author

Yashh56 commented Jun 15, 2024

Hey Maintainer Please Review this PR

weh
weh previously requested changes Jun 17, 2024
Copy link
Contributor

@weh weh left a comment

Choose a reason for hiding this comment

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

It looks like the changes requested by @Snailedlt are still not resolved.
The original and plain version are still squished, and not round anymore.

squished

@weh
Copy link
Contributor

weh commented Jun 17, 2024

...also original and plain could be aliased since it is only a single color (?)

@weh
Copy link
Contributor

weh commented Jun 17, 2024

And i forgot also that the original and plain should not have that much spacing around the icon.
So to summarize again:

  • Fix the squished aspect ratio
  • Remove the spacing around the Icon
  • Remove the plain-wordmark / original-wordmark alias
        "aliases": [
            {
                "base": "plain",
                "alias": "original"
            }
        ]

@Snailedlt
Copy link
Collaborator

Refer to guides under this issue as well as the wiki if you're unsure how to fix the issues :)

@Yashh56
Copy link
Contributor Author

Yashh56 commented Jun 17, 2024

how can i squished aspect ratio ?

@Yashh56
Copy link
Contributor Author

Yashh56 commented Jun 17, 2024

I used Resize to resize the aspect ratio

@Snailedlt
Copy link
Collaborator

@Yashh56 If you're using inkscape, make sure you check the box to scale proportionally
image
Otherwise the svg will get stretched to fit the viewbox.

Here's two different guides to resize to a square. Both can be found as comments in the issue I linked to earlier:

@Snailedlt Snailedlt added the feature:icon Use this label for pull requests when a new icon is ready to be added to the collection label Jul 1, 2024
Copy link
Contributor

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60


Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

@Yashh56
Copy link
Contributor Author

Yashh56 commented Oct 26, 2024

Hey @Snailedlt, Can you review my changes :)

@Yashh56 Yashh56 requested a review from weh November 1, 2024 16:16
Copy link
Contributor

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60


Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

Copy link
Member

@canaleal canaleal left a comment

Choose a reason for hiding this comment

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

Ive optimized the svgs using this tutorial and made sure there were no artifacts in icomoon.

The eps file can be removed as it's not needed anymore and because the original icon is simple enough to be used as the plain icon, the plain svg was removed and replaced with an alias.

image

@canaleal canaleal dismissed stale reviews from weh and Snailedlt December 23, 2024 02:12

All issues have been resolved in the latest commit.

Copy link
Contributor

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60


Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

@canaleal canaleal merged commit dea97b3 into devicons:develop Dec 23, 2024
5 checks passed
@canaleal canaleal mentioned this pull request Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature:icon Use this label for pull requests 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.

4 participants