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

New icon proposal #1

Merged
merged 1 commit into from
Feb 22, 2021
Merged

New icon proposal #1

merged 1 commit into from
Feb 22, 2021

Conversation

ragnilorenzo
Copy link
Contributor

Hi, these are the files relating to the new icon proposal we discussed in issue #6 on woeusb-ng branch.

I uploaded both the source code from pixelmator pro and the png export.

I kept the original resolution of 1501x1501 because the files are pretty light; if you need it can scale down to 512x512px or even to a lower resolution.

These files are licensed under GPL v3

SPDX-License-Identifier: GPL-3.0-or-later
Signed-off-by: Lorenzo Ragni lorenzo.ragni@outlook.it

(I hope I didn't make a mess with the pull procedure)

SPDX-License-Identifier: GPL-3.0-or-later
Signed-off-by: Lorenzo Ragni <lorenzo.ragni@outlook.it>
@brlin-tw
Copy link
Member

brlin-tw commented Feb 22, 2021

Screenshot_20210222_182512

You appear to be adding empty files, try adding them properly to the commit.

Feel free to close this PR and file a new one, when necessary.

@WaxyMocha
Copy link
Member

Screenshot_20210222_112739_org mozilla firefox
Looks like it's more of github issue, because you can open them.

@brlin-tw brlin-tw merged commit ff6862d into WoeUSB:main Feb 22, 2021
@brlin-tw
Copy link
Member

Looks like it's more of github issue, because you can open them.

Yep, seems that GitHub is messing with me. Thanks for the help.

@WaxyMocha
Copy link
Member

I wanted to ask for .svg first 😛, but ok. Can we agree that for this repository (since it's for both projects), before merging changes we need to have both of us approve a PR?

@brlin-tw
Copy link
Member

Can we agree that for this repository (since it's for both projects), before merging changes we need to have both of us approve a PR?

I'm too rushed for approving, apologies.

Let's continue the conversation in the original issue.

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.

3 participants