Hi Florian,
I vote for switching columns/rows. I think a maximum of four
columns (or are you planning to extend the zones too?) results in
a better usability than an additional horizontal scrollbar. And
the other GUI-pages grow vertically too. The Network Settings
could be placed in the first row(s), just like a NIC. What about
alternating the background colour (light grey/dark gray) of the
rows like on the firewall rules and the logs pages?
Different topic, but related to yours: I'm currently running
multiple SSIDs with hostapd on my IPFire machines (one for green
and one for blue) by creating the bridges in an initscript and
editing the hostapd-config manually. Based on your new
Zoneconfig-Feature (it creates the bridges required by hostapd and
makes things a lot easier) it is possible to write a patch for the
wlanap.cgi to implement this properly. Are you or anybody else
already working on this? I would join in or start it ...
Hello Alexander,
thank you for your feedback!
1. Yes, applying these settings requires a restart. I also think we should inform users of that in the GUI.
2. This is actually a major issue. I thought of two possible fixes: either the columns/rows get switched so the table grows vertically; or we introduce a scrollbar. Personally I think a scrollbar would be a better idea as I plan to also introduce network settings to the zoneconf interface so the site could get rather large if a user has many NICs on their machine. Also I think the height of the site should be consistent.
What do you think?
Regards
Florian
-------- Original-Nachricht --------
An 13. Mai 2019, 23:58, Alexander Koch schrieb:
Hello Florian,
thank you for providing this new feature. I think it is a big enhancement to the user experience to bring this to the GUI. I tested the current nightly build (next/9d959ac1) and found two little issues:
- The changes will not be applied until I reboot the machine. Is this the expected behaviour? If yes: I think there should be a hint on the page to inform the user about it and the file /var/run/need_reboot should be created to activate the corresponding banner "An update requires a restart!" in the GUI. Maybe the translations should be updated too. E.g. "An update or config change requires a reboot!
- If I use the GUI-Theme "ipfire-legacy" and the system has more than three NICSs, the GUI-Content will overlap the sidemenu. I attached a screenshot with this.
Regards, Alex
From: Florian Bührle [mailto:erdlof@protonmail.com]Sent: Saturday, 11 May 2019, 15:57 CESTSubject: [FEATURE] Zone configuration
Hey all,
I made a new web interface that allows users to assign network interfaces to zones. Check out the wiki article https://wiki.ipfire.org/configuration/network/zoneconf.
This feature is already part of the "next" branch of the IPFire 2.x main tree. Please review it and check for bugs :)
Regards
Flo