From mboxrd@z Thu Jan  1 00:00:00 1970
From: Leo Hofmann <hofmann@leo-andres.de>
To: development@lists.ipfire.org
Subject: Re: [V2] Backup.cgi: Rework and altering of functionalities and GUI
 for better usability
Date: Wed, 26 May 2021 17:58:06 +0200
Message-ID: <9f484979-ed0b-9c01-3b95-43b7cdc19b64@leo-andres.de>
In-Reply-To: <84fe99d7-da07-b4f2-0705-4dd5d5934645@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7083933809088066370=="
List-Id: <development.lists.ipfire.org>

--===============7083933809088066370==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hi Matthias,

You can style the table with CSS, you don't have to change anything in the CG=
I for that.
I did this e.g. in zoneconf.cgi with the "nth-child" selector and it works gr=
eat:
https://developer.mozilla.org/en-US/docs/Web/CSS/:nth-child
https://git.ipfire.org/?p=3Dipfire-2.x.git;a=3Dblob;f=3Dhtml/cgi-bin/zoneconf=
.cgi;h=3D62f18e3086cfa1c66ad3be5366083ab6d9e7ae45;hb=3Drefs/heads/master#l90
(In a perfect world, the CSS would not be mixed into the CGI, but hopefully y=
ou see how it works)

Best
Leo

Am 26.05.2021 um 17:32 schrieb Matthias Fischer:
> Hi Adolf,
>
> On 26.05.2021 13:01, Adolf Belka wrote:
>> Hi Matthias,
>>
>> On 22/05/2021 16:53, Matthias Fischer wrote:
>>> Still based on https://bugzilla.ipfire.org/show_bug.cgi?id=3D12588
>>>
>>> Hi,
>>>
>>> in the meantime I worked on this - a bit - again. Currently, this CGI is
>>> running here under Core 156 with no seen problems.
>>>
>>> I've made several changes and adjustments.
>>>
>>> First:
>>> - I kept the icons - buttons didn't look as I hoped they would. ;-)
>>>
>>> - I kept the old icon order "Download - Delete - Save".
>>>
>>> Changelog:
>>> - Cleaned up the code a bit, some if-queries were unnessecary (if
>>> ($cgiparams{'ACTION'} eq...).
>>>
>>> - Therefore, some lang strings became obsolete ('downloadaddon' and
>>> 'downloadiso'.
>>>
>>> - Separated the ISO files from the IPF files for better overview.
>>>
>>> - Added query for deleting a backup file (onclick=3D\"return confirm...).
>>>
>>> - Added 'onclick' feature =3D> deleting a file must be acknowledged.
>>>
>>> Biggest change:
>>> - Added tables - as MT mentioned.
>>>
>>> Details can be seen in the attached screenshots.
>>>
>>> Thoughts? Opinions? I'd like to get some feedback before pushing this... =
;-)
>> It looks okay to me. Alternating shades of grey for the rows as mentioned =
by Michael would also be fine.
> Nice. Thanks. But...
>
> At the moment I have no idea how to change this CGI to show these
> alternating rows. I never did this. For a start I looked into some other
> CGIs to get a grip (e.g.: dns, firewall, fwhosts, hosts, services,
> wakeonlan, wio and log.dat).
> But alternate rows seem to be a bit more complicated to me than the
> tables. Hm.
> [I'm not an experienced programmer with CGI (call me
> "scriptkiddie" ;-) ).]
>
> If everything fails, I'll have to ask for hints how to do this. I'll see
> how far I can get.
>
>> Any chance of a diff so I can try it out on a testbed system?
> No problem. Work in progress...
>
>> By the way, FYI there is another backup umbrella bug https://bugzilla.ipfi=
re.org/show_bug.cgi?id=3D11320
> I'm aware of this. Not much time these days... ;-)
>
> Best,
> Matthias

--===============7083933809088066370==--