Am 5/13/22 um 09:55 schrieb Daniel Tschlatscher: > On 5/12/22 17:13, Thomas Lamprecht wrote: >> Some other points to improve: >> >> - setting the online help, maybe to 'proxmox_node_management' which would >> make >> it point to: >> >> https://pve.proxmox.com/pve-docs/chapter-sysadmin.html#proxmox_node_management >> (the chapter could maybe do good with some addition w.r.t. first start >> delay > > This sounds to me like a suggestion to set the online help for the panel, > i.e. the > button in the top right corner. But that's already included in the patch. > Or are you suggesting to put it somewhere else too?
None of the edit windows have it set. the top right corner is somewhat nice sometimes but the real useful one is the one from the edit window, which is much more visible, and especially the only one available if the window is open as the background is masked then, having to close the window with some values already half entered is the worst. > >> >> - It's not clear what unit the start delay has, you could mention that >> explicitly, >> simplest way to do so is adding a ' (s)' to the fieldLabel after the >> gettext. > > The field does not display a value while displaying Default, though it > explicitly states > 'Seconds' after any value is set. I think it would become very clear to > anybody then That's too late, I need to know what I'm entering *on entering* not sometimes later, when I already set a possible bogus value; while it hasn't that big of an affect in this specific case it should be the general approach to UX that the user knows what they enter, otherwise a UI is close to useless. > what unit this setting represents. > Still, adding ' (s)' probably wouldn't kill me either. 😉 Note that the "Default" in the grid is also wrong for unset, it should state "None". _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
