luci icon indicating copy to clipboard operation
luci copied to clipboard

Suggestion for improvement in switch menu

Open siegfriedhoellrigl1 opened this issue 1 year ago • 7 comments

It would be very nice if we could have a "Comment" field for each port under : Network - Switch menu.

siegfriedhoellrigl1 avatar Oct 04 '24 10:10 siegfriedhoellrigl1

What fields do you see today?

systemcrash avatar Oct 04 '24 12:10 systemcrash

Today I can only make comments for the VLANs. Not for the physical Ports.

siegfriedhoellrigl1 avatar Oct 04 '24 14:10 siegfriedhoellrigl1

Which device are you running? The Network -> Switch menu isn't present on targets which has migrated to DSA (Distributed Switch Architecture), so if we want to have a place to make comments like this regarding the physical ports it should probably not be there, see https://github.com/openwrt/luci/blob/master/modules/luci-mod-network/root/usr/share/luci/menu.d/luci-mod-network.json#L11-L12

dannil avatar Oct 04 '24 16:10 dannil

At the moment I am using 3 TP-Links : Archer A7 v5 TL-WDR4300 v1 TL-WR1043ND v2. So you think, this is useless, because the switch menu will disappear in a short time, correct ?

siegfriedhoellrigl1 avatar Oct 07 '24 11:10 siegfriedhoellrigl1

At the moment I am using 3 TP-Links : Archer A7 v5 TL-WDR4300 v1 TL-WR1043ND v2. So you think, this is useless, because the switch menu will appear in a short time, correct ?

Maybe not useless, some devices will probably never get DSA and stay on swconfig, so for them the switch menu would still appear. But, if we're thinking of a general solution about being able to add comments to the physical ports, there maybe could be a nice solution that would incorporate both swconfig and DSA targets.

EDIT: I took a look at all your listed devices and they're all part of ath79 and that target still uses swconfig, which is why you can still see and use the switch menu component. For example, on my Netgear R7800 I can't see it since it's part of ipq806x, which has migrated to DSA.

Screenshot_48

dannil avatar Oct 07 '24 15:10 dannil

Admittedly, it's harmless to have a comment field, since it's only ever used in the GUI here. Lower level scripts aren't looking for it. DSA stuff is under Interfaces, right?

systemcrash avatar Oct 07 '24 16:10 systemcrash

Admittedly, it's harmless to have a comment field, since it's only ever used in the GUI here. Lower level scripts aren't looking for it. DSA stuff is under Interfaces, right?

Yes, for example VLAN:s are configured as a new device under interfaces with DSA. If moving this to some "common" place (it would maybe even need a new UI) is too hard I'm fine with just adding a comment field as well.

dannil avatar Oct 07 '24 17:10 dannil

Instead of comment it would be better to use description.

stokito avatar Mar 04 '25 14:03 stokito