fixed (un-)ordered list generation problems (disabled varargs[HtmlEle…

…ment] procs)
nirokay · Dec 19, 2024 · 4f7c3f8 · 4f7c3f8
1 parent 7bcf05a
commit 4f7c3f8
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/tests/test_html.nim b/tests/test_html.nim
@@ -71,3 +71,10 @@ test "Children - Adding":
         footer("Thank you for reading :3")
     )
     check element.getChildren().len() == 4
+
+test "(Un)Ordered lists":
+    let
+        uList: HtmlElement = unorderedList(@["eggs", "milk", "bacon"])
+        oList: HtmlElement = orderedList(@["eggs", "milk", "bacon"])
+    check $uList == "<ul><li>eggs</li><li>milk</li><li>bacon</li></ul>"
+    check $oList == "<ol><li>eggs</li><li>milk</li><li>bacon</li></ol>"
diff --git a/websitegenerator.nimble b/websitegenerator.nimble
@@ -1,6 +1,6 @@
 # Package
 
-version       = "2.2.0"
+version       = "2.2.1"
 author        = "nirokay"
 description   = "A static html/css generator."
 license       = "GPL-3.0-only"

diff --git a/websitegenerator/common/html.nim b/websitegenerator/common/html.nim
@@ -320,9 +320,9 @@ proc option*(value, text: string): HtmlElement =
     newHtmlElement("option", text).addattr("value", value).forceClosingTag()
 
 proc ol*(children: seq[HtmlElement]): HtmlElement = newHtmlElement("ol", children).forceClosingTag() ## Ordered list element
-proc ol*(children: varargs[HtmlElement]): HtmlElement = ol(children.toSeq()) ## Ordered list element
+#proc ol*(children: varargs[HtmlElement]): HtmlElement = ol(children.toSeq()) ## Ordered list element
 proc ul*(children: seq[HtmlElement]): HtmlElement = newHtmlElement("ul", children).forceClosingTag() ## Unordered list element
-proc ul*(children: varargs[HtmlElement]): HtmlElement = ul(children.toSeq()) ## Unordered list element
+#proc ul*(children: varargs[HtmlElement]): HtmlElement = ul(children.toSeq()) ## Unordered list element
 
 proc li*(text: string): HtmlElement = newHtmlElement("li", text).forceClosingTag() ## List item element
 proc li*(children: seq[HtmlElement]): HtmlElement = newHtmlElement("li", children).forceClosingTag() ## li element