janpio / ionic-epubjs

Ionic 3 app using Epub.js
https://janpio.github.io/ionic-epubjs/
31 stars 20 forks source link

Font Change Error #42

Open ibby89 opened 6 years ago

ibby89 commented 6 years ago

Attempting to change font in-book returning fuzzy error:

_updatePageTitle title = Open: The Philosophy and Practices that are Revolutionizing Education and Science
book.ts:138:4
_updateTotalPages
book.ts:116:4
not found, try fuzzy match
epub.min.js:8:37802
not found, try fuzzy match
epub.min.js:8:37802
not found, try fuzzy match
epub.min.js:8:37802
not found, try fuzzy match
epub.min.js:8:37802
_updateTotalPages totalPages =  816

Font does not change.

janpio commented 6 years ago

Is this one of the books included here in this repo? What are the exact steps to reproduce? What platform and device did you encounter this problem?

ibby89 commented 6 years ago

I ran it on labs within the browser. This was using the 'Open (unpacked)' main menu item within the repo.

I ran with the browser console open (I'm using Firefox) because in my app that I am building using the repo, the font/background wasn't changing when using the on-screen tools. When I read 'Open' and I try to change the background, this works correctly, as does increasing and decreasing the font size. However, when I try to change the font-family to say 'Athelas', I get the 'not found, try fuzzy match' error in the console and the font does not update even though it says it updates the whole book.

I've tested this in labs using Chrome with the same result. Not sure what's going on.

janpio commented 6 years ago

I can confirm this is broken when running via e.g. ionic serve.

I think this used to work before though :/

janpio commented 6 years ago

Tested on an Android device: Works just fine. Interesting.

ibby89 commented 6 years ago

Think it's a labs issue? Thing is, I don't have a mac to test on an iPhone sim (I only have an iPhone :().

janpio commented 6 years ago

Also works on iPhone 8.

I don't really know what is going on here to be honest.