Disable code line numbers if requested

iamgio · Jul 22, 2024 · 9c5741c · 9c5741c
1 parent 75ba7ea
commit 9c5741c
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/core/src/main/kotlin/eu/iamgio/quarkdown/rendering/html/BaseHtmlNodeRenderer.kt b/core/src/main/kotlin/eu/iamgio/quarkdown/rendering/html/BaseHtmlNodeRenderer.kt
@@ -82,12 +82,13 @@ open class BaseHtmlNodeRenderer(context: Context) : TagNodeRenderer<HtmlTagBuild
             tag("code") {
                 +escapeCriticalContent(node.content)
 
-                if (!node.showLineNumbers) {
-                    // Prevents line numbers to show.
-                    attribute("class", "nohljsln") // TODO doesnt work with tag, but does with buildTag
-                }
+                classes(
+                    // Sets the code language.
+                    node.language?.let { "language-$it" },
+                    // Disables line numbers.
+                    "nohljsln".takeUnless { node.showLineNumbers },
+                )
             }
-                .optionalAttribute("class", node.language?.let { "language-$it" })
         }
 
     override fun visit(node: HorizontalRule) =

diff --git a/core/src/main/kotlin/eu/iamgio/quarkdown/rendering/html/HtmlTagBuilder.kt b/core/src/main/kotlin/eu/iamgio/quarkdown/rendering/html/HtmlTagBuilder.kt
@@ -79,10 +79,22 @@ class HtmlTagBuilder(
      * Applies a CSS style via the `style` attribute to this tag.
      * The attribute is _not_ added if the generated CSS string is empty.
      * @param init CSS builder initialization
+     * @return this for concatenation
      * @see css
      */
     fun style(init: CssBuilder.() -> Unit) = optionalAttribute("style", css(init).takeUnless { it.isEmpty() })
 
+    /**
+     * Applies a sequence of class names via the `class` attribute to this tag.
+     * @param classes class names. `null` elements are ignored. The attribute is not applied if all elements are `null`
+     * @return this for concatenation
+     */
+    fun classes(vararg classes: String?) =
+        optionalAttribute(
+            "class",
+            classes.asSequence().filterNotNull().joinToString(separator = " ").takeIf { it.isNotEmpty() },
+        )
+
     /**
      * @return this builder and its nested content into stringified HTML code.
      */