RC 1.7.4

RELEASE_NOTES.md

@@ -1,5 +1,18 @@
 # PyGPSClient Release Notes
 
+### RELEASE 1.7.4
+
+FIXES:
+
+1. Fix memory leaks in levelsview_frame and spectrum_frame if legend toggled.
+
+ENHANCEMENTS:
+
+1. Add App Configuration Options panel to set internal App configuration parameters (*previously only available via manual edits of json file*). **NB**: User must save configuration and restart app after any changes.
+1. Add "Toggle Settings" button to Banner panel for quick access to Settings (docked or undocked).
+1. Streamline UBX Legacy Configuration panel for improved access on low res displays.
+1. Enhanced validation messaging in UBX and NMEA configuration panels.
+
 ### RELEASE 1.7.3
 
 FIXES:

docs/pygpsclient.rst

@@ -20,6 +20,14 @@ pygpsclient.app module
    :undoc-members:
    :show-inheritance:
 
+pygpsclient.app\_config\_dialog module
+--------------------------------------
+
+.. automodule:: pygpsclient.app_config_dialog
+   :members:
+   :undoc-members:
+   :show-inheritance:
+
 pygpsclient.attitude\_frame module
 ----------------------------------
 
@@ -500,14 +508,6 @@ pygpsclient.ubx\_preset\_frame module
    :undoc-members:
    :show-inheritance:
 
-pygpsclient.ubx\_solrate\_frame module
---------------------------------------
-
-.. automodule:: pygpsclient.ubx_solrate_frame
-   :members:
-   :undoc-members:
-   :show-inheritance:
-
 pygpsclient.uni\_handler module
 -------------------------------
 

pyproject.toml

@@ -54,7 +54,7 @@ dependencies = [
     "Pillow>=12.0.0",
     "pygnssutils>=1.2.7",
     "pyunigps>=1.0.0",
-    "pynmeagps>=1.1.5",
+    "pynmeagps>=1.1.7",
     "pyubx2>=1.3.5",
 ]
 

src/pygpsclient/_version.py

@@ -8,4 +8,4 @@
 :license: BSD 3-Clause
 """
 
-__version__ = "1.7.3"
+__version__ = "1.7.4"

src/pygpsclient/about_dialog.py

@@ -12,7 +12,18 @@
 
 import logging
 from platform import machine, python_version
-from tkinter import Button, Checkbutton, Frame, IntVar, Label, Tcl
+from tkinter import (
+    CENTER,
+    EW,
+    NSEW,
+    Button,
+    Checkbutton,
+    Frame,
+    IntVar,
+    Label,
+    Tcl,
+    ttk,
+)
 from webbrowser import open_new_tab
 
 from PIL import Image, ImageTk
@@ -21,7 +32,6 @@
     CLICK_CURSOR,
     ERRCOL,
     ICON_APP128,
-    ICON_GITHUB,
     ICON_SPONSOR,
     INFOCOL,
     LICENSE_URL,
@@ -61,7 +71,6 @@ def __init__(self, app, *args, **kwargs):  # pylint: disable=unused-argument
         self.__app = app  # Reference to main application class
         self.logger = logging.getLogger(__name__)
         self._img_icon = ImageTk.PhotoImage(Image.open(ICON_APP128).resize((64, 64)))
-        self._img_github = ImageTk.PhotoImage(Image.open(ICON_GITHUB).resize((32, 32)))
         self._img_sponsor = ImageTk.PhotoImage(Image.open(ICON_SPONSOR))
         self._checkonstartup = IntVar()
         self._checkonstartup.set(self.__app.configuration.get("checkforupdate_b"))
@@ -79,16 +88,23 @@ def _body(self):
         """
 
         self._frm_body = Frame(self.container)
-        self._lbl_icon = Label(self._frm_body, image=self._img_icon, borderwidth=0)
-        self._lbl_descs = []
-        for txt in ABOUTTXT:
-            self._lbl_descs.append(
-                Label(
-                    self._frm_body,
-                    text=txt,
-                    borderwidth=0,
-                )
-            )
+        self._lbl_icon = Label(
+            self._frm_body, image=self._img_icon, borderwidth=0, anchor=CENTER
+        )
+        self._lbl_desc = Label(
+            self._frm_body,
+            text=ABOUTTXT,
+            wraplength=400,
+            justify=CENTER,
+            anchor=CENTER,
+        )
+        self._lbl_github = Label(
+            self._frm_body,
+            text=GITHUB_URL,
+            foreground=INFOCOL,
+            cursor=CLICK_CURSOR,
+            anchor=CENTER,
+        )
         tkv = Tcl().call("info", "patchlevel")
         self._lbl_python_version = Label(
             self._frm_body,
@@ -97,14 +113,16 @@ def _body(self):
                 f"Python: {python_version()}  Tk: {tkv}  "
                 f"Spatial: {SQLSTATUS[self.__app.db_enabled]}"
             ),
+            anchor=CENTER,
         )
         self._lbl_lib_versions = []
         for nam, ver in LIBVERSIONS.items():
             self._lbl_lib_versions.append(
                 Label(
                     self._frm_body,
                     text=f"{nam}: {ver}",
-                    borderwidth=0,
+                    anchor=CENTER,
+                    border=0,
                     highlightthickness=0,
                 )
             )
@@ -123,44 +141,49 @@ def _body(self):
             self._frm_body,
             image=self._img_sponsor,
             cursor=CLICK_CURSOR,
-        )
-        self._lbl_github = Label(
-            self._frm_body,
-            text=GITHUB_URL,
-            fg=INFOCOL,
-            cursor=CLICK_CURSOR,
+            anchor=CENTER,
         )
         self._lbl_copyright = Label(
             self._frm_body,
             text=COPYRIGHT,
+            foreground=INFOCOL,
             cursor=CLICK_CURSOR,
+            anchor=CENTER,
         )
 
     def _do_layout(self):
         """
         Arrange widgets in dialog.
         """
 
-        i = 0
-        self._frm_body.grid(column=0, row=0, padx=5, pady=5, ipadx=5, ipady=5)
-        self._lbl_icon.grid(column=0, row=1, columnspan=2, padx=3, pady=0)
-        for i, lbl in enumerate(self._lbl_descs):
-            lbl.grid(column=0, row=2 + i, columnspan=2, padx=3, pady=0)
-        self._lbl_python_version.grid(column=0, row=3 + i, columnspan=2, padx=3, pady=1)
-        n = 4 + i
+        self._frm_body.grid(column=0, row=0, ipadx=2, ipady=2, sticky=NSEW)
+        self._lbl_icon.grid(column=0, row=0, columnspan=2, padx=3, pady=0, sticky=EW)
+        self._lbl_desc.grid(column=0, row=1, columnspan=2, padx=3, pady=0, sticky=EW)
+        self._lbl_github.grid(column=0, row=2, columnspan=2, padx=3, pady=0, sticky=EW)
+        ttk.Separator(self._frm_body).grid(
+            column=0, row=3, columnspan=2, padx=3, pady=3, sticky=EW
+        )
+        self._lbl_python_version.grid(
+            column=0, row=4, columnspan=2, padx=3, pady=1, sticky=EW
+        )
         for i, lbl in enumerate(self._lbl_lib_versions):
-            lbl.grid(column=0, row=n + i, columnspan=2, padx=2)
+            lbl.grid(column=0, row=5 + i, columnspan=2, padx=2, pady=0, sticky=EW)
+        lv = len(self._lbl_lib_versions)
         self._btn_checkupdate.grid(
-            column=0, row=1 + n + i, ipadx=3, ipady=3, padx=3, pady=3
+            column=0, row=6 + lv, ipadx=3, ipady=3, padx=3, pady=3
         )
         self._chk_checkupdate.grid(
-            column=1, row=1 + n + i, ipadx=3, ipady=3, padx=3, pady=3
+            column=1, row=6 + lv, ipadx=3, ipady=3, padx=3, pady=3
+        )
+        ttk.Separator(self._frm_body).grid(
+            column=0, row=7 + lv, columnspan=2, padx=3, pady=3, sticky=EW
         )
         self._lbl_sponsoricon.grid(
-            column=0, row=2 + n + i, columnspan=2, padx=3, pady=3
+            column=0, row=8 + lv, columnspan=2, padx=3, pady=3, sticky=EW
+        )
+        self._lbl_copyright.grid(
+            column=0, row=9 + lv, columnspan=2, padx=3, pady=3, sticky=EW
         )
-        self._lbl_github.grid(column=0, row=3 + n + i, columnspan=2, padx=3, pady=0)
-        self._lbl_copyright.grid(column=0, row=4 + n + i, columnspan=2, padx=3, pady=0)
 
     def _attach_events(self):
         """
@@ -238,7 +261,7 @@ def _check_for_update(self, *args, **kwargs):  # pylint: disable=unused-argument
                 txt += f" - Latest version is {latest}"
                 col = ERRCOL
             self._lbl_lib_versions[i]["text"] = txt
-            self._lbl_lib_versions[i]["fg"] = col
+            self._lbl_lib_versions[i]["foreground"] = col
         updates = [nam for (nam, current, latest) in versions if latest != current]
         if len(updates) > 0:
             self.set_status_label("Updates available", OKCOL)
@@ -272,10 +295,10 @@ def _set_update_btn_mode(self, update: bool):
 
         if update:
             self._btn_checkupdate["text"] = "UPDATE"
-            self._btn_checkupdate["fg"] = OKCOL
+            self._btn_checkupdate["foreground"] = OKCOL
             self._btn_checkupdate.bind("<Button>", self._do_update)
         else:
             self._btn_checkupdate["text"] = "CHECK FOR UPDATES"
-            self._btn_checkupdate["fg"] = INFOCOL
+            self._btn_checkupdate["foreground"] = INFOCOL
             self._btn_checkupdate.bind("<Button>", self._check_for_update)
         self.__app.update_idletasks()