ionic-epubjs icon indicating copy to clipboard operation
ionic-epubjs copied to clipboard

Font Change Error

Open ibby89 opened this issue 7 years ago • 6 comments

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.

ibby89 avatar Feb 26 '18 12:02 ibby89

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?

janpio avatar Feb 26 '18 12:02 janpio

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.

ibby89 avatar Feb 26 '18 13:02 ibby89

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

I think this used to work before though :/

janpio avatar Mar 01 '18 15:03 janpio

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

janpio avatar Mar 01 '18 16:03 janpio

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 :().

ibby89 avatar Mar 01 '18 16:03 ibby89

Also works on iPhone 8.

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

janpio avatar Mar 01 '18 21:03 janpio