Network Overhaul#139
Conversation
- Implemented completely new, custom-developed core network features like speed test integration, signal strength, latency, custom dns-switching -etc. - Overhauled the user interface and visual design for a modernised look. - Optimised the code for smoothness and efficiency while sustaining all new and improved features
Added new properties for networking configuration.
|
Hmmm, am not really sure about how to link this to the existing event in the Kanban EDIT: Okay, I think i figured it out! |
|
That's really sick !! I just want to make a pre code review about some stuff I noticed 
The speed test is shown for every other networks. I don't think this is supposed to happen. 
The DNS can overflow when the app is shrinken. Maybe wrap them after the comma. That's pretty much all I noticed. Anyway, great job, I love it ! |
Replaced latency, downloadSpeed, and uploadSpeed properties with a JSON-serialised map for speed test results.
|
@LeVraiArdox all done :P Each network shows their unique stats, as intended.
Extremely long DNS addresses no longer overflow from the tab when the window's dimensions are decreased
|
LeVraiArdox
left a comment
LeVraiArdox
left a comment
There was a problem hiding this comment.
Some stuff to change in the code. Also, I would prefer you to make backend logic in C++ (see network.cpp and network.hpp) than using the frontend file to add backend logic. Or, if you don't want to dirt your hands, make a Network.qml file in services/ that does these stuff.
Aye aye, captain! |
Removed the speedTestResultsJson property from Config.qml.
Refactor Wifi.qml to simplify network property access and improve code clarity. Removed redundant active network retrieval logic and updated property bindings for better performance.
|
@LeVraiArdox Wifi.qml now uses |
| // Currently active (connected) network, sourced directly from the C++ | ||
| // Network singleton - Network.active already tracks exactly this (kept in | ||
| // sync with NetworkManager's active access point via activeChanged), so | ||
| // there's no need to re-derive it by scanning root.networks here. |
There was a problem hiding this comment.
Comment end might be unnecessary
| // Resolved via a Process at startup since Quickshell.env() isn't available | ||
| // on this quickshell build (0.2.0.r131). Empty until ensureCacheDirProcess | ||
| // finishes, at which point speedTestCacheFile.path picks it up. | ||
| property string homeDir: "" |
There was a problem hiding this comment.
Use Directories instead (modules/common/Directories.qml). There is a cache directory too.
There was a problem hiding this comment.
Still too much backend logic that can be written in the cpp singleton. Keep in mind this should be almost only frontend. You might wanna rewrite most of the backend functions in the cpp
2 participants
Network Overhaul
What's New
Sensitive Info & Connection Details
Sensitive Infois switched off, all sensitive information likeIP addresses,BSSIDS- etc are hidden for privacy.Connection Details
If enabled, displays automatically when connected via Ethernet or Wi-Fi.
The section has several useful details about the current connection, such as:
Frequency is no longer shown with the combined channel like,
2462. It is now shown more aesthetically, example, "2,4GHz, Channel 11"Speed test directly integrated, so the user can perform a network test directly from the sleex settings if they wish. Uses the cloudflare provider.
On Wifi
On Ethernet
Custom DNS
Improved the look of Available Networks
Improved the look of the search feature
Integration of QR Codes
Full Showcase