From 07c6310ffdc05f74e7e0f35951896ac42f1cd200 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Sat, 25 Feb 2023 12:46:21 +0300
Subject: [PATCH] better defaults + minor fixes + notes...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 examples/spread-as-background.tex |  1 +
 photobook.cls                     | 13 +++++++++++++
 2 files changed, 14 insertions(+)

diff --git a/examples/spread-as-background.tex b/examples/spread-as-background.tex
index b8d1e99..160dbd2 100644
--- a/examples/spread-as-background.tex
+++ b/examples/spread-as-background.tex
@@ -49,6 +49,7 @@
 \end{pagecell}
 
 
+% - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 
 \begin{spreadtopages*}{bg-spread-image}
 	\imagecell[fill, clearance=-\bleed]{}{\TestImage}%
diff --git a/photobook.cls b/photobook.cls
index 4fb5c62..d2ac551 100644
--- a/photobook.cls
+++ b/photobook.cls
@@ -44,6 +44,8 @@
 % XXX see where we need to \ignorespaces...
 % XXX might be a good idea to add a spine calculator...
 % XXX might be fun to add a 'box' layout to build book boxes...
+% XXX add a grid cell / image cell (???)
+%		make it trivial to define AxB cell layouts and fill them (fully/partially)
 % XXX manual / book:
 %		- basic usage
 %			- setup
@@ -601,6 +603,8 @@
 \RequirePackage{trimclip}
 %% \href{https://ctan.org/pkg/xcolor}{xcolor},
 \RequirePackage{xcolor}
+%% \href{https://ctan.org/pkg/pagecolor}{pagecolor},
+\RequirePackage{pagecolor}
 %% \href{https://ctan.org/pkg/colorspace}{colorspace},
 \RequirePackage{colorspace}
 %% \href{https://ctan.org/pkg/graphicx}{graphicx},
@@ -1133,6 +1137,12 @@
 	paperwidth=\bleedblockwidth, paperheight=\bleedblockheight,
 ]{geometry}
 
+% colors...
+% This prevents \pagecolor from getting drawn over other elements...
+\pagecolor{white}
+\pagecolor{none}
+\color{black}
+
 
 % - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 % pre-document...
@@ -2435,6 +2445,9 @@
 			\begin{minipage}[t][\cellheight][#2]{\cellwidth}%
 				\begin{\photobook@captioncell@align}%
 					\ignorespaces%
+					% XXX BUG: for some reason the text here will not line-break...
+					%		putting it in a minipage here does nothing while the same
+					%		minipage on the lient size fixes the issue...
 					#3%
 				\end{\photobook@captioncell@align}%
 			\end{minipage}