Skip to content

Comments

Starling icon#63

Open
prayerie wants to merge 2 commits intopebble-dev:masterfrom
prayerie:sicon
Open

Starling icon#63
prayerie wants to merge 2 commits intopebble-dev:masterfrom
prayerie:sicon

Conversation

@prayerie
Copy link

Starling is a UK bank.

Previews:

25px

25px

50px

50px

80px

80px

@lavglaab
Copy link
Collaborator

These look good! Can you check the pixel alignment of all your points here? I'm seeing a lot of blurriness and fuzzy anti-aliasing in the pdc-tool previews that would indicate a line isn't properly aligned. (Even-width strokes should be aligned on the boundary between pixels, and odd-width strokes should be centered within a pixel)

@prayerie
Copy link
Author

These look good! Can you check the pixel alignment of all your points here? I'm seeing a lot of blurriness and fuzzy anti-aliasing in the pdc-tool previews that would indicate a line isn't properly aligned. (Even-width strokes should be aligned on the boundary between pixels, and odd-width strokes should be centered within a pixel)

oh thats strange, i was basing it on the web version of pdc tool (screenshots in original post) and adjusting it to reduce the fuzziness there. i assume the inkscape plugin would be better for this so ill do that in a bit!

@lavglaab
Copy link
Collaborator

Had a look at cleaning up the shapes. What do you think of these?
Starling_25px
Starling_50px
Starling_80px

@prayerie
Copy link
Author

Had a look at cleaning up the shapes. What do you think of these?

Starling_25px

Starling_50px

Starling_80px

ah looks good :) sorry i was kinda fumbling around idk why mine are so messy lol

@lavglaab
Copy link
Collaborator

Hm, on second thought, the 25px form reads better if it's simplified.
Starling_25px 2

@prayerie
Copy link
Author

Hm, on second thought, the 25px form reads better if it's simplified.

Starling_25px 2

yeah i was worried with my 25px one it might accidentally get mistaken for a music disc? i think either way i was trying too hard to stay within the proportions of their original app icon lol

@lavglaab
Copy link
Collaborator

No worries! If you'll rebase your PR to one commit, of these files, I'll go ahead and merge it.

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.

2 participants