fix(ALConfigWidget): optimize the logic when delete user list item

src/gui/ALConfigWidget.py

@@ -20,8 +20,8 @@ from PySide6.QtGui import QCloseEvent
 
 from gui.Ui_ALConfigWidget import Ui_ALConfigWidget
 from gui.SeatMapWidget import SeatMapWidget
-
 from gui.SeatMapTable import seats_maps
+
 from utils.ConfigReader import ConfigReader
 from utils.ConfigWriter import ConfigWriter
 
@@ -596,8 +596,12 @@ class ALConfigWidget(QWidget, Ui_ALConfigWidget):
 
         current_item = self.UserListWidget.currentItem()
         if current_item:
-            self.UserListWidget.takeItem(self.UserListWidget.row(current_item))
-            self.UserListWidget.setCurrentItem(None)
+            current_index = self.UserListWidget.row(current_item)
+            self.UserListWidget.takeItem(current_index)
+            if current_index < self.UserListWidget.count():
+                self.UserListWidget.setCurrentRow(current_index)
+            else:
+                self.UserListWidget.setCurrentItem(None)
 
     @Slot()
     def onShowPasswordCheckBoxChecked(

src/gui/SeatFrame.py

@@ -28,9 +28,9 @@ class SeatFrame(QFrame):
         super().__init__(parent)
         self.__seat_number = seat_number
         self.__is_selected = False
-        self.setUpUi()
+        self.setupUi()
 
-    def setUpUi(
+    def setupUi(
         self
     ):
 

src/gui/SeatMapWidget.py

@@ -40,7 +40,7 @@ class SeatMapWidget(QWidget):
         self.__seats_data = seats_data
         self.__selected_seats = []
         self.__seat_frames = {}
-        self.setUpUi()
+        self.setupUi()
         self.connectSignals()
 
     @staticmethod
@@ -55,7 +55,7 @@ class SeatMapWidget(QWidget):
         return seat_number.zfill(3)
 
 
-    def setUpUi(
+    def setupUi(
         self
     ):