Merge pull request #28 from DavidCEllis/fix-linux-local-images

Linux requires an explicit file:/// URI in order for local links to work

Author: DavidCEllis

src/splitguides/ui/main_window.py

@@ -385,7 +385,9 @@ def update_notes(self, idx, refresh=False):
                 notes=self.notes.render_splits(start, end),
             )
 
-            self.ui.notes.setHtml(html, baseUrl=self.notefile)
+            note_uri = Path(self.notefile).absolute().as_uri()
+
+            self.ui.notes.setHtml(html, baseUrl=note_uri)
             self.split_index = idx
 
     def open_settings(self):

tests/test_ui/test_main_window.py

@@ -223,7 +223,10 @@ def test_update_notes(qtbot, fake_link, idx):
         css=main_window.css,
         notes="Fake Splits",
     )
-    fake_note_ui.setHtml.assert_called_once_with("Fake HTML", baseUrl="Notes_URL")
+
+    note_path = Path("Notes_URL").absolute().as_uri()
+
+    fake_note_ui.setHtml.assert_called_once_with("Fake HTML", baseUrl=note_path)
 
     assert main_window.split_index == used_idx