From: Scott Worley <scottworley@scottworley.com>
Date: Sat, 30 Dec 2023 11:19:11 +0000 (-0800)
Subject: Change default layout: below → beside
X-Git-Url: http://git.scottworley.com/paperdoorknob/commitdiff_plain/372211762c57033dfef33075bd593ec5d426f3c7?ds=inline;hp=47e940089d73e13a8f7f1d12344c2eb9bcaa3454

Change default layout: below → beside

:(

Below layout is nicer, but has two problems:

1. The page break logic seems to be broken.  Some pages have content run
   off the bottom.  Other pages get a single line.  This seems to be a
   bug in the wrapstuff package.

2. There's a problem I don't understand where LaTeX complains about the
   \wrapstuffclear following a \footnote, saying:

      ! LaTeX kernel Error: Not in vertical mode.

      Starting a paragraph with \RawIndent or \RawNoindent
      (or \para_raw_indent: or \para_raw_noindent:) is only allowed
      if LaTeX is in vertical mode.

Two problems
---

diff --git a/args.py b/args.py
index 67d8c3f..b765fc6 100644
--- a/args.py
+++ b/args.py
@@ -56,9 +56,9 @@ See https://faculty.bard.edu/bloch/geometry.pdf for details
         default=20)
     parser.add_argument(
         '--layout',
-        default='below',
+        default='beside',
         help='Whether to put character and author information `beside` or `below` the icon ' +
-             '(default: below)')
+             '(default: beside)')
     parser.add_argument(
         '--out',
         help='The filename stem at which to write output ' +
diff --git a/paperdoorknob_test.py b/paperdoorknob_test.py
index 8527dd3..e01327c 100644
--- a/paperdoorknob_test.py
+++ b/paperdoorknob_test.py
@@ -15,7 +15,7 @@ import paperdoorknob
 
 from testing.fakeserver import FakeGlowficServer
 from fetch import DirectFetcher, FakeFetcher, Fetcher
-from glowfic import ContentOnlyLayout, BelowIconLayout
+from glowfic import ContentOnlyLayout, BesideIconLayout
 from images import FakeImageStore
 from spec import Spec
 from texify import DirectTexifier, PandocTexifier, VerifyingTexifier
@@ -85,7 +85,7 @@ Pretty sure.
                 lambda x: x,
                 lambda x: None,
                 lambda x: x,
-                BelowIconLayout(PandocTexifier('pandoc'), 20),
+                BesideIconLayout(PandocTexifier('pandoc'), 20),
                 None,
                 out,
                 lambda _: None)