-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Firefox compatibility #37
Comments
See #36 😄 Edit: and I think I'm having the same experience regarding the other elements as well (as to which ones work and don't work). |
Thank you @JDGrimes for fixing that 🎉 I can't reproduce your webfonts issue though, Helen, can you clarify? Here's what I'm seeing: Note, it's possibly working for me because I have both Merriweather and SF UI fonts installed on my machine. Inline controls, insertion dialog, those are dummy elements so far, mostly to get a feel for the UI. I opened a separate ticket to discuss how far to take this prototype: #51 |
Here's what I see: These errors appear in my console:
|
That is super interesting, and turns out to be a bit of a 🐰🕳 In the past there seems to have been a bug specifically with the Google Fonts version of Merriweather, see https://bugzilla.mozilla.org/show_bug.cgi?id=1239176 But that bug should be fixed. However this ticket suggests that Nightly and Aurora versions of Firefox may be having this issue still today: https://bugzilla.mozilla.org/show_bug.cgi?id=1239176 — any chance you're running one of those versions? |
I use the developer edition which is Aurora - currently at 53.0a2 (2017-02-09). |
That could explain the issue. Seems like it's upstream with either Google Fonts or with Firefox. That said, it's probably worth discussing whether to use Merriweather in the first place. I think it's lovely, but Georgia might do fine as well. |
Closing for now, let's reopen if it isn't fixed in Google Fonts or Aurora. |
…ncy-installer-script-3 Restores some changes that shouldn't have been made.
As always, I get the great honor of reporting that some of this doesn't appear to work in Firefox, which is my primary browser. I use the developer edition.
Things that don't work as compared to Chrome:
Things that do:
Things that don't work in either (assuming expected):
The text was updated successfully, but these errors were encountered: