8000 new icon: kdeneon (original,plain) by dotarjun · Pull Request #1777 · devicons/devicon · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

new icon: kdeneon (original,plain) #1777

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

Closed
wants to merge 0 commits into from
Closed

Conversation

dotarjun
Copy link
Contributor

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

#1754

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

https://en.wikipedia.org/wiki/KDE_neon

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.

  • Optimize SVG code. Use SVGOMG.
  • Remove height and width and replace by viewBox.

@Panquesito7 Panquesito7 added the feature:icon PR when a new icon is ready to be added to the collection label May 31, 2023
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.

There also seem to be merge conflicts.
Please fix those. If you need any help, let us know. 🙂

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.

Hello there! 👋🏼
Since you based this pull request on your master branch dotarjun:master, I recommend you to do another pull request but based on your develop branch.

Here goes some git commands to help on this procedure:

git checkout develop                # Go to "develop" branch.
git merge upstream/develop develop  # Update your "develop" branch with "devicons:develop".
git push origin                     # Push the update to your remote "develop" branch.
git branch add_kdeneon              # Create a "add_kdeneon" branch.
git checkout add_kdeneon            # Go to "add_kdeneon" branch.
git checkout master icons/kdeneon   # From your current "master" branch pick up just the icons.

Now that you're still on add_kdeneon branch open devicon.json and add your entry, between karma: line 4917 and keras: line 4918 entries:

    {
        "name": "kdeneon",
        "altnames": [],
        "tags": [
            "linux",
            "operating system",
            "open-source"
        ],
        "versions": {
            "svg": [
                "original",
                "plain"
            ],
            "font": [
                "plain"
            ]
        },
        "color": "#21769a",
        "aliases": []
    },

Once done just publish (commit) this new branch and make the pull request again. 🙂

After that you can close this pull request too. 👍🏼


You can also reset the changes you made on your master branch by the last commit 1119b9f84c0290e0f0b38982099a2bd027a48bf1:

git reset --hard 1119b9f84c0290e0f0b38982099a2bd027a48bf1
git push -f origin

💡 Remember in future contributions to always base your branches on your develop branch.

Wish for the best! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comme 72D6 nt
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.

4 participants
0