inuitcss / base.page

[DEPRECATED] [go to intuitcss/inuitcss] Base page-level styling.
https://github.com/inuitcss/inuitcss
13 stars 6 forks source link

Remove default font-smoothing. #4

Closed amiuhle closed 8 years ago

amiuhle commented 9 years ago

Fixes #3.

I really think this shouldn't be in html as a default. As @kizu pointed out in #3, this article explains it quite good: http://usabilitypost.com/2012/11/05/stop-fixing-font-smoothing/

I'm having a serious issue right now, where #0081c7 text on white background is rendered in a much lighter color when font-smoothing is set to antialiased.

nenadjelovac commented 9 years ago

I think it would be better to make this opt-in, as @BPScott suggested in his comment on other issue (but probably without @extend).

Let's see how @csswizardry feels about that.

amiuhle commented 9 years ago

I don't even think it should be an opt-in on html, just the %pixel-antialiasing class @BPScott suggested. Nothing stops you from manually adding it to html if you really want to, but actually I don't see any point.

But of course, nothing stops me from opting in or out, so that would be fine, too :) So yeah, let's see what Harry has to say about this.