widgets: Ensure Bus Monitor and Plotter both spawn QApplication with Fusion Style enabled#127
Conversation
…Fusion Style enabled Opening Bus Monitor and Plotter open new QApplications which default to the system style - so this will ensure all QApplications that are spawned use the Fusion style
There was a problem hiding this comment.
Pull request overview
Ensures that subprocess-spawned Qt applications (Bus Monitor and Plotter) use the same “Fusion” style as the main GUI app, preventing platform-default style mismatches.
Changes:
- Set Qt application style to “Fusion” in the Plotter subprocess entry point.
- Set Qt application style to “Fusion” in the Bus Monitor subprocess entry point.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| dronecan_gui_tool/widgets/plotter/init.py | Applies Fusion style to the Plotter’s spawned QApplication for consistent UI styling. |
| dronecan_gui_tool/widgets/bus_monitor/init.py | Applies Fusion style to the Bus Monitor’s spawned QApplication for consistent UI styling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| logger.info('Plotter process started with PID %r', os.getpid()) | ||
| app = QApplication(sys.argv) # Inheriting args from the parent process | ||
|
|
||
| # Set the style for Qt6 to be same across systems |
There was a problem hiding this comment.
Minor grammar: "to be same across systems" reads awkwardly. Consider changing to "to be the same across systems" (or similar) to improve clarity.
| # Set the style for Qt6 to be same across systems | |
| # Set the Qt6 style to be the same across systems |
| logger.info('Bus monitor process started with PID %r', os.getpid()) | ||
| app = QApplication(sys.argv) # Inheriting args from the parent process | ||
|
|
||
| # Set the style for Qt6 to be same across systems |
There was a problem hiding this comment.
Minor grammar: "to be same across systems" reads awkwardly. Consider changing to "to be the same across systems" (or similar) to improve clarity.
| # Set the style for Qt6 to be same across systems | |
| # Set the Qt6 style to be the same across systems |
Opening Bus Monitor and Plotter open new QApplications which default to the system style - so this will ensure all QApplications that are spawned use the Fusion style