<aside>-tag in a template causes conflict in testing-tool for accessibility

[grashalm4u]

Hello Bugbuster,

I am currently using Contao 4.13.1 with PHP 8.0.16., and I have integrated your extension for banner graphics in my footer at https://die-schreibmaus.de

I've checked my site for accessibility several times over the past few days using the Rocket Validator tool https://rocketvalidator.com. Among other things, I get the following error message, which is related to the banner module:
"Aside should not be contained in another landmark".

After some research I found out that (at least) the template I use "mod_banner_list_all.html5" has this "aside" tag included at the beginning and at the end. According to the Rocket Validator error message, "aside" must stand on its own and not be embedded in other landmarks.

Since your banner module is integrated in my footer, I almost always have the footer landmark as the upper level, so I can hardly use "aside" here. I just replaced the "aside" tag with the "section" tag and found that the error no longer appears. Being a complete layman, I don't know if using "section" is appropriate and correct. Perhaps there is a better tag for it. But the error message is gone now.

Maybe you can check if it makes sense to replace the "aside" tag in the banner module templates with another tag that doesn't raise this conflict when testing the site for accessibility.

Greetings, grashalm

[BugBuster1701]

https://community.contao.org/de/showthread.php?82699-Test-auf-Barrierefreiheit-zeigt-jede-Menge-Fehler&p=557214&viewfull=1#post557214

[BugBuster1701]

vermutlich macht es tatsächlich Sinn, das <aside>-Attribut in Deinen Banner-Templates um eine Role, ein Label, oder einen einmaligen Title zu ergänzen. Dann bräuchte ich beim nächsten Update des Banner-Moduls dieses Template nicht mehr von Hand anzupassen.

[BugBuster1701]

$this->Template->banners = $arrResults;
$this->Template->bmid = "bmid".$module_id;

Idee Modul ID, ist aber noch nicht überall vorhanden in den Methoden.
Modul ID deshalb, falls jemand mehrere Banner Module im FE hat, damit die eindeutig wird.

[BugBuster1701]

In Version 1.3.3 eingebaut.

[BugBuster1701]

Nächster Versuch, title kommt nun mit. Noch als master, demnächst kommt neue Version.

[BugBuster1701]

in Version 1.3.4 eingebaut.

[grashalm4u]

Wahrscheinlich reicht es aus, das aside-tag einfach durch ein div-tag im template zu ersetzen. s.a die letzten beiden posts in unserem thread: https://community.contao.org/de/showthread.php?82699-Test-auf-Barrierefreiheit-zeigt-jede-Menge-Fehler&p=559427#post559427

[grashalm4u]

Hallo Bugbuster,

ich habe mir heute noch einmal einen kostenpflichtigen Testzeitraum mit der Pro-Version des RocketValidators gegönnt und meine Seite noch einmal auf Barrierefreiheit getestet. Leider erscheint die Fehlermeldung zum "aside"-Tag immer noch. Sei doch so gut, und ersetzte mal das "aside" durch ein "div", und teste dann, ob alle Funktionalitäten deiner Templates weiterhin laufen.
Nachdem ich das Template "mod_banner_list_all.html5" in den Templates-Ordner kopiert, und "aside" durch "div" ersetzt habe, taucht der besagte Fehler im RocketValidator nicht mehr auf. Das könnte die Lösung des Problems sein, wie ich ja im vorigen Post schon beschrieben hatte. Ich würde mich freuen, wenn ich diese Fehlermeldung mit der nächsten Version Deines Banner-Moduls ad acta legen könnte.

Vielen Dank und herzliche Grüße,

grashalm4u