[PR #690] One icon per connection type in NetworkManager module #7900

New issue

Open

opened 2026-05-23 02:56:06 +01:00 by JakeStanger · 0 comments

JakeStanger commented

2026-05-23 02:56:06 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/JakeStanger/ironbar/pull/690
Author: @Zedfrigg
Created: 8/4/2024
Status: 🔄 Open

Base: master ← Head: feat/networkmanager-multi-icon

📝 Commits (10+)

05e0083 feat: add networkmanager module
3e9cf85 Merge branch 'JakeStanger:master' into feat/networkmanager
33d367c feat: networkmanager module is now an optional Cargo feature
b92c8d6 feat: add icon size config and cellular support to networkmanager module
b96d696 docs: add networkmanager module documentation
4b40455 Merge branch 'JakeStanger:master' into feat/networkmanager
4df4805 docs: remove todo for icon size config
cdc196d fix: missing cfg attribute for networkmanager module in bar loading
85db13d fix: yet another missing cfg attribute for networkmanager module
9031438 Merge branch 'master' into feat/networkmanager

📊 Changes

8 files changed (+498 additions, -297 deletions)

View changed files

📝 Cargo.lock (+12 -64)
📝 Cargo.toml (+2 -2)
📝 src/clients/mod.rs (+1 -1)
➖ src/clients/networkmanager.rs (+0 -171)
➕ src/clients/networkmanager/dbus.rs (+84 -0)
➕ src/clients/networkmanager/event.rs (+18 -0)
➕ src/clients/networkmanager/mod.rs (+239 -0)
📝 src/modules/networkmanager.rs (+142 -59)

📄 Description

Draft PR to track the status of the implementation. Will watch all NM connections and figure out if they should be categorised as ethernet, wifi, cellular, VPN or other. If there is at least one connection in a category, an icon will be shown for it indicating its status in more detail.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/JakeStanger/ironbar/pull/690 **Author:** [@Zedfrigg](https://github.com/Zedfrigg) **Created:** 8/4/2024 **Status:** 🔄 Open **Base:** `master` ← **Head:** `feat/networkmanager-multi-icon` --- ### 📝 Commits (10+) - [`05e0083`](https://github.com/JakeStanger/ironbar/commit/05e00830308a2de11ed478c9ba25e7acb8f480f5) feat: add networkmanager module - [`3e9cf85`](https://github.com/JakeStanger/ironbar/commit/3e9cf85c2cd72f7d221d4fad7fcd331e387674a8) Merge branch 'JakeStanger:master' into feat/networkmanager - [`33d367c`](https://github.com/JakeStanger/ironbar/commit/33d367cdf45b4214671ed8c3e16da01f13391326) feat: networkmanager module is now an optional Cargo feature - [`b92c8d6`](https://github.com/JakeStanger/ironbar/commit/b92c8d65ce800585a20d0d693921be11ce644907) feat: add icon size config and cellular support to networkmanager module - [`b96d696`](https://github.com/JakeStanger/ironbar/commit/b96d696a74ad8145f0dee6cf3e2a11fc86ce9f27) docs: add networkmanager module documentation - [`4b40455`](https://github.com/JakeStanger/ironbar/commit/4b40455932cc392f44d7f704396efbfe33d7b1d5) Merge branch 'JakeStanger:master' into feat/networkmanager - [`4df4805`](https://github.com/JakeStanger/ironbar/commit/4df480522828061b74de1d3e952ed56f8807ffbb) docs: remove todo for icon size config - [`cdc196d`](https://github.com/JakeStanger/ironbar/commit/cdc196dc6fc4a5922581d5c91e2be49777eaada5) fix: missing cfg attribute for networkmanager module in bar loading - [`85db13d`](https://github.com/JakeStanger/ironbar/commit/85db13d2ef3e91dc2c0b124942c25db19bf26252) fix: yet another missing cfg attribute for networkmanager module - [`9031438`](https://github.com/JakeStanger/ironbar/commit/9031438f5643bdb81f046f5a0e6fef6bdb36f804) Merge branch 'master' into feat/networkmanager ### 📊 Changes **8 files changed** (+498 additions, -297 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+12 -64) 📝 `Cargo.toml` (+2 -2) 📝 `src/clients/mod.rs` (+1 -1) ➖ `src/clients/networkmanager.rs` (+0 -171) ➕ `src/clients/networkmanager/dbus.rs` (+84 -0) ➕ `src/clients/networkmanager/event.rs` (+18 -0) ➕ `src/clients/networkmanager/mod.rs` (+239 -0) 📝 `src/modules/networkmanager.rs` (+142 -59) </details> ### 📄 Description Draft PR to track the status of the implementation. Will watch all NM connections and figure out if they should be categorised as ethernet, wifi, cellular, VPN or other. If there is at least one connection in a category, an icon will be shown for it indicating its status in more detail. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

JakeStanger added the

pull-request

label

2026-05-23 02:56:06 +01:00