From 535bc505f76e46c9765a5c3b333a40be18b9599e Mon Sep 17 00:00:00 2001 From: aclist <92275929+aclist@users.noreply.github.com> Date: Thu, 12 Nov 2026 02:06:20 +0900 Subject: [PATCH] chore: update comments --- dzgui/model/servers.py | 4 ++-- dzgui/views/components/right_panel.py | 5 +++-- dzgui/views/components/statusbar.py | 2 +- dzgui/views/dialogs/filepicker.py | 1 + dzgui/views/pages/offline.py | 1 - 5 files changed, 7 insertions(+), 6 deletions(-) diff --git a/dzgui/model/servers.py b/dzgui/model/servers.py index 115655b..c1e5200 100644 --- a/dzgui/model/servers.py +++ b/dzgui/model/servers.py @@ -341,8 +341,8 @@ class ServerModelManager: return config_man.add_saved_server(fqip) - # FIXME: this is valid if saved servers tab is already open, - # but not if app was just booted; causes single row to appear prematurely + # TODO: this is valid if saved servers tab is already open, + # but not if tab is unloaded; causes single row to appear prematurely if proxy_man.has_control_model() is False: proxy_man.push(records) else: diff --git a/dzgui/views/components/right_panel.py b/dzgui/views/components/right_panel.py index c04bba7..078bb49 100644 --- a/dzgui/views/components/right_panel.py +++ b/dzgui/views/components/right_panel.py @@ -106,14 +106,15 @@ class RightPanel(Gtk.Box): if page.loaded is True: return self.filters_vbox.set_sensitive(False) - # TODO: unless it is lan page - # self.refresh_button.set_sensitive(False) + self.refresh_button.set_sensitive(False) def _on_lan_page_init(self, emitter: "Emitter") -> None: self.filters_vbox.set_sensitive(False) def _on_servers_loaded(self, emitter: "Emitter", context: "ServerTab") -> None: # TODO: similar logic on notebook page change + # TODO: wipe maps store when changing tabs if model is None + state = self.controller.has_server_model() self.filters_vbox.set_sensitive(state) diff --git a/dzgui/views/components/statusbar.py b/dzgui/views/components/statusbar.py index 43e5ff7..fd1ad81 100644 --- a/dzgui/views/components/statusbar.py +++ b/dzgui/views/components/statusbar.py @@ -101,7 +101,7 @@ class Statusbar(Gtk.Grid): if type(context) is NotebookPage: self.prior_enum = context self.prior_status = "" - # FIXME: CalcDist is being called when table is not loaded + if dist is None: self.set_by_context(context, "") return diff --git a/dzgui/views/dialogs/filepicker.py b/dzgui/views/dialogs/filepicker.py index 7951ae7..7fd1359 100644 --- a/dzgui/views/dialogs/filepicker.py +++ b/dzgui/views/dialogs/filepicker.py @@ -54,6 +54,7 @@ class FolderPicker(Gtk.FileChooserDialog): return None if res == Gtk.ResponseType.OK: folder = self.get_current_folder() + # TODO: this should never be possible if folder is None: try: uri = self.get_uri() diff --git a/dzgui/views/pages/offline.py b/dzgui/views/pages/offline.py index 141c722..6398e4a 100644 --- a/dzgui/views/pages/offline.py +++ b/dzgui/views/pages/offline.py @@ -136,7 +136,6 @@ class FolderHBox(HBox): self.folder = "" self.label.set_label("") self.unset_button.hide() - # FIXME: rename signal self.emitter.emit("custom_mods_unloaded", self) def get_button(self) -> Gtk.Button: