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

[OJS] tinymce error (downloadable font: OS/2: bad linegap: -32 ) with Firefox #2108

Closed
ajnyga opened this issue Dec 16, 2016 · 7 comments
Closed
Assignees
Milestone

Comments

@ajnyga
Copy link
Collaborator

ajnyga commented Dec 16, 2016

While testing OJS3 I noticed that my co-workers Firefox 50 keeps on getting this error in console:
downloadable font: OS/2: bad linegap: -32 (font-family: "tinymce" style:normal weight:normal stretch:normal src index:1) source: http://mydomain.fi/lib/pkp/lib/vendor/tinymce/tinymce/skins/lightgray/fonts/tinymce.woff skin.min.css:1:30806

I use Firefox 50 as well, but for some reason I do not get that.

There are a lot of similar errors reported when I googled around. This seems to have at least some sort of solution: TryGhost/Casper#154

@ajnyga ajnyga changed the title [OJS] tinymce error with Firefox [OJS] tinymce error (downloadable font: OS/2: bad linegap: -32 ) with Firefox Dec 16, 2016
@asmecher
Copy link
Member

@NateWr, any idea?

@ajnyga
Copy link
Collaborator Author

ajnyga commented Dec 16, 2016

Hi @asmecher,

I am getting this in Firefox 50 / OSX. As I mentioned earlier, in FF50/Win7 I could not see this.

Someone here roundcube/roundcubemail#4788 says that:

The font issue is probably unrelated and fixed in git with update to TinyMCE 4.1.8.

And it seems that OJS3 is using 4.1.7.

Edit: ok, here is the actual issue from tinymce git:
tinymce/tinymce#2136

@NateWr
Copy link
Contributor

NateWr commented Dec 19, 2016

Thanks for the detective work @ajnyga! @asmecher any concerns updating the bundle of TinyMCE? Should we hold that for a major update (3.1)?

@asmecher
Copy link
Member

Go for it on the master branch; we can back-port to ojs-stable-3_0_1 if needed and once we're relatively sure it's stable.

@NateWr NateWr added this to the OJS 3.0.2 milestone Dec 19, 2016
@NateWr NateWr self-assigned this Dec 19, 2016
@NateWr NateWr modified the milestones: OJS 3.1, OJS 3.0.2 Jan 31, 2017
NateWr added a commit to NateWr/pkp-lib that referenced this issue Jun 15, 2017
NateWr added a commit to NateWr/tinymce that referenced this issue Jun 15, 2017
@NateWr
Copy link
Contributor

NateWr commented Jun 15, 2017

I updated to the latest version (v4.6.4), which is a big jump. I did some clicking around and didn't notice any issues. I also took the liberty of changing the main body text to white to match other input fields.

PRs (tests running):
#2589
pkp/tinymce#17
pkp/ojs#1422

NateWr added a commit to NateWr/pkp-lib that referenced this issue Jun 16, 2017
NateWr added a commit to NateWr/pkp-lib that referenced this issue Jun 19, 2017
@NateWr
Copy link
Contributor

NateWr commented Jun 19, 2017

Tests passed. Not much code to review, but @bozana could you take a look at this to ensure it's working smoothly?

bozana added a commit that referenced this issue Jun 20, 2017
bozana added a commit to pkp/ojs that referenced this issue Jun 20, 2017
pkp/pkp-lib#2108 Update submodules with latest version of TinyMCE
bozana added a commit to pkp/tinymce that referenced this issue Jun 20, 2017
@bozana
Copy link
Collaborator

bozana commented Jun 20, 2017

@NateWr, great, works well! All merged, I think.

@NateWr NateWr closed this as completed Jun 20, 2017
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

4 participants