From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Fischer To: development@lists.ipfire.org Subject: Re: Backup: Rework and altering of functionalities and GUI for better usability Date: Sun, 18 Apr 2021 16:19:09 +0200 Message-ID: In-Reply-To: <56f8a537-1da1-6800-661d-9412d60a6b6a@ipfire.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5520999640681168316==" List-Id: --===============5520999640681168316== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On 18.04.2021 15:54, Adolf Belka wrote: > Hi Matthias, >=20 > I like the 02_like_01-with_tables.png version Yep. That's my favorite, too. "Buttons and tables" weren't so nice at all. The last version (backup.cgi.23.png) from today looks even better. IMHO. Please see my posting from today (11:17 o'clock). > The table layout works very well. Thanks. Sometimes is was bit fiddly. I can only test on 19". I'm not really sure how it looks on bigger monitors... > The dustbin icon for the delete also is very clear. These are all original icons from IPFire. I there are no further wishes or improvements I would push this in the next days. Best, Matthias > Regards, >=20 > Adolf. >=20 >=20 > Sent from my Desktop Computer >=20 > On 17/04/2021 09:38, Matthias Fischer wrote: >> Hi, >> >> On 22.03.2021 12:36, Michael Tremer wrote: >>> Hello, >>> >>>> On 21 Mar 2021, at 18:25, Matthias Fischer wrote: >>>> >>>> Based on https://bugzilla.ipfire.org/show_bug.cgi?id=3D12588 >>>> >>>> Hi, >>>> >>>> in the meantime I worked on this - a bit. >>> Very good idea. This page is quite essential, but the design is quite com= plicated and difficult to use. >> I noticed that... ;-) >> >>>> I've made several changes and adjustments which could be implemented one >>>> by one - if they are wanted/accepted: >>>> >>>> - Changed icons to buttons =3D> better readability. >> I went back to *original* IPFire images - for a short overview of the >> whole process, please take a look at the attachments. Thanks for the >> tips with 'go-bottom.png'! ;-) >> >>> Yes. I am not sure whether =E2=80=9CBackup=E2=80=9D is a good choice, bec= ause it could be interpreted as a noun. Maybe =E2=80=9CBackup Now=E2=80=9D or= =E2=80=9CUpdate Backup=E2=80=9D would be a better choice. Does anybody have = any ideas? >>> >>>> - For buttons to work, I added two lang strings for button labeling. >>> No problem here. >>> >>>> - Cleaned up the code a bit, some if-queries were unnessecary (if >>>> ($cgiparams{'ACTION'} eq...). >>>> >>>> - Changed the order of the (Addon-)buttons to "Download - Save - Delete" >>>> (old: Download was in the middle). >>> What is the rationale here? >>> >>> Why is this order better? >> As explained in another posting, this came up because other GUI pages >> use exactly this image order. But by adding tables, I didn't get this to >> work as I wanted. So I kept to the old order. Still works. ;-) >> >>>> - Marked IPF-files bold =3D> better readability (I already sent this in >>>> one day: =3D> https://patchwork.ipfire.org/patch/3124/). >> Obsolete. If someone wants it, it can be easily added again. >> >>> Can we not have a proper table here? Like this: >>> >>> | Package | Last Backup | Size | | >>> | wio | 2021-03-22 11:31 | 8 kB | [Download] [Backup] [Delete] | >> At your command... Done. ;-)) >> >> Its just the old order: [Download] [Delete] [Backup] >> >>>> - Added query for deleting a backup file (onclick=3D\"return confirm...). >>> Very good idea, too. >> Implemented. >> >>> Can we make this a function that we can add to various other places, too?= Like firewall rules? >> Sorry - I took a look, but don't know how to do this (the best way). No >> idea. >> >>>> Details in the attached screenshot. >>>> >>>> Proposed ToDo - as a start: Alter the info text for restoring to make >>>> clear that only the addon *configs* are restored, not the addons themsel= ves. >>> Yes. >>> >>>> Thoughts? Opinions? >>> Very good ideas. Please keep them all in individual patches so that they = can be reviewed one by one :) >> For a laugh: I wrote so much versions in the last days that somewhere in >> between I didn't know which 'backup.cgi' was which. Sometimes its >> helpful to choose meaningful file names... ;-) >> >> The attached screenshots show some states of the development process and >> the most striking changes. Some I could not split into individual patches. >> >> Today I came to the conclusion to send these shots to the list so >> everyone can take a look. I personally think that (perhaps?) it looks >> somehow better now. But before going on, I would like to get some feedback. >> >> Thoughts? >> >> Best, >> Matthias >> >> P.S.: Could "someone" please tell me how we could use the following >> "sub" ('backup.cgi', Core 155, line 107)!? >> >> ... >> sub refreshpage{&Header::openbox( 'Waiting', 1, "> http-equiv=3D'refresh' content=3D'1;'>" );print "
> src=3D'/images/clock.gif' alt=3D'' />
> color=3D'red'>$Lang::tr{'pagerefresh'}
";&Header::closebox(= );} >> ... >> >> Exactly the same sub is used in 'pakfire.cgi' (line 47 and line 207), >> but I've absolutely no idea how and where (which line in 'backup.cgi'!?) >> we could implement this sub in 'backup.cgi'. >> Otherwise its - perhaps - obsolete!? >=20 --===============5520999640681168316==--