From af8e0d6618f85fdc8543904aa1c2da4e0cdf9ce4 Mon Sep 17 00:00:00 2001
From: "Salvador E. Tropea" <salvador@inti.gob.ar>
Date: Fri, 29 Dec 2023 08:56:44 -0300
Subject: [PATCH] [KiRi] Diff canvas background configurable

---
 kibot/out_kiri.py             | 11 +++++++++++
 kibot/resources/kiri/kiri.css |  5 ++++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/kibot/out_kiri.py b/kibot/out_kiri.py
index 51ad3f2a..0d8f7ba1 100644
--- a/kibot/out_kiri.py
+++ b/kibot/out_kiri.py
@@ -61,12 +61,18 @@ class KiRiOptions(VariantOptions):
             """ *Selects the color theme. Only applies to KiCad 6.
                 To use the KiCad 6 default colors select `_builtin_default`.
                 Usually user colors are stored as `user`, but you can give it another name """
+            self.background_color = "#FFFFFF"
+            """ Color used for the background of the diff canvas """
             self.keep_generated = False
             """ *Avoid PCB and SCH images regeneration. Useful for incremental usage """
         super().__init__()
         self._expand_id = 'diff'
         self._expand_ext = 'pdf'
 
+    def config(self, parent):
+        super().config(parent)
+        self.validate_colors(['background_color'])
+
     def get_targets(self, out_dir):
         # TODO: Implement
         return [self._parent.expand_filename(out_dir, self.output)]
@@ -185,6 +191,7 @@ class KiRiOptions(VariantOptions):
                 for ln in src:
                     ln_stripped = ln.strip()
                     if ln_stripped.startswith('<script src="'):
+                        # Embed Java Scripts
                         fn = ln_stripped[13:].split('"')[0]
                         with open(os.path.join(src_dir, fn), 'rt') as f:
                             script = f.read()
@@ -192,8 +199,10 @@ class KiRiOptions(VariantOptions):
                         dst.write(script)
                         dst.write('\n</script>\n')
                     elif ln_stripped.startswith('<link rel="stylesheet" href="'):
+                        # Embed CSS
                         fn = ln_stripped[29:].split('"')[0]
                         if fn == 'layer_colors.css':
+                            # Create the colors
                             script = ''
                             for id, color in self._color_theme.layer_id2color.items():
                                 script += f'.layer_color_{id} {{ color: {color[:7]}; }}\n'
@@ -207,6 +216,8 @@ class KiRiOptions(VariantOptions):
                                         with open(os.path.join(src_dir, fns), 'rt') as f:
                                             svg = f.read().strip()
                                         script += "\t--svg: url('data:image/svg+xml;utf8,"+svg+"');\n"
+                                    elif lns.startswith('#svg-id { background-color: '):
+                                        script += f'#svg-id {{ background-color: {self.background_color}; }}\n'
                                     else:
                                         script += lns
                         else:
diff --git a/kibot/resources/kiri/kiri.css b/kibot/resources/kiri/kiri.css
index f5624712..ea51ce47 100644
--- a/kibot/resources/kiri/kiri.css
+++ b/kibot/resources/kiri/kiri.css
@@ -166,10 +166,13 @@ html, body {
 #svg-id {
 /*	border: 1px solid #111;*/
 	margin-top: 15px;
-	background-color: #000000;
 	border-radius: 5px;
 }
 
+/* Keep it separated for simple parsing */
+#svg-id { background-color: #ffffff; }
+
+
 /* ==============================
 ** ============================*/