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.

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

change styling of ties to something that works on OS X/Safari #1

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@madwort

madwort commented Mar 27, 2014

Hey Corey,
It's a lot less elegant, but it looks much better on my machine! How does it look for you?
Tom

@coreymwamba

This comment has been minimized.

Show comment
Hide comment
@coreymwamba

coreymwamba Mar 29, 2014

Owner

Hello Tom!

It renders too high or low on all the browsers I have to test [old Opera, Chromium 33, Firefox 28]. So I* think the rendering is Safari-specific. Have you tried setting percentages for vertical-align instead?

ASIDE: I hate Webkit. I really do. All of them. I also worked out that CSS transforms don't work on inline elements in one browsing engine: Webkit. All of them. I'd say you should file a bug report to Safari but if the general sibling indicator fix is anything to go by, we'd both have died before it got fixed.

Owner

coreymwamba commented Mar 29, 2014

Hello Tom!

It renders too high or low on all the browsers I have to test [old Opera, Chromium 33, Firefox 28]. So I* think the rendering is Safari-specific. Have you tried setting percentages for vertical-align instead?

ASIDE: I hate Webkit. I really do. All of them. I also worked out that CSS transforms don't work on inline elements in one browsing engine: Webkit. All of them. I'd say you should file a bug report to Safari but if the general sibling indicator fix is anything to go by, we'd both have died before it got fixed.

@madwort

This comment has been minimized.

Show comment
Hide comment
@madwort

madwort Mar 29, 2014

Hmmm, ok, how about this http://static.madwort.co.uk/musify/ - same movement but using vertical-align. I think it may have the same problem for you - I'm starting to think it's how OS X is rendering the character…

On 29 Mar 02014, at 04:08, coreymwamba notifications@github.com wrote:

Hello Tom!

It renders too high or low on all the browsers I have to test [old Opera, Chromium 33, Firefox 28]. So I* think the rendering is Safari-specific. Have you tried setting percentages for vertical-align instead?

ASIDE: I hate Webkit. I really do. All of them. I also worked out that CSS transforms don't work on inline elements in one browsing engine: Webkit. All of them. I'd say you should file a bug report to Safari but if the general sibling indicator fix is anything to go by, we'd both have died before it got fixed.


Reply to this email directly or view it on GitHub.

madwort commented Mar 29, 2014

Hmmm, ok, how about this http://static.madwort.co.uk/musify/ - same movement but using vertical-align. I think it may have the same problem for you - I'm starting to think it's how OS X is rendering the character…

On 29 Mar 02014, at 04:08, coreymwamba notifications@github.com wrote:

Hello Tom!

It renders too high or low on all the browsers I have to test [old Opera, Chromium 33, Firefox 28]. So I* think the rendering is Safari-specific. Have you tried setting percentages for vertical-align instead?

ASIDE: I hate Webkit. I really do. All of them. I also worked out that CSS transforms don't work on inline elements in one browsing engine: Webkit. All of them. I'd say you should file a bug report to Safari but if the general sibling indicator fix is anything to go by, we'd both have died before it got fixed.


Reply to this email directly or view it on GitHub.

@coreymwamba

This comment has been minimized.

Show comment
Hide comment
@coreymwamba

coreymwamba Mar 31, 2014

Owner

Yes, it's the same - but if you can change the spacing by altering the value of vertical-align, then I'm thinking one could write some JS to change the value based on the user agent...

Owner

coreymwamba commented Mar 31, 2014

Yes, it's the same - but if you can change the spacing by altering the value of vertical-align, then I'm thinking one could write some JS to change the value based on the user agent...

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