From 2c452719d06bf730cb97eb233ceb55a9d6846ac9 Mon Sep 17 00:00:00 2001
From: John Factotum <50942278+johnfactotum@users.noreply.github.com>
Date: Thu, 9 Mar 2023 22:20:22 +0800
Subject: [PATCH] Paginator: add option for maximum number of columns

Also: allow it to be positioned relative to the parent
---
 paginator.js | 32 ++++++++++++++++++++++----------
 reader.html  |  8 ++++++--
 2 files changed, 28 insertions(+), 12 deletions(-)

diff --git a/paginator.js b/paginator.js
index 44f2b41..556f277 100644
--- a/paginator.js
+++ b/paginator.js
@@ -337,6 +337,7 @@ class View {
 // NOTE: everything here assumes the so-called "negative scroll type" for RTL
 export class Paginator {
     #element = document.createElement('div')
+    #maxSizeContainer = document.createElement('div')
     #container = document.createElement('div')
     #header = document.createElement('div')
     #footer = document.createElement('div')
@@ -361,12 +362,21 @@ export class Paginator {
             boxSizing: 'border-box',
             width: '100%',
             height: '100%',
-            position: 'absolute',
+            position: 'relative',
+            overflow: 'hidden',
         })
-        this.#element.append(this.#container)
+        this.#element.append(this.#maxSizeContainer)
+        Object.assign(this.#maxSizeContainer.style, {
+            width: '100%',
+            height: '100%',
+            margin: 'auto',
+            position: 'relative',
+        })
+        this.#maxSizeContainer.append(this.#container)
         Object.assign(this.#container.style, {
             width: '100%',
             height: '100%',
+            margin: 'auto',
         })
 
         const marginalStyle = {
@@ -375,10 +385,10 @@ export class Paginator {
             fontFamily: 'system-ui',
             opacity: '.5',
         }
-        Object.assign(this.#header.style, marginalStyle, { top: '0' })
-        Object.assign(this.#footer.style, marginalStyle, { bottom: '0' })
-        this.#element.append(this.#header)
-        this.#element.append(this.#footer)
+        Object.assign(this.#header.style, marginalStyle)
+        Object.assign(this.#footer.style, marginalStyle)
+        this.#maxSizeContainer.append(this.#header)
+        this.#maxSizeContainer.append(this.#footer)
 
         new ResizeObserver(() => this.render()).observe(this.#element)
         this.#container.addEventListener('scroll', debounce(() => {
@@ -402,13 +412,14 @@ export class Paginator {
         // this is needed because the iframe does not fill the whole element
         this.#element.style.background = background
 
-        const { flow, margin, gap, maxColumnWidth } = this.layout
+        const { flow, margin, gap, maxColumnWidth, maxColumns } = this.layout
 
         if (flow === 'scrolled') {
             // FIXME: vertical-rl only, not -lr
             this.#element.setAttribute('dir', vertical ? 'rtl' : 'ltr')
             this.#element.style.padding = '0'
             this.#container.style.overflow ='scroll'
+            this.#maxSizeContainer.style.maxWidth = 'none'
             const columnWidth = this.layout.maxColumnWidth
 
             this.heads = null
@@ -419,6 +430,7 @@ export class Paginator {
             return { flow, margin, gap, columnWidth }
         }
 
+        this.#maxSizeContainer.style.maxWidth = `${maxColumns * maxColumnWidth}px`
         const { width, height } = this.#container.getBoundingClientRect()
         const size = vertical ? height : width
         const divisor = Math.ceil(size / maxColumnWidth)
@@ -434,12 +446,12 @@ export class Paginator {
             height: marginalHeight,
             gridTemplateColumns: `repeat(${divisor}, 1fr)`,
             gap: `${gap}px`,
-            padding: `0 ${gap}px`,
+            padding: `0 ${gap / 2}px`,
             fontSize: `min(.75em, ${marginalHeight})`,
             lineHeight: marginalHeight,
         }
-        Object.assign(this.#header.style, marginalStyle)
-        Object.assign(this.#footer.style, marginalStyle)
+        Object.assign(this.#header.style, marginalStyle, { top: `-${paddingV}` })
+        Object.assign(this.#footer.style, marginalStyle, { bottom: `-${paddingV}` })
         const heads = makeMarginals(divisor)
         const feet = makeMarginals(divisor)
         this.heads = heads.map(el => el.children[0])
diff --git a/reader.html b/reader.html
index ec375bb..74b3cf8 100644
--- a/reader.html
+++ b/reader.html
@@ -15,8 +15,12 @@
         }
     }
 }
+html {
+    height: 100%;
+}
 body {
-    margin: 0;
+    margin: 0 auto;
+    height: 100%;
     font: menu;
     font-family: system-ui, sans-serif;
 }
@@ -28,7 +32,7 @@ body {
     text-align: center;
 }
 #drop-target h1 {
-    font-weight: 900; 
+    font-weight: 900;
 }
 #file-button {
     font: inherit;