Skip to content
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

Assistant font switches 'if' for 'fi' on iOS (Chrome and Safari) #391

Closed
joeforce opened this issue Sep 30, 2016 · 14 comments
Closed

Assistant font switches 'if' for 'fi' on iOS (Chrome and Safari) #391

joeforce opened this issue Sep 30, 2016 · 14 comments

Comments

@joeforce
Copy link

I'm having an issue using Assistant. It works perfectly on Desktop, even responsively in the browser. But when I browse my page is Safari or Chrome (iOS 10) it switches if to fi. For example, "different" becomes "dfiferent" and "diffuse" becomes "dfifuse". I saw a similar issue with Merriweather sans:

#184

Is there any workaround? Changing the typeface this late in the game is going to be a lot of work.

Here's a sample of the spelling issue:

img_2928

Also, if you search the page for "dfi" it doesn't find it. The font is rendering wrong and the word is actually spelled correctly.

Thanks!

Joe

@davelab6
Copy link
Member

Thanks @joeforce :) I filed an issue on the upstream repo :)

@davelab6 davelab6 added this to the Bugs in Font Files Requiring Rebuild milestone Sep 30, 2016
@georgiecel
Copy link

@joeforce I had the same problem then found this issue 😄 If you look at the Merriweather issue again, @davelab6 posted some workaround CSS that worked for me.

@miscavage
Copy link

+1

@davelab6
Copy link
Member

This is just source sans pro, please use that until this is fixed

@aybmab
Copy link

aybmab commented Feb 22, 2017

Is there any eta on this?

@davelab6 what do you mean by just use sans pro? (apologies - I'm a font noob)

@davelab6
Copy link
Member

The latin part of the family is (mean to be lol) an exact copy of Source Sans Pro. https://fonts.google.com/specimen/Source+Sans+Pro

@banbanbana
Copy link

Having issues on iOS but not desktop. Posted it on Twitter here:
https://twitter.com/awdoes/status/841306893800878082

@davelab6
Copy link
Member

@laerm0 can be hotfixed like playfair was, swapping the components

@laerm0
Copy link
Contributor

laerm0 commented Aug 25, 2017

Okay, I'll take care of it today.

@laerm0
Copy link
Contributor

laerm0 commented Aug 25, 2017

The binaries we're serving are correct, both glyph and OT.

screen region 2017-08-25 at 19 29 29

It also renders fine.

screen region 2017-08-25 at 19 32 47

Am I missing something or is this no longer an issue?

@m4rc1e
Copy link
Collaborator

m4rc1e commented Aug 30, 2017

@laerm0 I'll take a look at this

@m4rc1e
Copy link
Collaborator

m4rc1e commented Aug 30, 2017

Ok, I've just tested the family on IOS using an Iphone SE, here's a jsbin

I can't replicate this issue:

img_0896

@joeforce could you look at the Jsbin I just made on a device running IOS please?

The font was updated on the 2017-08-28, this may have solved the issue.

@m4rc1e
Copy link
Collaborator

m4rc1e commented Aug 30, 2017

The family contains Hebrew so this may an issue with some browsers and how they handle RTL scripts (just an assumption, not fact).

@davelab6
Copy link
Member

@joeforce I'll close this as none of us can reproduce, but please reopen if you can still see this happening

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

No branches or pull requests

9 participants