decidim icon indicating copy to clipboard operation
decidim copied to clipboard

Fix inappropriate roles menu and menuitem in footer links

Open Stef-Rousset opened this issue 10 months ago • 9 comments

:tophat: What? Why?

This PR removes inappropriate roles menu and menuitem in footer. It is issued from the audit of Angers city (page 33), and corresponds to criterias 2.5.3 and 4.1.2 from WCAG.

Testing

As a user, go to the dev tools and check that roles menu and menuitem are not present in footer links.

:camera: Screenshots

456488617-c09273a4-2fed-4508-8af5-31f35876498a

:hearts: Thank you!

Stef-Rousset avatar Jun 19 '25 12:06 Stef-Rousset

@Stef-Rousset Can you fix this pipeline ?

alecslupu avatar Jul 05 '25 06:07 alecslupu

@Stef-Rousset Can you fix this pipeline ?

hello @alecslupu , I will take a look this week !

Stef-Rousset avatar Jul 08 '25 13:07 Stef-Rousset

Hello @alecslupu , I made an update to fix the failing accessibility tests on account page. Please let me know if it's ok !

Stef-Rousset avatar Jul 09 '25 08:07 Stef-Rousset

@Stef-Rousset Any updates on this matter ? ( the same described in #14951)

alecslupu avatar Jul 22 '25 11:07 alecslupu

Hello @alecslupu , I have added the js again for account page, everything seems ok now !

Stef-Rousset avatar Jul 28 '25 10:07 Stef-Rousset

@Stef-Rousset I have checked again this, and the previous error i have mentioned (WGAC error ) can be fixed if i add a timeout on axe script.

However, if i apply this fix, the user menu will not be collapsed.

alecslupu avatar Aug 28 '25 14:08 alecslupu

@Stef-Rousset I have checked again this, and the previous error i have mentioned (WGAC error ) can be fixed if i add a timeout on axe script.

However, if i apply this fix, the user menu will not be collapsed.

Hello @alecslupu , I removed the js added, so that you can apply your fix ! I have also fixed 2 failing tests. However, some tests seems to be unstable, as you can see on the img, the failing tests before I triggered the CI are not the same as the ones that fails now...

Capture d’écran 2025-09-02 à 08 59 06

Stef-Rousset avatar Sep 02 '25 08:09 Stef-Rousset

Having 0.31 releases, we do not longer release 0.29.x series as per policy. Removing release: v0.29

alecslupu avatar Nov 20 '25 10:11 alecslupu

Hey @Stef-Rousset is this still being worked on? I've noticed there are conflicts and its currently 190 commits behind the develop branch.

Do you have time to take a look into this?

greenwoodt avatar Dec 09 '25 08:12 greenwoodt