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 icons #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add icons #2

wants to merge 2 commits into from

Conversation

vedam
Copy link
Member

@vedam vedam commented May 12, 2019

  • add icons for use in text-editors
  • add folder structure
  • add small README
  • add package.json for versioning

#closes #1

Hey,
pffhheeww, sorry for taking this long, but Adobe Illustrator punished me for using symbols.
Took some time to figure this out.

It's possible to shave some bits of the .svgs by handcrafting out title-attributes and g-tags if wished.
I'll do that, but want to ask first.

Native speakers: run through the README and feel free to correct where necessary.

concerning icons in Text-editors
I saw on vscode-icons #2033 that there are added Icons already. Does that mean, the extension-providers care of that themselves?
Or do WE have to open pull-requests on all the different icon-extensions (there are quite a few, having a look at vscode Visual Studio Code alone)

concerning license
I'm a bit unsure, if running under MIT means, that everybody is able to use/abuse our Logo?
Hopefully someone who has a clue can help with this.

- add icons for use in text-editors
- add folder structure
- add small README
- add package for versioning

#closes #1
Copy link
Member

@mindrones mindrones left a comment

Choose a reason for hiding this comment

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

awesome, 👌 💯 for me, well done!

.gitignore Show resolved Hide resolved
Copy link
Member

@mindrones mindrones left a comment

Choose a reason for hiding this comment

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

Noted some minor needed fixes

README.md Outdated

## License

[]()
Copy link
Member

Choose a reason for hiding this comment

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

missing Licence

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah. I'm unsure about this and asked in the comment.

Copy link
Member

@mindrones mindrones May 18, 2019

Choose a reason for hiding this comment

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

Oops, right, sorry.

I had a look at creative commons (https://creativecommons.org/choose/):

  • if you don't want others to derive anything from the logos there's CC BY-NC-ND 4.0, that is Attribution-NonCommercial-NoDerivatives 4.0 International;
  • if you're ok with others making new things with the logos there's CC BY-NC-SA 4.0, that is Attribution-NonCommercial-ShareAlike 4.0 International.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
* Either 1x or 4x resolution
* .png or .svg

![svelte icon 32x32](https://github.com/sveltejs/branding/icons/svelte-32.png)
Copy link
Member

Choose a reason for hiding this comment

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

I think it should be ![svelte icon 32x32](./icons/svelte-32.png) so that it works independently from the branch we're in.

- use `svelte-vertical` if your Layout isn't suited a for the use of `svelte-horizontal`
- for print you'll have to separate the colors from RGB to 4c yourself, according to your print-process

![svelte-horizontal logo](https://github.com/sveltejs/branding/logos/svelte-horizontal.png)
Copy link
Member

Choose a reason for hiding this comment

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

and here ![svelte icon 32x32](./logos/svelte-horizontal.png)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add small icons
2 participants