Render icons with better font hinting #1272

CoolOppo · Apr 18, 2015

It's extremely subtle, but it'll make the icons look slightly better.

If anybody's interested, see Wikipedia's explanation of font hinting for more info.

chrisaljoudi · Apr 18, 2015

Hi @CoolOppo!

Thanks for that. The only thing is, your render of icon38@2x.png appears to be 78x78 instead of 76x76.

Other than that, looks great. Thanks again.

CoolOppo · Apr 18, 2015

My bad; I'll fix that right now.

CoolOppo · Apr 18, 2015

Okay, it's fixed! Sorry for that. Good thing you were paying attention because I obviously was not.

CoolOppo · Apr 18, 2015

Wait a second: the old version of icon64@2x.png was definitely better. All the other ones I changed are totally alright, but that one actually is blurrier after my change. I'll fix that.

~~Edit: hold off on merging still. I'm not quite happy with it.~~

CoolOppo · Apr 18, 2015

Alright, I am finally happy with these icons! Go ahead and merge if you like.

CoolOppo Re-render icons with font hinting

b65518a

CoolOppo Fix size of icon38@2x.png

448116f

chrisaljoudi merged commit ec22cb7 into chrisaljoudi:master Apr 18, 2015
1 check passed

1 check passed

Details continuous-integration/travis-ci/pr The Travis CI build passed

CoolOppo	`Fix icon64@2x's hinting`	`1cd2491`
CoolOppo	`Fix icon38's hinting`	`74d62ce`
CoolOppo	`Forgot about the Chromium-specific ones!`	`005ec08`
CoolOppo	`Made "off" icons lighter`	`2e2b7f5`
CoolOppo	`Made "off" icons even lighter`	`32c1eef`

chrisaljoudi/uBlock

Render icons with better font hinting #1272

Labels

Milestone

Assignee

2 participants

1 check passed