#21628 is closed as requested.

Looks good - can you fix hound please

Additionally we have a merge conflict in this files:

administrator/templates/atum/css/template-rtl.min.css
administrator/templates/atum/css/template.min.css 

@wojsmol can you help me? I don't know how to resolve conflicts in .min.css files.

@chmst see chmst#1

@chmst I send PR reverting my last changes.

All right. I don't understand the conflicts, because the .css and min.css files should be generated from the scss files.

accept my last PR then I prepare good one.

It is merged. Is there something else I have to do?

chmst#3

@chmst All conflicting css files can be deleted from this PR because they are added to .gitignore in the current 4.0-dev. Additionally please delete following files and folders

administrator/templates/atum/html/com_patchtester_meins
nbproject
administrator/components/com_media/package-lock.json

cannot find dthe package-lock.json any more.

cannot find dthe package-lock.json any more.

Ok, then we have following conflicting files list

administrator/components/com_admin/tmpl/sysinfo/default.php
administrator/templates/atum/css/bootstrap.css
administrator/templates/atum/css/bootstrap.min.css
administrator/templates/atum/css/template-rtl.css
administrator/templates/atum/css/template-rtl.min.css
administrator/templates/atum/css/template.css
administrator/templates/atum/css/template.min.css
installation/template/css/template.css
installation/template/css/template.min.css

see my previous comment #21724 (comment)

@chmst If you need any help then simply write

I'm trying right now, but cannot reproduce the branch. I had done the npm for testing other pr's.

@wojsmol I cannot reproduce the conflicts in my repo. The simple way sowud be to close this pr and make a new one?

Simply merge 4.0-dev from joomla/joomla-cms into this PR branch. Then you revise conflicts.

@chmst Please see chmst#4

@chmst I must miss same changes in administrator/components/com_admin/tmpl/sysinfo/default.php. Please compare with https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/components/com_admin/tmpl/sysinfo/default.php

@wojsmol - Thank you, I will resolve this when I am back home. The < form > tag is not needed.

@wojsmol hope that the conflict is resolved now. Thank you for your patience

I have tested this item ✅ successfully on cd203ad

I have tested this item ✅ successfully on cd203ad

@chmst Please remove administrator/components/com_media/package-lock.json from this PR.

@brianteeman all ok here from your side?

@laoneo it was good when I first checked it - not sure if there have been significant changes since

Do you mind to have another look?

Wait a moment. There is another PR which has changed something in the library. I can resolve this as soon as the other change is merged.

@laoneo I will put it on my todo for tonight

I have tested this item ✅ successfully on cf368e5

I have tested this change in Joomla 4 Alpha 5 release succesfully. There are no visible differences in the View System Information before and after patch.

I have also tested with Wave Automatic Test and I got the error of an empty th which belongs to the php info about mysqlnd statistics which was empty in my server but I do not think it's related with this PR.

@carcam thank you for testing. And good finding of the empty in the php-info. Let's make another PR for the sysinfo model, to avoid empty table headers.

RTC

I tested this patch. Fieldset and legend elements have been rightly removed. But...
The "caption" element should be used in the table instead of the legend element. Without the caption element, users of screen readers will not get the information contained in the table.

I'm going to merge this as a good piece of progress even though it would appear there's some more to do here from the comments by @zwiastunsw

@wilsonge thank you! So I can continue without raising conflicts ;)