]> git.proxmox.com Git - pve-manager-legacy.git/commit
ui: firewall grid: try balance between flex and good overview
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Mon, 22 Feb 2021 13:30:53 +0000 (14:30 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Mon, 22 Feb 2021 13:36:33 +0000 (14:36 +0100)
commita6101d1f55a7b296e10b2666a76739f146051e55
tree7fd1ec95a82914392044744b1c78dd857aadd713
parentba3e230317ba129c16ca358e0c80e43a0cff618f
ui: firewall grid: try balance between flex and good overview

Recently, with the Proxmox Backup sync/verify/.. job grids I had to
change my mind regarding use of flex, overuse of it is as bad as
having all to fixed widths as it spreads out the data a lot on big
screens, making it hard to follow as lots of eye movement is
required.

What we ideally would have is a "maxFlexWidth", i.e., a maximal
auto-flex width which still allows wider columns if a user manually
drags it wider. "maxWidth" comes close, but is a hard limit and so
use it only in cases where its quite clear that wider columns make no
sense (surely someone will disagree even for those, but well...).

Basic solution is to add lots more flex ratio to comment so that the
others won't get to big fast, keeping the actual relevant config of a
rule somewhat close by (thus easier to read/grasp/parse)

Also move the source/destination port columns after their direction
column and abbreviate their header title.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
www/manager6/grid/FirewallRules.js