Matthias,
I can’t comment on everything, but I do know that I have consistently been confused by the icons on the backup page.
Confirmation dialogs seem good, so long as we don’t go overboard and ask “are you sure” for every tiny action.
Tom
On Nov 24, 2018, at 4:39 PM, Matthias Fischer matthias.fischer@ipfire.org wrote:
Hi,
Something similar has been posted before ([PATCH] Cosmetics: New icons for backup.cgi), but unfortunately the discussion was not continued for very long.
Triggered by:
https://forum.ipfire.org/viewtopic.php?f=17&t=21372
and
https://forum.ipfire.org/viewtopic.php?f=17&t=21882
I'd like to know whether there is any interest in developing this any further. In the meantime, patches were built for 'backup.cgi' which not only update the icons and their order of precedence for better readability and usability. In an additional step, the 'delete' icons got 'onclick'-confirmation boxes (by Erik!) to prevent accidental deletions.
Another step could be to add this confirmation to other CGIs. This additional query could prevent some trouble in the future as it happened to the original poster of the second link above. He accidentally deleted an OVPN client. No query... I'm already testing this functionality in other contexts (e.g., firewall rules, dhcp, hosts) and it works.
And the last question: each current lang file contains up to ~600 unused translation strings. These are summing up to about 5000 strings which are not used or duplicates and could be deleted. Current size of all lang files is 1.2MB - we actually need about 900KB. A current patch can be found here:
https://git.ipfire.org/?p=people/ummeegge/ipfire-2.x.git;a=commit;h=acbf3208...
At the beginning of this year I built some patches for these unused strings and some other corrections (https://patchwork.ipfire.org/patch/1605/) but sadly got no reaction at all. Is there any interest, should we re-edit the unused translations?
What do you think? Opinions and comments are welcome...
Best, Matthias