From 084914aec6da2f0a7fb0988ec74b4796382103c8 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Sat, 9 Mar 2024 17:49:43 -0800
Subject: [PATCH 01/10] initial stab & functionality of spine marks

---
 src/book.js                 | 55 ++++++++++++++++++++++++++++++-------
 src/html/page_layout.html   |  9 ++++++
 src/models/configuration.js |  1 +
 src/signatures.js           | 15 +++++++---
 src/utils/drawing.js        | 49 ++++++++++++++++++++++++++++++---
 src/utils/formUtils.js      |  1 +
 src/utils/renderUtils.js    |  4 +++
 7 files changed, 116 insertions(+), 18 deletions(-)

diff --git a/src/book.js b/src/book.js
index af666c1..7f9c0a9 100644
--- a/src/book.js
+++ b/src/book.js
@@ -10,7 +10,7 @@ import { PAGE_LAYOUTS, PAGE_SIZES } from './constants.js';
 import { updatePageLayoutInfo } from './utils/renderUtils.js';
 import JSZip from 'jszip';
 import { loadConfiguration } from './utils/formUtils.js';
-import { drawFoldlines, drawCropmarks, drawSpineMarks } from './utils/drawing.js';
+import { drawFoldlines, drawCropmarks, drawSpineMark, drawSigOrderMark } from './utils/drawing.js';
 import { calculateDimensions, calculateLayout } from './utils/layout.js';
 import { interleavePages, embedPagesInNewPdf } from './utils/pdf.js';
 
@@ -21,6 +21,7 @@ import { interleavePages, embedPagesInNewPdf } from './utils/pdf.js';
  * @property {string|number} info - page # or 'b'
  * @property {boolean} isSigStart
  * @property {boolean} isSigEnd
+ * @property {number} signatureNum - which signature is this page in. 0 based
  */
 
 /**
@@ -81,6 +82,7 @@ export class Book {
     this.flyleafs = configuration.flyleafs;
     this.cropmarks = configuration.cropMarks;
     this.pdfEdgeMarks = configuration.pdfEdgeMarks;
+    this.sigOrderMarks = configuration.sigOrderMarks;
     this.cutmarks = configuration.cutMarks;
     this.format = configuration.sigFormat;
     if (configuration.sigFormat === 'standardsig') {
@@ -308,7 +310,7 @@ export class Book {
       this.format == 'customsig'
     ) {
       // Only generate the first signature for preview
-      const pagesArr = isPreview ? this.rearrangedpages.slice(0, 1) : this.rearrangedpages;
+      const pagesArr = this.rearrangedpages;
       const signatures = [{}];
       const makeSignatures = async () => {
         const tasks = pagesArr.map(async (pages, i) => {
@@ -316,6 +318,7 @@ export class Book {
           signatures[i] = { name: `${this.filename}_signature${i}` };
           [signatures[i].front, signatures[i].back] = await this.createSignatures({
             pageIndexDetails: pages,
+            maxSigCount: pagesArr.length
           });
         });
         await Promise.all(tasks);
@@ -332,7 +335,8 @@ export class Book {
           await Promise.all(tasks);
         };
         await duplexSignatures();
-        previewPdf = signatures[0].duplex;
+        console.log("Shark")
+        // previewPdf = signatures[0].duplex;
       }
 
       if (this.print_file != 'aggregated' && !isPreview) {
@@ -353,7 +357,7 @@ export class Book {
         await saveSignatures();
       }
 
-      if (this.print_file != 'signatures' && !isPreview) {
+      if (this.print_file != 'signatures') {
         const saveAggregate = async () => {
           const aggregate = {
             front: !this.duplex ? await PDFDocument.create() : null,
@@ -399,8 +403,11 @@ export class Book {
               this.zip.file(`${this.filename}_typeset.pdf`, pdfBytes);
             });
           }
+          console.log("Do I have this?  ",aggregate.duplex.getPageCount())
+          previewPdf = aggregate.duplex
+          return aggregate
         };
-        await saveAggregate();
+        var results = await saveAggregate();
       }
 
       var rotationMetaInfo =
@@ -450,14 +457,17 @@ export class Book {
    *   (conditionally) populates the destPdf and (conditionally) generates the outname PDF
    *
    * @param {Object} config - object /w the following parameters:
-   * @param {PageInfo[]} config.pageList : objects that contain 3 values: { isSigStart: boolean, isSigEnd: boolean, info: either the page number or 'b'}
+   * @param {PageInfo[]} config.pageList : see documentation at top of file
    * @param {boolean} config.back : is 'back' of page  (boolean)
    * @param {boolean} config.alt : alternate pages (boolean)
+   * @param {number} config.maxSigCount 
    * @return reference to the new PDF created
    */
   async writepages(config) {
+    console.log("Write pages ", config)
     const pagelist = config.pageList;
     const back = config.back;
+    const maxSigCount = config.maxSigCount;
     const filteredList = [];
     const blankIndices = [];
     pagelist.forEach((pageInfo, i) => {
@@ -467,6 +477,7 @@ export class Book {
         blankIndices.push(i);
       }
     });
+
     const [outPDF, embeddedPages] = await embedPagesInNewPdf(this.managedDoc, filteredList);
 
     blankIndices.forEach((i) => embeddedPages.splice(i, 0, 'b'));
@@ -482,7 +493,7 @@ export class Book {
     let side2flag = back;
 
     while (block_end <= pagelist.length) {
-      const sigDetails = config.pageList.slice(block_start, block_end);
+      const sigDetails = pagelist.slice(block_start, block_end);
       side2flag = this.draw_block_onto_page({
         outPDF: outPDF,
         embeddedPages: embeddedPages,
@@ -492,10 +503,12 @@ export class Book {
         papersize: this.papersize,
         positions: positions,
         cropmarks: this.cropmarks,
+        sigOrderMarks: this.sigOrderMarks,
         pdfEdgeMarks: this.pdfEdgeMarks,
         cutmarks: this.cutmarks,
         alt: config.alt,
         side2flag: side2flag,
+        maxSigCount: maxSigCount,
       });
       block_start += offset;
       block_end += offset;
@@ -507,7 +520,8 @@ export class Book {
    *
    * @param {Object} config - object /w the following parameters:
    * @param {string|null} config.outname : name of pdf added to ongoing zip file. Ex: 'signature1duplex.pdf' (or null if no signature file needed)
-   * @param {PageInfo[]} config.sigDetails : objects that contain 3 values: { isSigStart: boolean, isSigEnd: boolean, info: either the page number or 'b'}
+   * @param {PageInfo[]} config.sigDetails : see documentation at top of file
+   * @param {number} config.maxSigCount: Total number of signatures
    * @param {boolean} config.side2flag : is 'back' of page  (boolean)
    * @param {[number, number]} config.papersize : paper size (as [number, number])
    * @param {number} config.block_start: Starting page index
@@ -529,10 +543,13 @@ export class Book {
     const outPDF = config.outPDF;
     const positions = config.positions;
     const foldmarks = config.cropmarks;
+    const sigOrderMarks = config.sigOrderMarks;
     const pdfEdgeMarks = config.pdfEdgeMarks;
     const cutmarks = config.cutmarks;
     const alt = config.alt;
+    const maxSigCount = config.maxSigCount
     let side2flag = config.side2flag;
+    console.log("burp I see ",maxSigCount)
 
     const block = config.embeddedPages.slice(block_start, block_end);
     const currPage = outPDF.addPage(papersize);
@@ -541,6 +558,7 @@ export class Book {
       ? drawFoldlines(side2flag, this.duplexrotate, papersize, this.per_sheet)
       : [];
     const drawLines = [...cropLines, ...foldLines];
+    const drawRects = [];
 
     block.forEach((page, i) => {
       if (page == 'b' || page === undefined) {
@@ -558,14 +576,27 @@ export class Book {
         console.error('Unexpected type for page: ', page);
       }
 
-      if (pdfEdgeMarks && (sigDetails[i].isSigStart || sigDetails[i].isSigEnd)) {
-        drawLines.push(drawSpineMarks(sigDetails[i], positions[i]));
+      if (sigDetails[i].isSigStart) {
+        if (pdfEdgeMarks) {
+          drawLines.push(drawSpineMark(true, positions[i], 5));
+        }
+        if (sigOrderMarks) {
+          drawRects.push(drawSigOrderMark(sigDetails[i], positions[i], maxSigCount, 5, 20));
+        }
+      }
+      else if (sigDetails[i].isSigEnd) {
+        if (pdfEdgeMarks) {
+          drawLines.push(drawSpineMark(false, positions[i], 5));
+        }
       }
     });
 
     drawLines.forEach((line) => {
       currPage.drawLine(line);
     });
+    drawRects.forEach((rect) => {
+      currPage.drawRectangle(rect);
+    })
 
     if (alt) {
       side2flag = !side2flag;
@@ -577,20 +608,24 @@ export class Book {
    * PDF builder base function for Classic (non-Wacky) layouts. Called by [createoutputfiles]
    *
    * @param {Object} config
+   * @param {number} config.maxSigCount
    * @param {PageInfo[][]} config.pageIndexDetails : a nested list of objects.
    */
   async createSignatures(config) {
+    console.log("createSignatures ",config)
     const pages = config.pageIndexDetails;
     const tasks = [
       this.writepages({
         pageList: pages[0],
         back: false,
         alt: false,
+        maxSigCount: config.maxSigCount,
       }),
       this.writepages({
         pageList: pages[1],
         back: true,
         alt: false,
+        maxSigCount: config.maxSigCount,
       }),
     ];
     const [pdfFront, pdfBack] = await Promise.all(tasks);
diff --git a/src/html/page_layout.html b/src/html/page_layout.html
index 4b55021..8b92f77 100644
--- a/src/html/page_layout.html
+++ b/src/html/page_layout.html
@@ -34,6 +34,15 @@ <h2>Page Layout</h2>
           data-balloon-pos="up"
         >
           <input type="checkbox" name="pdf_edge_marks" /></span
+      ></label><br>
+      <label
+        >Add signature order marks (spine)
+        <span
+          data-balloon-length="medium"
+          aria-label="Puts staggered lines down the different signature spines to indicate order"
+          data-balloon-pos="up"
+        >
+          <input type="checkbox" name="sig_order_marks" /></span
       ></label>
     </div>
   </div>
diff --git a/src/models/configuration.js b/src/models/configuration.js
index 27f370c..181c5b0 100644
--- a/src/models/configuration.js
+++ b/src/models/configuration.js
@@ -82,6 +82,7 @@ export const schema = z.object({
   cropMarks: urlSafe(coercedBoolean).default(false),
   cutMarks: urlSafe(coercedBoolean).default(false),
   pdfEdgeMarks: urlSafe(coercedBoolean).default(false),
+  sigOrderMarks: urlSafe(coercedBoolean).default(false),
   pageScaling,
   pagePositioning,
   mainForeEdgePaddingPt: urlSafe(z.coerce.number()).default(0),
diff --git a/src/signatures.js b/src/signatures.js
index 4a35528..0db40ed 100644
--- a/src/signatures.js
+++ b/src/signatures.js
@@ -77,12 +77,13 @@ export class Signatures {
     const newsigs = [];
 
     //      Use the booklet class for each signature
-    this.signaturepagelists.forEach((pagerange) => {
+    this.signaturepagelists.forEach((pagerange, i) => {
       const pagelistdetails = this.booklet(
         pagerange,
         this.duplex,
         this.per_sheet,
-        this.duplexrotate
+        this.duplexrotate,
+        i
       );
       newsigs.push(pagelistdetails);
     });
@@ -122,8 +123,9 @@ export class Signatures {
    * @param {boolean} duplex - Whether both front and back sides go in the same file or not.
    * @param {number} per_sheet - number of pages per sheet (front and back combined)
    * @param {boolean} duplexrotate - whether to rotate alternating sheets or not.
+   * @param {number} sig_num - signature number (0 indexed)
    */
-  booklet(pages, duplex, per_sheet, duplexrotate) {
+  booklet(pages, duplex, per_sheet, duplexrotate, sig_num) {
     const pagelistdetails = duplex ? [[]] : [[], []];
     const { front, rotate, back } = BOOKLET_LAYOUTS[per_sheet];
 
@@ -139,19 +141,21 @@ export class Signatures {
     let front_end = center;
     let back_start = center;
     let back_end = center + pageblock;
-
+    console.log("outside")
     while (front_start >= 0 && back_end <= pages.length) {
       const front_block = pages.slice(front_start, front_end);
       const back_block = pages.slice(back_start, back_end);
 
       const block = [...front_block, ...back_block];
 
+      console.log(" ~~~~ SIG NUM "+sig_num+" on "+front_start+" --> "+back_end)
       front_config.forEach((pnum) => {
         const page = block[pnum - 1]; //page layouts are 1-indexed, not 0-index
         pagelistdetails[0].push({
           info: page,
           isSigStart: front_start == 0 && pnum == 1,
           isSigEnd: front_start == 0 && pnum == block.length,
+          signatureNum: sig_num,
         });
       });
 
@@ -163,11 +167,14 @@ export class Signatures {
           info: page,
           isSigStart: front_start == 0 && pnum == 1,
           isSigEnd: front_start == 0 && pnum == block.length,
+          signatureNum: sig_num,
         });
       });
 
       // Update all our counters
+      console.log("stepa "+front_start)
       front_start -= pageblock;
+      console.log("stepb "+front_start)
       front_end -= pageblock;
       back_start += pageblock;
       back_end += pageblock;
diff --git a/src/utils/drawing.js b/src/utils/drawing.js
index 9b366c8..6e26411 100644
--- a/src/utils/drawing.js
+++ b/src/utils/drawing.js
@@ -113,14 +113,14 @@ export function drawCropmarks(papersize, per_sheet) {
 }
 
 /**
- * @param {import("../book.js").PageInfo} sigDetails - page info object
+ * @param {boolean} draw_top_mark - true to draw mark at top of PDF, false for bottom of PDF
  * @param {import("../book.js").Position} position - position info object
+ * @param {number} w - width of the line in pts
  * @returns {Line}
  */
-export function drawSpineMarks(sigDetails, position) {
-  const w = 5;
+export function drawSpineMark(draw_top_mark, position, w) {
   let startX, startY, endX, endY;
-  if (sigDetails.isSigStart) {
+  if (draw_top_mark) {
     [startX, startY] = position.spineMarkTop;
     [endX, endY] = position.spineMarkTop;
   } else {
@@ -148,6 +148,47 @@ export function drawSpineMarks(sigDetails, position) {
   return drawLineArgs;
 }
 
+
+/**
+ * @param {import("../book.js").PageInfo} sigDetails - page info object
+ * @param {import("../book.js").Position} position - position info object
+ * @param {number} maxSigCount - number of total signatures
+ * @param {number} w - width of the mark in pts
+ * @param {number} suggested_h - suggested height of the mark in pts (can be scaled down to fit all marks between PDF top/bottom)
+ * @returns {Line}
+ */
+export function drawSigOrderMark(sigDetails, position, maxSigCount, w, suggested_h) {
+  const top = drawSpineMark(true, position, w);
+  const bottom = drawSpineMark(false, position, w);
+
+  let x = top.start.x
+  let y = top.start.y
+
+  const dist = (position.rotation == 0) ? top.start.y - bottom.start.y : top.start.x - bottom.start.x
+  let h = Math.min(suggested_h, dist/maxSigCount)
+  const offset = h * sigDetails.signatureNum;
+  console.log("Looking at signature ",sigDetails.signatureNum," of ",maxSigCount," PDF top/bottom distance ",dist," results in ",h," (",suggested_h," vs ",(dist/maxSigCount),") order mark height w/ offset ",offset," (width ",w,")")
+
+  if (position.rotation == 0) {
+    h = h * -1;
+    y -= offset;
+  } else {
+    const temp = h;
+    h = w;
+    w = temp * -1;
+    x -= offset
+  }
+  
+  return {
+    x: x,
+    y: y,
+    width: w,
+    height: h,
+    borderWidth: 0,
+    color: rgb(0, 0, 0),
+    opacity: 0.5,
+  };
+}
 /**
  * @param {number} x
  * @param {number} ystart
diff --git a/src/utils/formUtils.js b/src/utils/formUtils.js
index a477b5b..68eccef 100644
--- a/src/utils/formUtils.js
+++ b/src/utils/formUtils.js
@@ -22,6 +22,7 @@ const fromFormToConfiguration = (form) =>
     paperRotation90: form.has('paper_rotation_90'),
     pageLayout: form.get('pagelayout'),
     cropMarks: form.has('cropmarks'),
+    sigOrderMarks: form.has('sig_order_marks'),
     pdfEdgeMarks: form.has('pdf_edge_marks'),
     cutMarks: form.has('cutmarks'),
     pageScaling: form.get('page_scaling'),
diff --git a/src/utils/renderUtils.js b/src/utils/renderUtils.js
index b627be1..ab48097 100644
--- a/src/utils/renderUtils.js
+++ b/src/utils/renderUtils.js
@@ -183,6 +183,10 @@ export function renderFormFromSettings(configuration) {
     document.querySelector("input[name='cropmarks']").checked = true;
   }
 
+  if (configuration.sigOrderMarks) {
+    document.querySelector("input[name='sig_order_marks']").checked = true;
+  }
+
   if (configuration.pdfEdgeMarks) {
     document.querySelector("input[name='pdf_edge_marks']").checked = true;
   }

From e1641cd8696778a1416f5ed3b5fe08acab515503 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Thu, 11 Apr 2024 21:13:42 -0500
Subject: [PATCH 02/10] cleaning up things...

---
 src/book.js          | 27 +++++++++++++--------------
 src/book.test.js     |  1 +
 src/signatures.js    |  5 +----
 src/utils/drawing.js |  2 ++
 4 files changed, 17 insertions(+), 18 deletions(-)

diff --git a/src/book.js b/src/book.js
index 7f9c0a9..56d28cf 100644
--- a/src/book.js
+++ b/src/book.js
@@ -290,6 +290,8 @@ export class Book {
    *      generate preview content AND a downloadable zip
    */
   async createoutputfiles(isPreview) {
+    // set this to `true` to enable full-book previews (placeholder till it's in the UI)
+    const fullPreviewDevHack = true
     const previewFrame = document.getElementById('pdf');
     let previewPdf = null;
 
@@ -310,13 +312,13 @@ export class Book {
       this.format == 'customsig'
     ) {
       // Only generate the first signature for preview
-      const pagesArr = this.rearrangedpages;
+      const pagesArr = (isPreview && !fullPreviewDevHack) ? this.rearrangedpages.slice(0, 1) : this.rearrangedpages;
       const signatures = [{}];
       const makeSignatures = async () => {
         const tasks = pagesArr.map(async (pages, i) => {
           console.log(pages);
           signatures[i] = { name: `${this.filename}_signature${i}` };
-          [signatures[i].front, signatures[i].back] = await this.createSignatures({
+          [signatures[i].front, signatures[i].back] = await this.createSignature({
             pageIndexDetails: pages,
             maxSigCount: pagesArr.length
           });
@@ -335,11 +337,10 @@ export class Book {
           await Promise.all(tasks);
         };
         await duplexSignatures();
-        console.log("Shark")
-        // previewPdf = signatures[0].duplex;
+        previewPdf = signatures[0].duplex;
       }
 
-      if (this.print_file != 'aggregated' && !isPreview) {
+      if (this.print_file != 'aggregated' && (!isPreview || fullPreviewDevHack)) {
         const saveSignatures = async () => {
           const tasks = signatures.map(async (sig) => {
             await sig.front?.save().then((pdfBytes) => {
@@ -357,7 +358,7 @@ export class Book {
         await saveSignatures();
       }
 
-      if (this.print_file != 'signatures') {
+      if (this.print_file != 'signatures' && (!isPreview || fullPreviewDevHack)) {
         const saveAggregate = async () => {
           const aggregate = {
             front: !this.duplex ? await PDFDocument.create() : null,
@@ -403,11 +404,10 @@ export class Book {
               this.zip.file(`${this.filename}_typeset.pdf`, pdfBytes);
             });
           }
-          console.log("Do I have this?  ",aggregate.duplex.getPageCount())
           previewPdf = aggregate.duplex
-          return aggregate
         };
-        var results = await saveAggregate();
+
+        await saveAggregate();
       }
 
       var rotationMetaInfo =
@@ -453,7 +453,7 @@ export class Book {
   }
 
   /**
-   * Part of the Classic (non-Wacky) flow. Called by [createsignatures].
+   * Part of the Classic (non-Wacky) flow. Called by [createsignature].
    *   (conditionally) populates the destPdf and (conditionally) generates the outname PDF
    *
    * @param {Object} config - object /w the following parameters:
@@ -464,7 +464,6 @@ export class Book {
    * @return reference to the new PDF created
    */
   async writepages(config) {
-    console.log("Write pages ", config)
     const pagelist = config.pageList;
     const back = config.back;
     const maxSigCount = config.maxSigCount;
@@ -549,7 +548,6 @@ export class Book {
     const alt = config.alt;
     const maxSigCount = config.maxSigCount
     let side2flag = config.side2flag;
-    console.log("burp I see ",maxSigCount)
 
     const block = config.embeddedPages.slice(block_start, block_end);
     const currPage = outPDF.addPage(papersize);
@@ -606,13 +604,14 @@ export class Book {
 
   /**
    * PDF builder base function for Classic (non-Wacky) layouts. Called by [createoutputfiles]
+   * Generates a single signature  (or is it just a single sheet? -- comments left long after coding)
+   * TODO : re-examine this logic and clean up this comment. What is going on??
    *
    * @param {Object} config
    * @param {number} config.maxSigCount
    * @param {PageInfo[][]} config.pageIndexDetails : a nested list of objects.
    */
-  async createSignatures(config) {
-    console.log("createSignatures ",config)
+  async createSignature(config) {
     const pages = config.pageIndexDetails;
     const tasks = [
       this.writepages({
diff --git a/src/book.test.js b/src/book.test.js
index a008292..d74e0c9 100644
--- a/src/book.test.js
+++ b/src/book.test.js
@@ -51,6 +51,7 @@ describe('Book model', () => {
     source_rotation: 'none',
     print_file: 'both',
     signatureconfig: [],
+    sigOrderMarks: false,
   };
 
   it('returns a new Book', () => {
diff --git a/src/signatures.js b/src/signatures.js
index 0db40ed..0f8a861 100644
--- a/src/signatures.js
+++ b/src/signatures.js
@@ -141,14 +141,13 @@ export class Signatures {
     let front_end = center;
     let back_start = center;
     let back_end = center + pageblock;
-    console.log("outside")
+
     while (front_start >= 0 && back_end <= pages.length) {
       const front_block = pages.slice(front_start, front_end);
       const back_block = pages.slice(back_start, back_end);
 
       const block = [...front_block, ...back_block];
 
-      console.log(" ~~~~ SIG NUM "+sig_num+" on "+front_start+" --> "+back_end)
       front_config.forEach((pnum) => {
         const page = block[pnum - 1]; //page layouts are 1-indexed, not 0-index
         pagelistdetails[0].push({
@@ -172,9 +171,7 @@ export class Signatures {
       });
 
       // Update all our counters
-      console.log("stepa "+front_start)
       front_start -= pageblock;
-      console.log("stepb "+front_start)
       front_end -= pageblock;
       back_start += pageblock;
       back_end += pageblock;
diff --git a/src/utils/drawing.js b/src/utils/drawing.js
index 6e26411..c44fad7 100644
--- a/src/utils/drawing.js
+++ b/src/utils/drawing.js
@@ -150,6 +150,8 @@ export function drawSpineMark(draw_top_mark, position, w) {
 
 
 /**
+ * TODO : these params should probably be pushed into a config... maybe next time/next pass
+ *
  * @param {import("../book.js").PageInfo} sigDetails - page info object
  * @param {import("../book.js").Position} position - position info object
  * @param {number} maxSigCount - number of total signatures

From 4d33c3619007c8b24cb3cd50f6e66dca8278df17 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Thu, 11 Apr 2024 21:20:49 -0500
Subject: [PATCH 03/10] fucking ;s....

---
 src/book.js          |  8 ++++----
 src/utils/drawing.js | 12 ++++++------
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/book.js b/src/book.js
index 56d28cf..549b6db 100644
--- a/src/book.js
+++ b/src/book.js
@@ -291,7 +291,7 @@ export class Book {
    */
   async createoutputfiles(isPreview) {
     // set this to `true` to enable full-book previews (placeholder till it's in the UI)
-    const fullPreviewDevHack = true
+    const fullPreviewDevHack = true;
     const previewFrame = document.getElementById('pdf');
     let previewPdf = null;
 
@@ -404,7 +404,7 @@ export class Book {
               this.zip.file(`${this.filename}_typeset.pdf`, pdfBytes);
             });
           }
-          previewPdf = aggregate.duplex
+          previewPdf = aggregate.duplex;
         };
 
         await saveAggregate();
@@ -546,7 +546,7 @@ export class Book {
     const pdfEdgeMarks = config.pdfEdgeMarks;
     const cutmarks = config.cutmarks;
     const alt = config.alt;
-    const maxSigCount = config.maxSigCount
+    const maxSigCount = config.maxSigCount;
     let side2flag = config.side2flag;
 
     const block = config.embeddedPages.slice(block_start, block_end);
@@ -594,7 +594,7 @@ export class Book {
     });
     drawRects.forEach((rect) => {
       currPage.drawRectangle(rect);
-    })
+    });
 
     if (alt) {
       side2flag = !side2flag;
diff --git a/src/utils/drawing.js b/src/utils/drawing.js
index c44fad7..7032833 100644
--- a/src/utils/drawing.js
+++ b/src/utils/drawing.js
@@ -163,13 +163,13 @@ export function drawSigOrderMark(sigDetails, position, maxSigCount, w, suggested
   const top = drawSpineMark(true, position, w);
   const bottom = drawSpineMark(false, position, w);
 
-  let x = top.start.x
-  let y = top.start.y
+  let x = top.start.x;
+  let y = top.start.y;
 
-  const dist = (position.rotation == 0) ? top.start.y - bottom.start.y : top.start.x - bottom.start.x
-  let h = Math.min(suggested_h, dist/maxSigCount)
+  const dist = (position.rotation == 0) ? top.start.y - bottom.start.y : top.start.x - bottom.start.x;
+  let h = Math.min(suggested_h, dist/maxSigCount);
   const offset = h * sigDetails.signatureNum;
-  console.log("Looking at signature ",sigDetails.signatureNum," of ",maxSigCount," PDF top/bottom distance ",dist," results in ",h," (",suggested_h," vs ",(dist/maxSigCount),") order mark height w/ offset ",offset," (width ",w,")")
+  console.log("Looking at signature ",sigDetails.signatureNum," of ",maxSigCount," PDF top/bottom distance ",dist," results in ",h," (",suggested_h," vs ",(dist/maxSigCount),") order mark height w/ offset ",offset," (width ",w,")");
 
   if (position.rotation == 0) {
     h = h * -1;
@@ -178,7 +178,7 @@ export function drawSigOrderMark(sigDetails, position, maxSigCount, w, suggested
     const temp = h;
     h = w;
     w = temp * -1;
-    x -= offset
+    x -= offset;
   }
   
   return {

From 92f10ed846b1c9cb29c30143f2f497687b90d43b Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Thu, 11 Apr 2024 21:22:12 -0500
Subject: [PATCH 04/10] dropping unneeded console.log

---
 src/utils/drawing.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/utils/drawing.js b/src/utils/drawing.js
index 7032833..58193e0 100644
--- a/src/utils/drawing.js
+++ b/src/utils/drawing.js
@@ -169,7 +169,7 @@ export function drawSigOrderMark(sigDetails, position, maxSigCount, w, suggested
   const dist = (position.rotation == 0) ? top.start.y - bottom.start.y : top.start.x - bottom.start.x;
   let h = Math.min(suggested_h, dist/maxSigCount);
   const offset = h * sigDetails.signatureNum;
-  console.log("Looking at signature ",sigDetails.signatureNum," of ",maxSigCount," PDF top/bottom distance ",dist," results in ",h," (",suggested_h," vs ",(dist/maxSigCount),") order mark height w/ offset ",offset," (width ",w,")");
+  // console.log("Looking at signature ",sigDetails.signatureNum," of ",maxSigCount," PDF top/bottom distance ",dist," results in ",h," (",suggested_h," vs ",(dist/maxSigCount),") order mark height w/ offset ",offset," (width ",w,")");
 
   if (position.rotation == 0) {
     h = h * -1;

From 087172b5f2fabe46f21e30614a40662f7d38b1f2 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Thu, 11 Apr 2024 22:01:08 -0500
Subject: [PATCH 05/10] prettier fixes

---
 src/book.js               | 10 +++++-----
 src/html/page_layout.html |  4 ++--
 src/utils/drawing.js      |  7 +++----
 3 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/src/book.js b/src/book.js
index 549b6db..7d989e2 100644
--- a/src/book.js
+++ b/src/book.js
@@ -312,7 +312,8 @@ export class Book {
       this.format == 'customsig'
     ) {
       // Only generate the first signature for preview
-      const pagesArr = (isPreview && !fullPreviewDevHack) ? this.rearrangedpages.slice(0, 1) : this.rearrangedpages;
+      const pagesArr =
+        isPreview && !fullPreviewDevHack ? this.rearrangedpages.slice(0, 1) : this.rearrangedpages;
       const signatures = [{}];
       const makeSignatures = async () => {
         const tasks = pagesArr.map(async (pages, i) => {
@@ -320,7 +321,7 @@ export class Book {
           signatures[i] = { name: `${this.filename}_signature${i}` };
           [signatures[i].front, signatures[i].back] = await this.createSignature({
             pageIndexDetails: pages,
-            maxSigCount: pagesArr.length
+            maxSigCount: pagesArr.length,
           });
         });
         await Promise.all(tasks);
@@ -460,7 +461,7 @@ export class Book {
    * @param {PageInfo[]} config.pageList : see documentation at top of file
    * @param {boolean} config.back : is 'back' of page  (boolean)
    * @param {boolean} config.alt : alternate pages (boolean)
-   * @param {number} config.maxSigCount 
+   * @param {number} config.maxSigCount
    * @return reference to the new PDF created
    */
   async writepages(config) {
@@ -581,8 +582,7 @@ export class Book {
         if (sigOrderMarks) {
           drawRects.push(drawSigOrderMark(sigDetails[i], positions[i], maxSigCount, 5, 20));
         }
-      }
-      else if (sigDetails[i].isSigEnd) {
+      } else if (sigDetails[i].isSigEnd) {
         if (pdfEdgeMarks) {
           drawLines.push(drawSpineMark(false, positions[i], 5));
         }
diff --git a/src/html/page_layout.html b/src/html/page_layout.html
index 8b92f77..fae3151 100644
--- a/src/html/page_layout.html
+++ b/src/html/page_layout.html
@@ -33,8 +33,8 @@ <h2>Page Layout</h2>
           aria-label="Puts small marks on the outermost folio of the signatures indicating the top and bottom of the PDF"
           data-balloon-pos="up"
         >
-          <input type="checkbox" name="pdf_edge_marks" /></span
-      ></label><br>
+          <input type="checkbox" name="pdf_edge_marks" /></span></label
+      ><br />
       <label
         >Add signature order marks (spine)
         <span
diff --git a/src/utils/drawing.js b/src/utils/drawing.js
index 58193e0..e9b76bf 100644
--- a/src/utils/drawing.js
+++ b/src/utils/drawing.js
@@ -148,7 +148,6 @@ export function drawSpineMark(draw_top_mark, position, w) {
   return drawLineArgs;
 }
 
-
 /**
  * TODO : these params should probably be pushed into a config... maybe next time/next pass
  *
@@ -166,8 +165,8 @@ export function drawSigOrderMark(sigDetails, position, maxSigCount, w, suggested
   let x = top.start.x;
   let y = top.start.y;
 
-  const dist = (position.rotation == 0) ? top.start.y - bottom.start.y : top.start.x - bottom.start.x;
-  let h = Math.min(suggested_h, dist/maxSigCount);
+  const dist = position.rotation == 0 ? top.start.y - bottom.start.y : top.start.x - bottom.start.x;
+  let h = Math.min(suggested_h, dist / maxSigCount);
   const offset = h * sigDetails.signatureNum;
   // console.log("Looking at signature ",sigDetails.signatureNum," of ",maxSigCount," PDF top/bottom distance ",dist," results in ",h," (",suggested_h," vs ",(dist/maxSigCount),") order mark height w/ offset ",offset," (width ",w,")");
 
@@ -180,7 +179,7 @@ export function drawSigOrderMark(sigDetails, position, maxSigCount, w, suggested
     w = temp * -1;
     x -= offset;
   }
-  
+
   return {
     x: x,
     y: y,

From ec38e2127c67450f87f5042f58e8b19bfd33eb06 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Sun, 28 Apr 2024 10:57:13 -0700
Subject: [PATCH 06/10] fix typos

---
 src/book.js          | 2 +-
 src/utils/drawing.js | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/book.js b/src/book.js
index b5f065c..41d5090 100644
--- a/src/book.js
+++ b/src/book.js
@@ -10,7 +10,7 @@ import { PAGE_LAYOUTS, PAGE_SIZES } from './constants.js';
 import { updatePageLayoutInfo } from './utils/renderUtils.js';
 import JSZip from 'jszip';
 import { loadConfiguration } from './utils/formUtils.js';
-import { drawFoldlines, drawCropmarks, drawSpineMarks, drawSigOrderMark, drawSewingMarks } from './utils/drawing.js';
+import { drawFoldlines, drawCropmarks, drawSpineMark, drawSigOrderMark, drawSewingMarks } from './utils/drawing.js';
 import { calculateDimensions, calculateLayout } from './utils/layout.js';
 import { interleavePages, embedPagesInNewPdf } from './utils/pdf.js';
 
diff --git a/src/utils/drawing.js b/src/utils/drawing.js
index 12fcf34..5fa98b8 100644
--- a/src/utils/drawing.js
+++ b/src/utils/drawing.js
@@ -121,7 +121,7 @@ export function drawCropmarks(papersize, per_sheet) {
   return lines;
 }
 
-/*
+/**
  * @param {@param {import("../book.js").PageInfo}} sigDetails - information about signature where marks will be printed
  * @param {import("../book.js").Position} position - position info object
  * @param {number} amount - amount of sewing crosses.

From 7489faa8cca15cd0189ecc2e6f3dde93239c3d21 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Tue, 30 Apr 2024 07:28:23 -0700
Subject: [PATCH 07/10] ran that prettier

---
 src/book.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/book.js b/src/book.js
index 41d5090..9192abc 100644
--- a/src/book.js
+++ b/src/book.js
@@ -10,7 +10,13 @@ import { PAGE_LAYOUTS, PAGE_SIZES } from './constants.js';
 import { updatePageLayoutInfo } from './utils/renderUtils.js';
 import JSZip from 'jszip';
 import { loadConfiguration } from './utils/formUtils.js';
-import { drawFoldlines, drawCropmarks, drawSpineMark, drawSigOrderMark, drawSewingMarks } from './utils/drawing.js';
+import {
+  drawFoldlines,
+  drawCropmarks,
+  drawSpineMark,
+  drawSigOrderMark,
+  drawSewingMarks,
+} from './utils/drawing.js';
 import { calculateDimensions, calculateLayout } from './utils/layout.js';
 import { interleavePages, embedPagesInNewPdf } from './utils/pdf.js';
 

From 4d17814ad1f48f321e30849c6f037c08dfb827fd Mon Sep 17 00:00:00 2001
From: Rebecca S <sithel@gmail.com>
Date: Sat, 4 May 2024 16:22:46 -0700
Subject: [PATCH 08/10] Pulling out PDF Markup into it's own chunk & allowing
 Sewing Marks on only some folios & some bug fixes (#115)

* now with a PDF Markup section + sewing marks on only some of the folios
---
 index.html                  |  1 +
 src/book.js                 | 10 ++--
 src/book.test.js            |  1 +
 src/html/crop_box.html      | 99 +++++++++++++++++++++++++++++++++++++
 src/html/page_layout.html   | 79 +----------------------------
 src/main.js                 |  6 +++
 src/models/configuration.js |  5 ++
 src/signatures.js           | 14 ++++++
 src/utils/clickHandlers.js  |  9 ++++
 src/utils/drawing.js        | 20 +++++++-
 src/utils/formUtils.js      |  3 +-
 src/utils/layout.js         |  2 +-
 src/utils/renderUtils.js    |  4 ++
 13 files changed, 165 insertions(+), 88 deletions(-)
 create mode 100644 src/html/crop_box.html

diff --git a/index.html b/index.html
index 250b012..f570ac6 100644
--- a/index.html
+++ b/index.html
@@ -33,6 +33,7 @@ <h1>Bookbinder JS</h1>
         <load src="src/html/source_manip.html" />
         <load src="src/html/printer.html" />
         <load src="src/html/page_layout.html" />
+        <load src="src/html/crop_box.html" />
         <load src="src/html/sig_format.html" />
         <load src="src/html/flyleaf.html" />
         <load src="src/html/preview.html" />
diff --git a/src/book.js b/src/book.js
index 9192abc..289c804 100644
--- a/src/book.js
+++ b/src/book.js
@@ -27,6 +27,7 @@ import { interleavePages, embedPagesInNewPdf } from './utils/pdf.js';
  * @property {string|number} info - page # or 'b'
  * @property {boolean} isSigStart
  * @property {boolean} isSigEnd
+ * @property {boolean} isSigMiddle
  * @property {number} signatureNum - which signature is this page in. 0 based
  */
 
@@ -97,6 +98,7 @@ export class Book {
     this.flyleafs = configuration.flyleafs;
     this.cropmarks = configuration.cropMarks;
     this.sewingMarks = {
+      sewingMarkLocation: configuration.sewingMarkLocation,
       isEnabled: configuration.sewingMarksEnabled,
       amount: configuration.sewingMarksAmount,
       marginPt: configuration.sewingMarksMarginPt,
@@ -237,13 +239,6 @@ export class Book {
       this.cropbox = newPage.getCropBox();
     }
 
-    console.log(
-      'The updatedDoc doc has : ',
-      this.managedDoc.getPages(),
-      ' vs --- ',
-      this.managedDoc.getPageCount()
-    );
-
     switch (this.format) {
       case 'perfect':
       case 'booklet':
@@ -616,6 +611,7 @@ export class Book {
         ? drawSewingMarks(
             sigDetails[i],
             positions[i],
+            sewingMarks.sewingMarkLocation,
             sewingMarks.amount,
             sewingMarks.marginPt,
             sewingMarks.tapeWidthPt
diff --git a/src/book.test.js b/src/book.test.js
index 7bd5476..b5a53a8 100644
--- a/src/book.test.js
+++ b/src/book.test.js
@@ -45,6 +45,7 @@ describe('Book model', () => {
       top: 0,
     },
     sewingMarks: {
+      sewingMarkLocation: 'all',
       amount: 3,
       isEnabled: false,
       marginPt: 72,
diff --git a/src/html/crop_box.html b/src/html/crop_box.html
new file mode 100644
index 0000000..d99ae60
--- /dev/null
+++ b/src/html/crop_box.html
@@ -0,0 +1,99 @@
+<div class="section">
+  <h2>PDF Markup</h2>
+  <i>(does not currently work with Wacky Small Layouts -- soon!)</i><br />
+  <span class="row">
+    <label
+      >Add Foldlines
+      <span
+        data-balloon-length="medium"
+        aria-label="Adds folding guidelines (ordered thickest to thinnest)"
+        data-balloon-pos="up"
+      >
+        <input type="checkbox" name="cropmarks" />
+      </span>
+    </label>
+  </span>
+
+  <span class="row">
+    <label
+      >Add Cutlines
+      <span data-balloon-length="medium" aria-label="Adds cutting guidelines" data-balloon-pos="up">
+        <input type="checkbox" name="cutmarks"
+      /></span>
+    </label>
+  </span>
+  <span class="row">
+    <label
+      >Add PDF bounds indicators (spine)
+      <span
+        data-balloon-length="medium"
+        aria-label="Puts small marks on the outermost folio of the signatures indicating the top and bottom of the PDF"
+        data-balloon-pos="up"
+      >
+        <input type="checkbox" name="pdf_edge_marks"
+      /></span>
+    </label>
+  </span>
+  <span class="row">
+    <label
+      >Add signature order marks (spine)
+      <span
+        data-balloon-length="medium"
+        aria-label="Puts staggered lines down the different signature spines to indicate order"
+        data-balloon-pos="up"
+      >
+        <input type="checkbox" name="sig_order_marks"
+      /></span>
+    </label>
+  </span>
+  <span class="row">
+    <label>Add marks for sewing</label>
+    <input type="checkbox" id="add_sewing_marks_checkbox" name="add_sewing_marks_checkbox" />
+    <details id="sewing_marks_details">
+      <summary>Detailed settings for sewing</summary>
+      <span class="row">
+        <label
+          >Draw marks on :
+          <select name="sewing_mark_locations" id="sewing_mark_locations">
+            <option value="all" selected="selected">center of every folio</option>
+            <option value="only_out">only on spine</option>
+            <option value="only_in">only in center of signature</option>
+            <option value="in_n_out">both spine and center of signature</option>
+          </select> </label
+        ><br />
+        Look at the image below.<br />
+        <label
+          >(A) Margin (distance where first point should be draw):
+          <input
+            type="number"
+            id="sewing_marks_margin_pt"
+            name="sewing_marks_margin_pt"
+            class="layout_margin_user_input_field" /><sub>pt</sub><br
+        /></label>
+
+        <label
+          >(B) Amount of sewing points:
+          <input
+            type="number"
+            id="sewing_marks_amount"
+            name="sewing_marks_amount"
+            class="layout_margin_user_input_field" /><br
+        /></label>
+
+        <label
+          >(C) Tape width:
+          <input
+            type="number"
+            id="sewing_marks_tape_width_pt"
+            name="sewing_marks_tape_width_pt"
+            class="layout_margin_user_input_field" /><sub>pt</sub><br
+        /></label>
+        <img
+          alt="sewing image - use tape width of 0pt to make a single dot (overlapping marks)"
+          src="/img/sewing_settings_explanation.png"
+          style="height: 100%; width: 100%; object-fit: contain"
+        />
+      </span>
+    </details>
+  </span>
+</div>
diff --git a/src/html/page_layout.html b/src/html/page_layout.html
index 7b6e176..dd510eb 100644
--- a/src/html/page_layout.html
+++ b/src/html/page_layout.html
@@ -9,43 +9,6 @@ <h2>Page Layout</h2>
       Sextodecimo - sixteen pages per side of sheet
     </option>
   </select>
-  <div>
-    <label
-      >Add Foldlines
-      <span
-        data-balloon-length="medium"
-        aria-label="Adds folding guidelines (ordered thickest to thinnest)"
-        data-balloon-pos="up"
-      >
-        <input type="checkbox" name="cropmarks" /></span
-    ></label>
-    <span style="display: inline-block; width: 10px"></span>
-    <label
-      >Add Cutlines
-      <span data-balloon-length="medium" aria-label="Adds cutting guidelines" data-balloon-pos="up"
-        ><input type="checkbox" name="cutmarks" /></span
-    ></label>
-    <div>
-      <label
-        >Add PDF bounds indicators (spine)
-        <span
-          data-balloon-length="medium"
-          aria-label="Puts small marks on the outermost folio of the signatures indicating the top and bottom of the PDF"
-          data-balloon-pos="up"
-        >
-          <input type="checkbox" name="pdf_edge_marks" /></span></label
-      ><br />
-      <label
-        >Add signature order marks (spine)
-        <span
-          data-balloon-length="medium"
-          aria-label="Puts staggered lines down the different signature spines to indicate order"
-          data-balloon-pos="up"
-        >
-          <input type="checkbox" name="sig_order_marks" /></span
-      ></label>
-    </div>
-  </div>
   <span class="row">
     <details>
       <summary>Folding instructions for quarto/octavo</summary>
@@ -86,49 +49,9 @@ <h2>Page Layout</h2>
     <label for="page_positioning">Page Positioning</label>
     <select name="page_positioning" id="page_positioning">
       <option value="centered" selected>Centered</option>
-      <option value="binding_alinged">Snug against binding edge</option>
+      <option value="binding_aligned">Snug against binding edge</option>
     </select>
   </span>
-
-  <span class="row">
-    <label>Add marks for sewing</label>
-    <input type="checkbox" id="add_sewing_marks_checkbox" name="add_sewing_marks_checkbox" />
-    <details>
-      <summary>Detailed settings for sewing</summary>
-      Look at the image below.
-      <span class="row">
-        <label>(A) Margin (distance where first point should be draw):</label>
-        <input
-          type="number"
-          id="sewing_marks_margin_pt"
-          name="sewing_marks_margin_pt"
-          class="layout_margin_user_input_field"
-        /><sub>pt</sub><br />
-
-        <label>(B) Amount of sewing points:</label>
-        <input
-          type="number"
-          id="sewing_marks_amount"
-          name="sewing_marks_amount"
-          class="layout_margin_user_input_field"
-        /><br />
-
-        <label>(C) Tape width:</label>
-        <input
-          type="number"
-          id="sewing_marks_tape_width_pt"
-          name="sewing_marks_tape_width_pt"
-          class="layout_margin_user_input_field"
-        /><sub>pt</sub><br />
-        <img
-          alt="sewing image"
-          src="/img/sewing_settings_explanation.png"
-          style="height: 100%; width: 100%; object-fit: contain"
-        />
-      </span>
-    </details>
-  </span>
-
   <span class="row">
     <div class="layout_margin_description">
       White Space Manipulation. All values are in points, relative to original document.<br /><br />1
diff --git a/src/main.js b/src/main.js
index 5a1b15c..0a97b9a 100644
--- a/src/main.js
+++ b/src/main.js
@@ -9,6 +9,7 @@ import {
   handleGenerateClick,
   handlePreviewClick,
   handleResetSettingsClick,
+  handleSewingMarksCheckboxState,
 } from './utils/clickHandlers.js';
 import { renderPaperSelectOptions } from './utils/renderUtils.js';
 
@@ -25,6 +26,7 @@ window.addEventListener('DOMContentLoaded', () => {
   const fileInput = document.getElementById('input_file');
   const inputs = document.querySelectorAll('input, select');
   const sourceRotation = document.getElementById('source_rotation');
+  const sewingMarks = document.getElementById('add_sewing_marks_checkbox');
   const sourceRotationExamples = Array.from(
     document.getElementsByClassName('source_rotation_example')
   );
@@ -53,4 +55,8 @@ window.addEventListener('DOMContentLoaded', () => {
       example.style.display = example.id === selectedValue ? 'block' : 'none';
     });
   });
+  sewingMarks.addEventListener('change', (e) => {
+    const willBeEnabled = e.srcElement.checked;
+    handleSewingMarksCheckboxState(willBeEnabled);
+  });
 });
diff --git a/src/models/configuration.js b/src/models/configuration.js
index e5a56a5..4006aae 100644
--- a/src/models/configuration.js
+++ b/src/models/configuration.js
@@ -35,6 +35,10 @@ const sourceRotation = urlSafe(
   z.enum(['none', '90cw', '90ccw', 'out_binding', 'in_binding'])
 ).default('none');
 
+const sewingMarkLocation = urlSafe(z.enum(['all', 'only_out', 'only_in', 'in_n_out'])).default(
+  'all'
+);
+
 /** @type { keyof typeof import("../constants").PAGE_SIZES } */
 const availablePaperSizes = Object.keys(PAGE_SIZES);
 
@@ -97,6 +101,7 @@ export const schema = z.object({
   flyleafs: urlSafe(z.coerce.number()).default(1),
 
   sewingMarksEnabled: urlSafe(coercedBoolean).default(false),
+  sewingMarkLocation,
   sewingMarksMarginPt: urlSafe(z.coerce.number()).default(72),
   sewingMarksAmount: urlSafe(z.coerce.number()).default(3),
   sewingMarksTapeWidthPt: urlSafe(z.coerce.number()).default(36),
diff --git a/src/signatures.js b/src/signatures.js
index 0f8a861..7bf27c5 100644
--- a/src/signatures.js
+++ b/src/signatures.js
@@ -148,26 +148,40 @@ export class Signatures {
 
       const block = [...front_block, ...back_block];
 
+      console.log(
+        `Looking front_config : block.length ${block.length} : given center ${center}, front_start ${front_start} - front_end ${front_end}, back_start ${back_start} - back_end ${back_end}, pages.length ${pages.length}`
+      );
       front_config.forEach((pnum) => {
         const page = block[pnum - 1]; //page layouts are 1-indexed, not 0-index
         pagelistdetails[0].push({
           info: page,
           isSigStart: front_start == 0 && pnum == 1,
           isSigEnd: front_start == 0 && pnum == block.length,
+          isSigMiddle: front_end == back_start && block.length / 2 + 1 == pnum,
           signatureNum: sig_num,
         });
+        console.log(
+          `  >> ${pnum}  :: ${page}  :: ${front_end == back_start && block.length / 2 + 1 == pnum}`
+        );
       });
 
       const backlist = this.duplex ? 0 : 1;
 
+      console.log(
+        `Looking back_config : given center ${center}, front_start ${front_start} - front_end ${front_end}, back_start ${back_start} - back_end ${back_end}, pages.length ${pages.length}`
+      );
       back_config.forEach((pnum) => {
         const page = block[pnum - 1];
         pagelistdetails[backlist].push({
           info: page,
           isSigStart: front_start == 0 && pnum == 1,
           isSigEnd: front_start == 0 && pnum == block.length,
+          isSigMiddle: front_end == back_start && block.length / 2 + 1 == pnum,
           signatureNum: sig_num,
         });
+        console.log(
+          `  >> ${pnum}  :: ${page}  :: ${front_end == back_start && block.length / 2 + 1 == pnum}`
+        );
       });
 
       // Update all our counters
diff --git a/src/utils/clickHandlers.js b/src/utils/clickHandlers.js
index 6c6e3a0..5517581 100644
--- a/src/utils/clickHandlers.js
+++ b/src/utils/clickHandlers.js
@@ -53,3 +53,12 @@ export function handleResetSettingsClick(book) {
   updateAddOrRemoveCustomPaperOption();
   updatePaperSelectOptionsUnits();
 }
+
+export function handleSewingMarksCheckboxState(sewingMarksEnabled) {
+  const sewingMarkDetailsEl = document.getElementById('sewing_marks_details');
+  if (sewingMarksEnabled) {
+    sewingMarkDetailsEl.setAttribute('open', 0);
+  } else {
+    sewingMarkDetailsEl.removeAttribute('open');
+  }
+}
diff --git a/src/utils/drawing.js b/src/utils/drawing.js
index 5fa98b8..3cd62a1 100644
--- a/src/utils/drawing.js
+++ b/src/utils/drawing.js
@@ -124,12 +124,20 @@ export function drawCropmarks(papersize, per_sheet) {
 /**
  * @param {@param {import("../book.js").PageInfo}} sigDetails - information about signature where marks will be printed
  * @param {import("../book.js").Position} position - position info object
+ * @param sewingMarkLocation - see ./models/configuration.js for possible values
  * @param {number} amount - amount of sewing crosses.
  * @param {number} marginPt - distance from the end of sheet of paper to kettle mark
  * @param {number} tapeWidthPt - distance between two points in a single sewwing cross.
  * @returns {Point[]}
  */
-export function drawSewingMarks(sigDetails, position, amount, marginPt, tapeWidthPt) {
+export function drawSewingMarks(
+  sigDetails,
+  position,
+  sewingMarkLocation,
+  amount,
+  marginPt,
+  tapeWidthPt
+) {
   // Here normalize coordinates to always think in x an y like this
   // | P        |H|    P |
   // |  A       |E|   A  |
@@ -139,7 +147,17 @@ export function drawSewingMarks(sigDetails, position, amount, marginPt, tapeWidt
   // |-POSITION-| |      |
 
   // Left pages have spine position on the edge :/
+  console.log('try to draw');
   if (position.isLeftPage) return [];
+  console.log('  on right');
+
+  if (sewingMarkLocation == 'only_out' && !sigDetails.isSigStart) return [];
+  console.log('  a');
+  if (sewingMarkLocation == 'only_in' && !sigDetails.isSigMiddle) return [];
+  console.log('  b');
+  if (sewingMarkLocation == 'in_n_out' && !(sigDetails.isSigStart || sigDetails.isSigMiddle))
+    return [];
+  console.log('  c');
 
   var arePageRotated = Math.abs(position.rotation) === 90;
   let totalSpineHeight = 0;
diff --git a/src/utils/formUtils.js b/src/utils/formUtils.js
index fb81749..89618a6 100644
--- a/src/utils/formUtils.js
+++ b/src/utils/formUtils.js
@@ -42,7 +42,8 @@ const fromFormToConfiguration = (form) =>
     paperSizeCustomWidth: form.get('paper_size_custom_width'),
     paperSizeCustomHeight: form.get('paper_size_custom_height'),
 
-    sewingMarksEnabled: form.get('add_sewing_marks_checkbox'),
+    sewingMarksEnabled: form.has('add_sewing_marks_checkbox'),
+    sewingMarkLocation: form.get('sewing_mark_locations'),
     sewingMarksMarginPt: form.get('sewing_marks_margin_pt'),
     sewingMarksAmount: form.get('sewing_marks_amount'),
     sewingMarksTapeWidthPt: form.get('sewing_marks_tape_width_pt'),
diff --git a/src/utils/layout.js b/src/utils/layout.js
index a040371..033792f 100644
--- a/src/utils/layout.js
+++ b/src/utils/layout.js
@@ -131,7 +131,7 @@ export function calculateDimensions(book) {
     top: padding_pt.top * sy,
   };
 
-  // page_positioning has 2 options: centered, binding_alinged
+  // page_positioning has 2 options: centered, binding_aligned
   const positioning = page_positioning;
 
   const xForeEdgeShiftFunc = function () {
diff --git a/src/utils/renderUtils.js b/src/utils/renderUtils.js
index 3a3fdba..101d378 100644
--- a/src/utils/renderUtils.js
+++ b/src/utils/renderUtils.js
@@ -3,6 +3,7 @@
 // file, You can obtain one at https://mozilla.org/MPL/2.0/.
 
 import { PAGE_SIZES } from '../constants';
+import { handleSewingMarksCheckboxState } from './clickHandlers.js';
 
 export function renderPageCount(book) {
   const pageCount = document.getElementById('page_count');
@@ -203,8 +204,11 @@ export function renderFormFromSettings(configuration) {
   ).checked = true;
 
   // Set french link stitches settings
+  handleSewingMarksCheckboxState(configuration.sewingMarksEnabled);
   document.querySelector('input[name="add_sewing_marks_checkbox"]').checked =
     configuration.sewingMarksEnabled;
+  document.querySelector('select[name="sewing_mark_locations"]').value =
+    configuration.sewingMarkLocation;
   document.querySelector('input[name="sewing_marks_margin_pt"]').value =
     configuration.sewingMarksMarginPt;
   document.querySelector('input[name="sewing_marks_amount"]').value =

From 466bca24c4f28ac14025bbfd497cab3a4fd65d77 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Sun, 5 May 2024 10:25:04 -0700
Subject: [PATCH 09/10] bumping version #

---
 package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package.json b/package.json
index 68f9189..4529599 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "bookbinder",
-  "version": "1.5.0",
+  "version": "1.6.0",
   "description": "An app to rearrange PDF pages for printing for bookbinding",
   "type": "module",
   "scripts": {

From 07972631768fe1b0ddd8eca95fe4665bc565a340 Mon Sep 17 00:00:00 2001
From: Sithel <sithel@gmail.com>
Date: Tue, 7 May 2024 07:44:24 -0700
Subject: [PATCH 10/10] turning off preview hack

---
 src/book.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/book.js b/src/book.js
index 289c804..27589dc 100644
--- a/src/book.js
+++ b/src/book.js
@@ -307,7 +307,7 @@ export class Book {
    */
   async createoutputfiles(isPreview) {
     // set this to `true` to enable full-book previews (placeholder till it's in the UI)
-    const fullPreviewDevHack = true;
+    const fullPreviewDevHack = false;
     const previewFrame = document.getElementById('pdf');
     let previewPdf = null;