Skip to content
This repository has been archived by the owner on Sep 27, 2019. It is now read-only.

incorrect rendering of output in firefox #52

Closed
phi-gamma opened this issue May 9, 2013 · 12 comments · Fixed by #53
Closed

incorrect rendering of output in firefox #52

phi-gamma opened this issue May 9, 2013 · 12 comments · Fixed by #53

Comments

@phi-gamma
Copy link
Member

Reported by Luigi.

I’m testing with xpdf, mupdf and okular (poppler) which are all okay. Anyone know what ff uses for rendering?

@khaledhosny
Copy link

pdf.js

@phi-gamma
Copy link
Member Author

this is caused by the compatibility hacks for fontspec et al

@khaledhosny thanks! we should compile luatex and $EDITOR with emscripten to get a complete, browser based tex environment.

@phi-gamma
Copy link
Member Author

Something’s wrong with caching; happens with luatex-plain but not with context; http://www.ntg.nl/pipermail/ntg-context/2013/073157.html

@phi-gamma phi-gamma reopened this May 10, 2013
@eroux
Copy link
Member

eroux commented May 12, 2013

http://mirror.math.ku.edu/tex-archive/macros/latex/contrib/fontspec/fontspec.pdf << apparently, there are bugs with XeTeX too... maybe it should be reported to mozilla?

@khaledhosny
Copy link

If anything is at error here, it is pdf.js not anyone else. No point for making workaround for it, those aren’t only the PDFs it renders badly.

@phi-gamma
Copy link
Member Author

This doesn’t appear to happen anymore with the latest code. @eroux can you test e.g. the luaotfload manual if it still results in garbage with firefox?

@eroux
Copy link
Member

eroux commented Jun 28, 2013

Manual with latest code is really garbage (maybe more than before) with Firefox 22... Sorry :)

@phi-gamma
Copy link
Member Author

Does it get better if you apply this patch to luaotfload-merged.lua:

diff --git a/luaotfload-merged.lua b/luaotfload-merged.lua
index 81e3fec..9d708bf 100644
--- a/luaotfload-merged.lua
+++ b/luaotfload-merged.lua
@@ -3143,7 +3143,7 @@ function caches.savedata(path,name,data)
   local luaname,lucname=makefullname(path,name)
   if luaname then
     texio.write(string.format("(save: %s)",luaname))
-    table.tofile(luaname,data,true,{ reduce=true })
+    table.tofile(luaname,data,true,{ reduce=false })
     if lucname and type(caches.compile)=="function" then
       os.remove(lucname) 
       texio.write(string.format("(save: %s)",lucname))

?

@eroux
Copy link
Member

eroux commented Jun 28, 2013

Well, it's pretty much the same...

@phi-gamma
Copy link
Member Author

Ah, sorry, forgot to mention that you need to delete the font cache.

@eroux
Copy link
Member

eroux commented Jun 28, 2013

Oh, sorry, it works even without your patch if I remove the cache.

@phi-gamma
Copy link
Member Author

Great, closing this one.

zauguin pushed a commit to zauguin/luaotfload that referenced this issue Apr 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants