From 567831883294494e18aeea8628369597f42d5b52 Mon Sep 17 00:00:00 2001 From: Jordan Taylor Date: Tue, 19 Dec 2023 13:41:31 +0000 Subject: [PATCH] Cleanuo --- app/controller/controller.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/app/controller/controller.py b/app/controller/controller.py index b0f1197..168fa26 100644 --- a/app/controller/controller.py +++ b/app/controller/controller.py @@ -109,12 +109,10 @@ def add_tag(self, tag: 'str'): self.curr_elems.tags_display_widget.clear() for tag in self.model.last_selected_table.get_tags(): self.curr_elems.tags_display_widget.addTag(tag) - # self.curr_elems.tags_display_widget.update() def remove_tag(self, tag: 'str'): self.model.last_selected_table.remove_tag(tag) - # Refresh the tag display self.curr_elems.tags_display_widget.clear() for tag in self.model.last_selected_table.get_tags(): self.curr_elems.tags_display_widget.addTag(tag) @@ -243,7 +241,7 @@ def load_preview( self.view.stop_load_animation() # The original signal emits two arguments, but this slot only takes one - # - why does this work? Granted, we don't need the context argument, but + # - why does this work? Granted, we don't *need* the context argument, but # still, it doesn't make sense why this doesn't throw an error. # TODO debug later def request_suppfile_preview(self, file_data: 'SuppFile'): @@ -264,7 +262,8 @@ def request_suppfile_preview(self, file_data: 'SuppFile'): def preview_processed_table( self, table: 'ProcessedTable', context: 'PageIdentity' ): - self.model.last_selected_table = table # hacky to do this here + # hacky to do this here, should have own method + self.model.last_selected_table = table self.curr_elems.tags_display_widget.clear() for tag in table.get_tags(): self.curr_elems.tags_display_widget.addTag(tag)