add resizeable_dialog_b config setting Fixes #250

Author: semuadmin

README.md

@@ -175,6 +175,7 @@ For more comprehensive installation instructions, please refer to [INSTALLATION.
 31. The behaviour of Toplevel ('pop-up') dialogs will depend on the screen resolution and 'transient' setting. If the width or height of a Toplevel dialog exceeds the screen resolution, the dialog will be displayed in a scrollable, resizeable window. Otherwise, the dialog is displayed as a fixed, non-resizeable panel.
     - A boolean configuration setting `transient_dialog_b` governs whether Toplevel dialogs are 'transient' (i.e. always on top of main application dialog) or not. Changing this setting to `0` allows Toplevel dialogs to be minimised independently of the main application window, but be mindful that some dialogs may end up hidden behind others e.g. "Open file/folder" dialogs. **If a file open button appears unresponsive, check that the "Open file/folder" panel isn't already open but obscured**. 
     - If you're accessing the desktop via a VNC session (e.g. to a headless Raspberry Pi) it is recommended to keep the setting at the default `1`, as VNC may not recognise keystrokes on overlaid non-transient windows.
+    - A boolean configuration setting `resizeable_dialog_b` governs whether *all* Toplevel dialogs are resizeable, irrespective of the default setting in `DialogState`.
 
 #### <a name="widgets">User-selectable widgets</a>
 ---

RELEASE_NOTES.md

@@ -1,5 +1,9 @@
 # PyGPSClient Release Notes
 
+### RELEASE 1.6.9
+
+1. Add manually-editable `resizeable_dialog_b` configuration setting as a workaround for screen scaling issues on some platforms (e.g. Ubuntu Wayland). Fixes #250.
+
 ### RELEASE 1.6.8
 
 1. Updates to experimental RINEX conversion dialog for pygnssutils >=1.2.1. Provisional support for RINEX 4.02 and GPS CNAV added.

pygpsclient.json

@@ -15,6 +15,7 @@
     "IMU Monitor": false,
     "checkforupdate_b": 0,
     "transient_dialog_b": 1,
+    "resize_dialog_b": 0,
     "guiupdateinterval_f": 0.2,
     "mapupdateinterval_n": 60,
     "defaultport_s": "USB",

src/pygpsclient/_version.py

@@ -8,4 +8,4 @@
 :license: BSD 3-Clause
 """
 
-__version__ = "1.6.8"
+__version__ = "1.6.9"

src/pygpsclient/configuration.py

@@ -109,6 +109,7 @@ def __init__(self, app):
             **self.widget_config,
             "checkforupdate_b": 1,
             "transient_dialog_b": 1,  # whether pop-up dialogs are on top of main app window
+            "resizeable_dialog_b": 0,  # whether pop-up dialogs are all resizeable
             "guiupdateinterval_f": GUI_UPDATE_INTERVAL,  # GUI widget update interval in seconds
             "mapupdateinterval_n": MAP_UPDATE_INTERVAL,
             "defaultport_s": RCVR_CONNECTION,

src/pygpsclient/toplevel_dialog.py

@@ -78,9 +78,13 @@ def __init__(self, app, dlgname: str, *args, **kwargs):
         self._dlgname = dlgname
         self.logger = logging.getLogger(f"{APPNAME}.{dlgname}")
         self.width, self.height = 300, 300  # initial, updated in finalise()
-        self._resizable = kwargs.pop(
-            "resizable", self.__app.dialog_state.state[self._dlgname].get(RESIZE, False)
-        )
+        if self.__app.configuration.get("resizeable_dialog_b") == 1:
+            self._resizable = True
+        else:
+            self._resizable = kwargs.pop(
+                "resizable",
+                self.__app.dialog_state.state[self._dlgname].get(RESIZE, False),
+            )
         transient = kwargs.pop(
             "transient", self.__app.configuration.get("transient_dialog_b")
         )

tests/pygpsdata_rinextest.log

@@ -141,7 +141,7 @@ def testconfiguration(self):
         self.assertEqual(cfg.get("lbandclientdrat_n"), 2400)
         self.assertEqual(cfg.get("userport_s"), "")
         self.assertEqual(cfg.get("spartnport_s"), "")
-        self.assertEqual(len(cfg.settings), 156)
+        self.assertEqual(len(cfg.settings), 157)
         kwargs = {"userport": "/dev/ttyACM0", "spartnport": "/dev/ttyACM1"}
         cfg.loadcli(**kwargs)
         self.assertEqual(cfg.get("userport_s"), "/dev/ttyACM0")