Fixed compilation when page numbering was not set and back references…

… are enabled Co-authored-by: St0wy <[email protected]> fixes #31
typst-community · Apr 27, 2024 · 3fd69d7 · 3fd69d7
1 parent 99584f9
commit 3fd69d7
Show file tree

Hide file tree

Showing 9 changed files with 13 additions and 5 deletions.
diff --git a/examples/full-example/main.pdf b/examples/full-example/main.pdf
diff --git a/examples/full-example/main.typ b/examples/full-example/main.typ
@@ -4,7 +4,7 @@
 
 #show: make-glossary
 
-#set page(numbering: "1", paper: "a5")
+#set page(paper: "a5")
 
 //I recommend setting a show rule for the links to that your reader understand that they can click on the references to go to the term in the glossary.
 #show link: set text(fill: blue.darken(60%))

diff --git a/examples/groups/groups.pdf b/examples/groups/groups.pdf
diff --git a/examples/groups/groups.typ b/examples/groups/groups.typ
@@ -4,7 +4,7 @@
 
 #show: make-glossary
 
-#set page(numbering: "1", paper: "a5")
+#set page(paper: "a5")
 
 //I recommend setting a show rule for the links to that your reader understand that they can click on the references to go to the term in the glossary.
 #show link: set text(fill: blue.darken(60%))

diff --git a/examples/import-terms-from-yaml-file/main.pdf b/examples/import-terms-from-yaml-file/main.pdf
diff --git a/examples/import-terms-from-yaml-file/main.typ b/examples/import-terms-from-yaml-file/main.typ
@@ -90,7 +90,8 @@
   print-glossary(entries, show-all: full)
 }
 
-#set page(numbering: "1")
+#set page(paper: "a5")
+
 #show: make-glossary
 
 = Test Document

diff --git a/examples/plural-example/main.pdf b/examples/plural-example/main.pdf
diff --git a/examples/plural-example/main.typ b/examples/plural-example/main.typ
@@ -4,7 +4,7 @@
 
 #show: make-glossary
 
-#set page(numbering: "1", paper: "a5")
+#set page(paper: "a5")
 
 #show link: set text(fill: blue.darken(60%))
 

diff --git a/glossarium.typ b/glossarium.typ
@@ -184,7 +184,14 @@ SOFTWARE.*/
                           pages.push(x.page())
                           (values: values, pages: pages)
                         },
-                      ).values.map(x => link(x)[#numbering(x.page-numbering(), ..counter(page).at(x))]).join(", ")
+                      ).values.map(x => {
+                         let page-numbering = x.page-numbering();
+                          if page-numbering == none {
+                            page-numbering = "1"
+                          }
+                          link(x)[#numbering(page-numbering, ..counter(page).at(x))]
+                        }
+                      ).join(", ")
                     }
                   }
                 }