From ffe2816118b3f15913f84133185b8b9af539c944 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Thu, 19 May 2022 13:04:55 +0100
Subject: [PATCH] Break out TextOptions::start_element (from currency)

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---
 src/currency.rs |  7 ++-----
 src/ui.rs       | 11 +++++++++++
 2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/src/currency.rs b/src/currency.rs
index 9d29bde7..61af1ff1 100644
--- a/src/currency.rs
+++ b/src/currency.rs
@@ -118,13 +118,10 @@ impl PieceTrait for Banknote {
     self.image.svg(f, vpid, gpc.face, &gpc.xdata)?;
     
     let value: &Value = gpc.xdata.get_exp()?;
-    let label_y_adj = self.label_options.size * SVG_FONT_Y_ADJUST_OF_FONT_SIZE;
 
     hwrite!(f,
-            r##"<{} text-align="center" text-anchor="middle" x="0" y="{}" fill="{}" font-size="{}">{}{}</text>"##,
-            HTML_TEXT_LABEL_ELEM_START,
-            label_y_adj, &self.label_options.colour, self.label_options.size,
-            value.qty, &self.currency)?;
+            r##"<{}>{}{}</text>"##,
+            &self.label_options.start_element(), value.qty, &self.currency)?;
   }
 
   #[throws(ApiPieceOpError)]
diff --git a/src/ui.rs b/src/ui.rs
index fb35c410..e31cc484 100644
--- a/src/ui.rs
+++ b/src/ui.rs
@@ -133,3 +133,14 @@ impl Display for AbbrevPresentationLayout {
     })?
   }
 }
+
+impl TextOptions {
+  pub fn start_element(&self) -> Html {
+    let y_adj = self.size * SVG_FONT_Y_ADJUST_OF_FONT_SIZE;
+    hformat!{
+ r##"{} text-align="center" text-anchor="middle" x="0" y="{}" fill="{}" font-size="{}px""##,
+             HTML_TEXT_LABEL_ELEM_START,
+             y_adj, &self.colour, self.size,
+    }
+  }
+}
-- 
2.30.2