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.

Sign up for GitHub

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

Jump to bottom

Icons for ARAnyM #5442

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

Icons for ARAnyM #5442

wants to merge 2 commits into from

Conversation

maxtron95
Copy link
Contributor

@palob palob added the merge label Jul 2, 2020
palob
palob requested changes Jul 2, 2020
View reviewed changes
Copy link
Member

@palob palob left a comment

Choose a reason for hiding this comment

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

There's a few areas where alignment and rounding isn't optimal which contributes to the overall wobbly look.

@maxtron95
Copy link
Contributor Author

maxtron95 commented Jul 2, 2020 •
edited
Loading

@palob Should I try to make the icon an Atari Computer like this instead, since this an Atari virtual machine software?

@palob
Copy link
Member

palob commented Jul 2, 2020

This might work, maybe incorporating the colours of the original icon or something.

@maxtron95
Copy link
Contributor Author

@palob What do you think of this new design?

aranym-ci

@palob
Copy link
Member

palob commented Jul 25, 2020

Looks ace but maybe a bit big and detailed.

Foggalong
Foggalong suggested changes Sep 11, 2020
View reviewed changes
Copy link
Contributor

@Foggalong Foggalong left a comment

Choose a reason for hiding this comment

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

Agreed new design is better but too big; possibly will need to be 1-2px higher after being resized also

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@palob palob palob requested changes

@Foggalong Foggalong Foggalong requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
merge
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maxtron95 @palob @Foggalong