networkWindow.py: refactor to include listview

wifiConnectivityWindow.ui: change horizontalLayout to a vertical layout with a listview and a vertical scrollbar
wifiConnectivityWindow.py: generated file from QtDesigner with some optimizations

Signed-off-by: Guilherme Costa <guiherme.costa@blockstec.com>

Author: Guilherme Costa

BlocksScreen/lib/panels/networkWindow.py

@@ -1,18 +1,16 @@
+import copy
 import logging
-import typing
-import copy 
 import subprocess
+import typing
 from functools import partial
 
 from PyQt6 import QtCore, QtGui, QtWidgets
-from PyQt6.QtCore import QRunnable, QThreadPool, QObject, pyqtSignal, QVariant
-from PyQt6.QtWidgets import QScroller, QScrollerProperties
+from PyQt6.QtCore import QRunnable, QThreadPool, QObject, pyqtSignal
 
 from lib.network import SdbusNetworkManagerAsync
 from lib.panels.widgets.popupDialogWidget import Popup
 from lib.ui.wifiConnectivityWindow_ui import Ui_wifi_stacked_page
 from lib.panels.widgets.keyboardPage import CustomQwertyKeyboard
-from lib.utils.blocks_frame import BlocksCustomFrame
 from lib.panels.widgets.loadPage import LoadScreen
 from lib.utils.list_model import EntryDelegate, EntryListModel, ListItem
 
@@ -216,9 +214,6 @@ def __init__(self, parent: typing.Optional[QtWidgets.QWidget], /) -> None:
 
         self.panel = Ui_wifi_stacked_page()
         self.panel.setupUi(self)
-        
-        self._setupUI()
-        
         self._provider = WifiIconProvider()
         self.ongoing_update: bool = False
 
@@ -237,16 +232,14 @@ def __init__(self, parent: typing.Optional[QtWidgets.QWidget], /) -> None:
 
         #View Models and Delegates
         self.model = EntryListModel()
-        self.model.setParent(self.network_list_widget)
+        self.model.setParent(self.panel.listView)
         self.entry_delegate = EntryDelegate()
-        self.network_list_widget.setModel(self.model)
-        self.network_list_widget.setItemDelegate(self.entry_delegate)
+        self.panel.listView.setModel(self.model)
+        self.panel.listView.setItemDelegate(self.entry_delegate)
         self.entry_delegate.item_selected.connect(self.ssid_item_clicked)
-        self.panel.network_backButton.clicked.connect(self.reset_view_model)
 
         # Network Scan
         self.build_network_list()
-
         self.network_list_worker = BuildNetworkList()
         self.network_list_worker.finished_network_list_build.connect(
             self.handle_network_list
@@ -412,8 +405,15 @@ def __init__(self, parent: typing.Optional[QtWidgets.QWidget], /) -> None:
 
         self.network_list_worker.build()
         self.request_network_scan.emit()
+        self.panel.listView.verticalScrollBar().valueChanged.connect(
+            self._handle_scrollbar
+        )
+        self.panel.verticalScrollBar.valueChanged.connect(self._handle_scrollbar)
+        self.panel.verticalScrollBar.valueChanged.connect(
+            lambda value: self.panel.listView.verticalScrollBar().setValue(value)
+        )
+        self.panel.verticalScrollBar.show()
         self.hide()
-        
         self.info_box_load()
 
         self.qwerty = CustomQwertyKeyboard(self)
@@ -476,20 +476,28 @@ def deleteLater(self) -> None:
 
     def showEvent(self, event: QtGui.QShowEvent | None) -> None:
         """Re-add clients to update list"""
+        # Block all touch events so multitouch is ignored
+        if event.type() in (
+            QtCore.QEvent.Type.TouchBegin,
+            QtCore.QEvent.Type.TouchUpdate,
+            QtCore.QEvent.Type.TouchEnd
+        ):
+            return True  # ignore the event entirely
+        
         self.build_model_list()
         return super().showEvent(event)
 
     def build_model_list(self) -> None:
         """Builds the model list (`self.model`) containing updatable clients"""
-        self.network_list_widget.blockSignals(True)
-        self.model.clear()
-
-        test:dict = copy.copy(self.saved_network)
-        if test.items():
-            for ssid,(signal,is_saved) in test.items():
+        self.panel.listView.blockSignals(True)
+        self.reset_view_model()
+        saved_networks:dict = copy.copy(self.saved_network)
+        if saved_networks.items():
+            for ssid,(signal,is_saved) in saved_networks.items():
                 self.add_network_entry(ssid=ssid, signal=signal, is_saved=is_saved)
-        
-        self.network_list_widget.blockSignals(False)
+            self._setup_scrollbar()
+
+        self.panel.listView.blockSignals(False)
 
     def saved_wifi_option_selected(self):
         _sender = self.sender()
@@ -721,7 +729,7 @@ def evaluate_network_state(self, nm_state: str = "") -> None:
         self.info_box_load(False)
         self.panel.wifi_button.setEnabled(True)
         self.panel.hotspot_button.setEnabled(True)
-        self.repaint()
+
 
 
         if (
@@ -877,7 +885,6 @@ def ssid_item_clicked(self, item: ListItem) -> None:
             return 
 
         _current_ssid_name = item.text
-        #_current_ssid_name = self.saved_network.get(item.text, {})
         self.selected_item = copy.copy(item)
         if (
             _current_ssid_name in self.sdbus_network.get_saved_ssid_names()
@@ -912,7 +919,6 @@ def update_network(
 
     @QtCore.pyqtSlot(list, name="finished-network-list-build")
     def handle_network_list(self, data: typing.Dict) -> None:
-        self.network_list_widget.blockSignals(True)
         for entry in data:
             if entry[0] == self.sdbus_network.hotspot_ssid:
                 continue
@@ -922,8 +928,6 @@ def handle_network_list(self, data: typing.Dict) -> None:
                 continue
             self.saved_network[entry[0]] = (entry[1], entry[2])
         self.build_model_list()
-        self.network_list_widget.blockSignals(False)
-               
         self.evaluate_network_state()
         QtCore.QTimer().singleShot(10000, lambda: self.network_list_worker.build())
 
@@ -937,7 +941,7 @@ def handle_button_click(self, ssid: str):
                 self.panel.network_activate_btn.show()
             else:
                 self.panel.network_activate_btn.hide()
-            self.panel.frame.repaint()
+            #self.panel.frame.repaint()
 
         else:
             self.setCurrentIndex(self.indexOf(self.panel.add_network_page))
@@ -1017,67 +1021,35 @@ def add_network_entry(self, ssid: str, signal: int, is_saved:str) -> None:
         """Adds a new item to the list model"""
 
         wifi_pixmap = self._provider.get_pixmap(signal=signal, state=is_saved)
-            
+        ssid = ssid if ssid is not "" else "UNKOWN"
         item = ListItem(
             text=ssid,
             left_icon=wifi_pixmap,
             right_text=is_saved,
             selected=False,
             allow_check=False,
             _lfontsize=17,
-            _rfontsize=13,
-            height=70,
+            _rfontsize=12,
+            height=80,
         )
         self.model.add_item(item)
+   
+    def _handle_scrollbar(self, value):
+        # Block signals to avoid recursion
+        self.panel.verticalScrollBar.blockSignals(True)
+        self.panel.verticalScrollBar.setValue(value)
+        self.panel.verticalScrollBar.blockSignals(False)
 
-    def _setupUI(self) -> None:
-        
-        """Sets up the UI components and layout for the network window."""
-
-        font_id = QtGui.QFontDatabase.addApplicationFont(
-            ":/font/media/fonts for text/Momcake-Bold.ttf"
+    def _setup_scrollbar(self) -> None:
+        self.panel.verticalScrollBar.setMinimum(
+            self.panel.listView.verticalScrollBar().minimum()
         )
-        font_family = QtGui.QFontDatabase.applicationFontFamilies(font_id)[0]
-        sizePolicy = QtWidgets.QSizePolicy(
-            QtWidgets.QSizePolicy.Policy.MinimumExpanding,
-            QtWidgets.QSizePolicy.Policy.MinimumExpanding,
+        self.panel.verticalScrollBar.setMaximum(
+            self.panel.listView.verticalScrollBar().maximum()
+        )
+        self.panel.verticalScrollBar.setPageStep(
+            self.panel.listView.verticalScrollBar().pageStep()
         )
-        sizePolicy.setHorizontalStretch(1)
-        #sizePolicy.setVerticalStretch(1)
-        self.setSizePolicy(sizePolicy)
-        self.setMinimumSize(QtCore.QSize(800, 500))
-        #self.setMaximumSize(QtCore.QSize(800, 500))
-        self.setLayoutDirection(QtCore.Qt.LayoutDirection.LeftToRight)
-        self.main_content_layout = QtWidgets.QHBoxLayout()
-        self.main_content_layout.setAlignment(QtCore.Qt.AlignmentFlag.AlignTop)
-                                              
-        font = QtGui.QFont()
-        font.setFamily(font_family)
-        font.setPointSize(24)
-        
-        self.header_title = QtWidgets.QLabel(self)
-        self.header_title.setMinimumSize(QtCore.QSize(100, 60))
-        self.header_title.setMaximumSize(QtCore.QSize(16777215, 60))
-        self.header_title.setFont(font)
-
-        # Timer for loading screen timeout
-        self._load_timer = QtCore.QTimer(self)
-                
-        #Buttons frame for update buttons
-        self.network_buttons_frame = BlocksCustomFrame()
-        #self.network_buttons_frame.setMinimumSize(QtCore.QSize(100, 100))
-        #self.network_buttons_frame.setMaximumSize(QtCore.QSize(800, 450))
-        
-        #List widget for update buttons
-        self.network_list_widget = QtWidgets.QListView(self.network_buttons_frame)
-        
-        self.network_buttons_layout = QtWidgets.QVBoxLayout()
-        self.network_buttons_layout.setContentsMargins(15, 20, 20, 5)
-        self.network_buttons_layout.addWidget(self.network_list_widget, 0, QtCore.Qt.AlignmentFlag.AlignBottom)
-        self.network_buttons_frame.setLayout(self.network_buttons_layout)
-
-        self.main_content_layout.addWidget(self.network_buttons_frame, 0)
-        self.setLayout(self.main_content_layout)
 
     def build_network_list(self) -> None:
         """Build available/saved network list with optimized palette setup."""
@@ -1106,56 +1078,4 @@ def set_brush_for_all_groups(palette, role, color, style=QtCore.Qt.BrushStyle.So
         set_brush_for_all_groups(palette, QtGui.QPalette.ColorRole.Link, (0, 0, 255, 0))
 
         # Apply palette
-        self.network_list_widget.setPalette(palette)
-        
-        # General QListView setup
-        self.network_list_widget.setFocusPolicy(QtCore.Qt.FocusPolicy.NoFocus)
-        self.network_list_widget.setStyleSheet("background-color:transparent")
-        self.network_list_widget.setFrameShape(QtWidgets.QFrame.Shape.NoFrame)
-        self.network_list_widget.setFrameShadow(QtWidgets.QFrame.Shadow.Sunken)
-        self.network_list_widget.setVerticalScrollBarPolicy(QtCore.Qt.ScrollBarPolicy.ScrollBarAsNeeded)
-        self.network_list_widget.setHorizontalScrollBarPolicy(QtCore.Qt.ScrollBarPolicy.ScrollBarAlwaysOff)
-        self.network_list_widget.setSizeAdjustPolicy(QtWidgets.QAbstractScrollArea.SizeAdjustPolicy.AdjustIgnored)
-        self.network_list_widget.setAutoScroll(False)
-        self.network_list_widget.setProperty("showDropIndicator", False)
-        self.network_list_widget.setDefaultDropAction(QtCore.Qt.DropAction.IgnoreAction)
-        self.network_list_widget.setAlternatingRowColors(False)
-        self.network_list_widget.setSelectionMode(QtWidgets.QAbstractItemView.SelectionMode.NoSelection)
-        self.network_list_widget.setSelectionBehavior(QtWidgets.QAbstractItemView.SelectionBehavior.SelectItems)
-        self.network_list_widget.setVerticalScrollMode(QtWidgets.QAbstractItemView.ScrollMode.ScrollPerPixel)
-        self.network_list_widget.setHorizontalScrollMode(QtWidgets.QAbstractItemView.ScrollMode.ScrollPerPixel)
-        self.network_list_widget.setUniformItemSizes(True)
-        self.network_list_widget.setSpacing(3)
-        
-        viewport = self.network_list_widget.viewport()
-        QScroller.grabGesture(viewport, QScroller.ScrollerGestureType.TouchGesture)
-        QScroller.grabGesture(viewport, QScroller.ScrollerGestureType.LeftMouseButtonGesture)
-
-        scroller = QScroller.scroller(viewport)
-        props = scroller.scrollerProperties()
-
-        props.setScrollMetric(
-            QScrollerProperties.ScrollMetric.VerticalOvershootPolicy,
-            QVariant(QScrollerProperties.OvershootPolicy.OvershootAlwaysOff)
-        )
-        props.setScrollMetric(
-            QScrollerProperties.ScrollMetric.OvershootDragResistanceFactor,
-            QVariant(1.0)
-        )
-        props.setScrollMetric(
-            QScrollerProperties.ScrollMetric.OvershootDragDistanceFactor,
-            QVariant(0.0)
-        )
-        props.setScrollMetric(
-            QScrollerProperties.ScrollMetric.OvershootScrollDistanceFactor,
-            QVariant(0.0)
-        )
-        props.setScrollMetric(
-            QScrollerProperties.ScrollMetric.OvershootScrollTime,
-            QVariant(0.0)
-        )
-
-        scroller.setScrollerProperties(props)
-
-        self.network_list_widget.setObjectName("network_list_widget")
-        self.panel.nl_content_layout.addWidget(self.network_list_widget)
+        self.panel.listView.setPalette(palette)

BlocksScreen/lib/ui/wifiConnectivityWindow.ui

@@ -6,7 +6,7 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>800</width>
+    <width>852</width>
     <height>480</height>
    </rect>
   </property>
@@ -40,7 +40,7 @@
 </string>
   </property>
   <property name="currentIndex">
-   <number>2</number>
+   <number>1</number>
   </property>
   <widget class="QWidget" name="main_network_page">
    <property name="sizePolicy">
@@ -697,7 +697,61 @@ using the buttons on the side.</string>
      </layout>
     </item>
     <item>
-     <layout class="QVBoxLayout" name="nl_content_layout"/>
+     <layout class="QHBoxLayout" name="horizontalLayout_2">
+      <item>
+       <widget class="QListView" name="listView">
+        <property name="sizePolicy">
+         <sizepolicy hsizetype="MinimumExpanding" vsizetype="MinimumExpanding">
+          <horstretch>1</horstretch>
+          <verstretch>1</verstretch>
+         </sizepolicy>
+        </property>
+        <property name="minimumSize">
+         <size>
+          <width>0</width>
+          <height>0</height>
+         </size>
+        </property>
+        <property name="cursor" stdset="0">
+         <cursorShape>BlankCursor</cursorShape>
+        </property>
+        <property name="styleSheet">
+         <string notr="true">background-color: rgba(255, 255, 255, 0);</string>
+        </property>
+        <property name="frameShape">
+         <enum>QFrame::NoFrame</enum>
+        </property>
+        <property name="frameShadow">
+         <enum>QFrame::Plain</enum>
+        </property>
+        <property name="verticalScrollBarPolicy">
+         <enum>Qt::ScrollBarAlwaysOff</enum>
+        </property>
+        <property name="horizontalScrollBarPolicy">
+         <enum>Qt::ScrollBarAlwaysOff</enum>
+        </property>
+        <property name="verticalScrollMode">
+         <enum>QAbstractItemView::ScrollPerPixel</enum>
+        </property>
+        <property name="horizontalScrollMode">
+         <enum>QAbstractItemView::ScrollPerPixel</enum>
+        </property>
+       </widget>
+      </item>
+      <item>
+       <widget class="CustomScrollBar" name="verticalScrollBar">
+        <property name="sizePolicy">
+         <sizepolicy hsizetype="Fixed" vsizetype="Minimum">
+          <horstretch>0</horstretch>
+          <verstretch>0</verstretch>
+         </sizepolicy>
+        </property>
+        <property name="orientation">
+         <enum>Qt::Vertical</enum>
+        </property>
+       </widget>
+      </item>
+     </layout>
     </item>
    </layout>
   </widget>
@@ -2754,6 +2808,11 @@ Type</string>
    <extends>QLabel</extends>
    <header location="global">lib.panels.widgets.loadWidget</header>
   </customwidget>
+  <customwidget>
+   <class>CustomScrollBar</class>
+   <extends>QScrollBar</extends>
+   <header location="global">lib.utils.blocks_Scrollbar</header>
+  </customwidget>
  </customwidgets>
  <resources>
   <include location="resources/background_resources.qrc"/>