From 912b05450202422564e9699a8d8cbb3740f9595b Mon Sep 17 00:00:00 2001 From: Evan Reichard Date: Tue, 24 Oct 2023 18:30:01 -0400 Subject: [PATCH] [fix] reader page --- README.md | 13 ++++++------- assets/reader/index.js | 17 ++++++++--------- 2 files changed, 14 insertions(+), 16 deletions(-) diff --git a/README.md b/README.md index 77ea5d0..ab84c21 100644 --- a/README.md +++ b/README.md @@ -18,19 +18,18 @@

+

Screenshots

- - --- WEB --- - - - --- PWA --- + Web App - PWA +

+

+ +

--- -[![Build Status](https://drone.va.reichard.io/api/badges/evan/BookManager/status.svg)](https://drone.va.reichard.io/evan/BookManager) - This is BookManager! Will probably be renamed at some point. This repository contains: - Web App / Progressive Web App (PWA) diff --git a/assets/reader/index.js b/assets/reader/index.js index b425e72..f7fb530 100644 --- a/assets/reader/index.js +++ b/assets/reader/index.js @@ -65,23 +65,22 @@ class EBookReader { this.rendition.hooks.content.register(getStats); /** - * Display @ CFI x 2 (Hack) - * I'm pretty sure this is because we set font size in the rendition - * render hook that resides in initRenditionListeners, and that the - * logic in ePub.js runs before or in parallel with the hook when - * setting the CFI position. + * Display @ CFI x 3 (Hack) * - * By running twice we ensure that the hook ran and set the font size - * already, which should now ensure the proper page on the second time - * around. + * This is absurd. Only way to get it to consistently show the correct + * page is to execute this three times. I tried the font hook, + * rendition hook, relocated hook, etc. No reliable way outside of + * running this three times. * - * Bug: https://github.com/futurepress/epub.js/issues/1194 + * Likely Bug: https://github.com/futurepress/epub.js/issues/1194 **/ await this.rendition.display(cfi); await this.rendition.display(cfi); + await this.rendition.display(cfi); // Highlight Element - DOM Has Element let { element } = await this.getCFIFromXPath(this.bookState.progress); + this.bookState.progressElement = element; this.highlightPositionMarker(); }