reorder frames and tiny improvements

Author: Lupin3000

app_controller.py

@@ -1,7 +1,7 @@
 from glob import glob
 from serial.tools import list_ports
 from logging import getLogger, debug, info, error
-from os.path import expanduser
+from os.path import expanduser, basename
 from customtkinter import CTkButton, CTkFrame
 from tkinter import filedialog, Event
 from webbrowser import open_new
@@ -68,6 +68,10 @@ def __init__(self):
         self.information.mac_info_btn.pack_forget()
         self.information.flash_status_btn.pack_forget()
 
+        # Erase Device
+        self.erase_device = FrameEraseDevice(self)
+        self.erase_device.erase_btn.configure(command=lambda: self._esptool_command("erase_flash"))
+
         # PlugIns
         self.plugins = FramePlugIns(self)
 
@@ -78,10 +82,6 @@ def __init__(self):
         self.plugins.mp_version_btn.pack_forget()
         self.plugins.mp_structure_btn.pack_forget()
 
-        # Erase Device
-        self.erase_device = FrameEraseDevice(self)
-        self.erase_device.erase_btn.configure(command=lambda: self._esptool_command("erase_flash"))
-
         # Flash Firmware
         self.flash_firmware = FrameFirmwareFlash(self)
 
@@ -213,7 +213,7 @@ def _disable_buttons(self) -> None:
 
         :return: None
         """
-        frames: Tuple[CTkFrame, ...] = (self.information, self.plugins, self.erase_device, self.flash_firmware)
+        frames: Tuple[CTkFrame, ...] = (self.information, self.plugins, self.erase_device)
 
         buttons = [
             widget
@@ -225,13 +225,15 @@ def _disable_buttons(self) -> None:
         for button in buttons:
             button.configure(state='disabled')
 
+        self.flash_firmware.flash_btn.configure(state='disabled')
+
     def _enable_buttons(self) -> None:
         """
         Enables specific UI buttons by changing their state to 'normal'.
 
         :return: None
         """
-        frames: Tuple[CTkFrame, ...] = (self.information, self.plugins, self.erase_device, self.flash_firmware)
+        frames: Tuple[CTkFrame, ...] = (self.information, self.plugins, self.erase_device)
 
         buttons = [
             widget
@@ -243,6 +245,8 @@ def _enable_buttons(self) -> None:
         for button in buttons:
             button.configure(state='normal')
 
+        self.flash_firmware.flash_btn.configure(state='normal')
+
     def _search_devices(self) -> None:
         """
         Updates the device dropdown menu with a list of available devices. If there are no
@@ -363,6 +367,7 @@ def _handle_firmware_selection(self) -> None:
         if file_path:
             self.__selected_firmware = file_path
             self.flash_firmware.firmware_checkbox.select()
+            self.flash_firmware.firmware_btn.configure(text=str(basename(file_path)[:15]))
         else:
             self.__selected_firmware = None
             self.flash_firmware.firmware_checkbox.deselect()

ui/frame_erase_device.py

@@ -20,7 +20,7 @@ def __init__(self, master, *args, **kwargs):
         super().__init__(master, *args, **kwargs)
         debug('Create Erase Device Frame')
 
-        self.grid(row=3, column=0, padx=10, pady=5, sticky="nsew")
+        self.grid(row=2, column=0, padx=10, pady=5, sticky="nsew")
         self.grid_columnconfigure(0, weight=1)
 
         self.label = CTkLabel(self, text='Erase')

ui/frame_plugins.py

@@ -20,7 +20,7 @@ def __init__(self, master, *args, **kwargs):
         super().__init__(master, *args, **kwargs)
         debug('Create PlugIns Frame')
 
-        self.grid(row=2, column=0, padx=10, pady=5, sticky="nsew")
+        self.grid(row=3, column=0, padx=10, pady=5, sticky="nsew")
         self.grid_columnconfigure(0, weight=1)
 
         self.label = CTkLabel(self, text='PlugIns')