From 3a97f54ea025ee21dc5e7beead8c3b08e72d7fbf Mon Sep 17 00:00:00 2001
From: thePR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com>
Date: Wed, 3 Jun 2026 08:34:44 -0400
Subject: [PATCH] book(C.5.6): per-tile master picker on Pages overview
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Each tile in the Pages overview now has a hover-revealed master picker
that overrides effectiveMasterIdForPage for that single entry.

- Empty selection ("Default (…)") falls through to spec.defaultMasterId.
- "No master" explicitly opts out via masterId === null.
- Otherwise persists the chosen master id on the manifest entry.
- Unified the reflow tile + pagination engine + physical-page enumerator
  to call effectiveMasterIdForPage instead of the local
  `p.masterId || defaultMasterId` short-circuit so the explicit-null
  semantics work everywhere.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 src/index.html | 37 ++++++++++++++++++++++++++++++++++---
 1 file changed, 34 insertions(+), 3 deletions(-)

diff --git a/src/index.html b/src/index.html
index 9dc05eb..3843032 100644
--- a/src/index.html
+++ b/src/index.html
@@ -1698,6 +1698,11 @@
       background: var(--warn, #fee2e2);
       border-color: var(--warn, #fca5a5);
     }
+    .bookPageTileMasterSel {
+      height: 22px; padding: 0 4px; font-size: .72rem;
+      border: 1px solid var(--line); background: #fff;
+      max-width: 120px;
+    }
     .bookPageTile.dragging { opacity: .5; }
     .bookPageTile.dragover {
       outline: 2px solid var(--accent);
@@ -8982,7 +8987,7 @@
           // as positioned text blocks on top. Tokens in master text
           // layers resolve to the first physical page of this
           // chapter segment.
-          const masterId = p.masterId || book.spec?.defaultMasterId || null;
+          const masterId = effectiveMasterIdForPage(p);
           const masterTarget = masterId ? `book-master:${masterId}` : null;
           const masterLayersRaw = masterTarget
             ? (state.layers || []).filter(l => l.target === masterTarget && !l.hidden
@@ -9031,6 +9036,30 @@
         // chrome stays out of the way until you point at them".
         const actions = document.createElement('div');
         actions.className = 'bookPageTileActions';
+        // Master picker — overrides effectiveMasterIdForPage(p) for
+        // this single entry. Empty value means "use default".
+        const masterSel = document.createElement('select');
+        masterSel.className = 'bookPageTileMasterSel';
+        masterSel.title = 'Master for this page';
+        const defaultMaster = book.masters.find(m => m.id === book.spec?.defaultMasterId);
+        const defaultLabel = defaultMaster ? `Default (${defaultMaster.label || 'Untitled'})` : 'Default (none)';
+        const opts = [`<option value="">${escapeHtml(defaultLabel)}</option>`];
+        opts.push(`<option value="__none__"${p.masterId === null ? ' selected' : ''}>No master</option>`);
+        for (const m of book.masters) {
+          opts.push(`<option value="${escapeHtml(m.id)}"${p.masterId === m.id ? ' selected' : ''}>${escapeHtml(m.label || 'Untitled')}</option>`);
+        }
+        masterSel.innerHTML = opts.join('');
+        masterSel.addEventListener('click', (e) => e.stopPropagation());
+        masterSel.addEventListener('change', (e) => {
+          e.stopPropagation();
+          pushHistory('book: page master override');
+          const v = masterSel.value;
+          if (v === '') delete p.masterId;
+          else if (v === '__none__') p.masterId = null;
+          else p.masterId = v;
+          render();
+        });
+        actions.appendChild(masterSel);
         const dup = document.createElement('button');
         dup.type = 'button';
         dup.className = 'bookPageTileAction';
@@ -9180,7 +9209,9 @@
       }
       const md = String(entry.chapterMarkdown || '');
       const styles = (book?.spec?.textStyles) || defaultBookTextStyles();
-      const masterId = entry.masterId || book?.spec?.defaultMasterId || null;
+      const masterId = (typeof effectiveMasterIdForPage === 'function')
+        ? effectiveMasterIdForPage(entry)
+        : (entry.masterId || book?.spec?.defaultMasterId || null);
       result.masterId = masterId;
 
       // Locate the text-frame layer on the master.
@@ -9483,7 +9514,7 @@
               manifestIdx,
               indexInEntry: i,
               type: 'reflow',
-              masterId: p.masterId || book.spec?.defaultMasterId || null,
+              masterId: effectiveMasterIdForPage(p),
               pageNumber: numberedHere ? printedNum : null,
               chapterTitle: currentChapter.title,
               chapterNumber: currentChapter.number,