8000 Safari 7 · Issue #27 · necolas/griddle · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Safari 7 #27

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 occ 8000 asionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
chriscoyier opened this issue Feb 10, 2014 · 5 comments
Open

Safari 7 #27

chriscoyier opened this issue Feb 10, 2014 · 5 comments

Comments

@chriscoyier
Copy link

I think there was some kind of change in Safari 7 that changed rounding (again). Ultimately it breaks griddle.

screen shot 2014-02-09 at 5 52 24 pm

Changing the letter-spacing from -0.31em to -0.32em fixes it, but then it's a bit ragged-right. Might wanna try the font-size: 0; thing or some other approach.
@7studio
Copy link
Collaborator
7studio commented Feb 11, 2014

Hi Chris,
Thank you to point out this issue. It seems this hack has many inconveniences (especially when there are a new broswer release) and is not a perennial solution in any way (it's my opinion since the beginning).
IMHO, that is why Nicolas chose to use another solution in his suitcss/grid project.
But each 'hack' to avoid this HTML problem have theirs drawbacks (e.g.: #24) while there are numerous other solutions without CSS (e.g.: Remove the spaces)
For now, I use this CSS property only with a compressed code (otherwise I don't).

@necolas
Copy link
Owner
necolas commented Feb 11, 2014

Yeah I don't use Sass anymore. If someone wants to put together a PR to implement the font-size hack, like in suitcss/grid, then I'll accept it.

@ferdinandsalis
Copy link

@necolas I am intrigued, why did you decide to move away from sass and just use plain css? Did you write about it?

@necolas
Copy link
Owner
necolas commented Feb 11, 2014

Haven't written about it. Might do at some point.

@joeyhoer
Copy link

Another possible solution may be to use the ch unit: http://codepen.io/joeyhoer/details/zxPaXZ/. Better browser support, but you still have to redefine the font-family, which may be undesirable.

@joeyhoer joeyhoer mentioned this issue Feb 10, 2015
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@chriscoyier @necolas @ferdinandsalis @joeyhoer @7studio
0