Bookhoard/foliate-js
1
0
Fork 0
mirror of https://github.com/john-okeefe/foliate-js.git synced 2026-09-09 11:29:14 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix keyboard 0 not resetting zoom in fixed-layout books

Browse Source 
When pressing 0 to reset zoom, the code was calling
removeAttribute('zoom') on the renderer, which removed the DOM
attribute but left the internal zoom/transform state stale. This meant
subsequent zoom operations would compound on the old hidden state.

Replace with the new resetZoom() method that properly clears the
numeric zoom and triggers a clean re-render.
This commit is contained in:
john-okeefe
2026-04-18 22:34:10 -04:00
parent 7ec14df593
commit 285b6711da
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
reader.js
+2 -2
View File
@@ -209,7 +209,7 @@ class Reader {
renderer instanceof FixedLayout, renderer instanceof FixedLayout,
); );
if (renderer instanceof FixedLayout) { if (renderer instanceof FixedLayout) {
renderer.removeAttribute("zoom"); renderer.resetZoom();
renderer.dragOffset = { x: 0, y: 0 }; renderer.dragOffset = { x: 0, y: 0 };
$("#zoom-reset").textContent = "100%"; $("#zoom-reset").textContent = "100%";
console.log("[Reader Debug] ✅ Zoom reset"); console.log("[Reader Debug] ✅ Zoom reset");
@@ -346,7 +346,7 @@ class Reader {
console.log("[Reader Debug] 🔄 Reset zoom via keyboard"); console.log("[Reader Debug] 🔄 Reset zoom via keyboard");
const renderer = this.view.renderer; const renderer = this.view.renderer;
if (renderer instanceof FixedLayout) { if (renderer instanceof FixedLayout) {
renderer.removeAttribute("zoom"); renderer.resetZoom();
renderer.dragOffset = { x: 0, y: 0 }; renderer.dragOffset = { x: 0, y: 0 };
$("#zoom-reset").textContent = "100%"; $("#zoom-reset").textContent = "100%";
console.log("[Reader Debug] ✅ Keyboard zoom reset"); console.log("[Reader Debug] ✅ Keyboard zoom reset");