All pages use role-attributes with invalid values #99

Closed
opened 2023-07-11 16:41:05 +02:00 by Moiety · 4 comments
Moiety commented 2023-07-11 16:41:05 +02:00 (Migrated from koodu.ubiqueros.com)

Pages: all pages tested (including: login, dashboard, create/edit page)

The role-attribute is used to set specific accessibility semantics; there’s only a limited set of valid values that apply to specific components. By the looks of it Fipamo uses the role-attribute purely for styling. Can you elaborate on why you chose this attribute?

One way to resolve this is to replace the role-attributes with classes.

As an example, from _frame.twig:

<nav role="top-nav">
	<div role="nav-left">
		…
	</div>
	<div role="title">
		…
	</div>
	<div role="nav-right">
		…
	</div>
</nav>
<div role="notify">
	…
</div>

None of the values provided here are valid; this causes accessibility issues.


Ro Edit
The following pages need to be updated to remove the invalid role attributes and make the appropriate adjustments to the scripting to longer reference these attributes.

  • Edit Login
  • Edit Init Forms (restore backup, password recovery)
  • Edit Index
  • Edit Page Listing
  • Edit Page
  • Edit Settings
  • Edit Navigation
Pages: all pages tested (including: login, dashboard, create/edit page) The `role`-attribute is used to set specific accessibility semantics; there’s only a limited set of [valid values](https://www.w3.org/TR/wai-aria-1.1/#roles_categorization) that apply to specific components. By the looks of it Fipamo uses the `role`-attribute purely for styling. Can you elaborate on why you chose this attribute? One way to resolve this is to replace the `role`-attributes with classes. As an example, from [_frame.twig](https://koodu.ubiqueros.com/are0h/Fipamo/src/commit/2f0b83b492300ebd9f98ddcc031298b64af1ff0a/brain/views/dash/_frame.twig#L18): ``` <nav role="top-nav"> <div role="nav-left"> … </div> <div role="title"> … </div> <div role="nav-right"> … </div> </nav> <div role="notify"> … </div> ``` None of the values provided here are valid; this causes accessibility issues. ____ *Ro Edit* The following pages need to be updated to remove the invalid role attributes and make the appropriate adjustments to the scripting to longer reference these attributes. - [x] Edit Login - [x] Edit Init Forms (restore backup, password recovery) - [x] Edit Index - [x] Edit Page Listing - [x] Edit Page - [x] Edit Settings - [x] Edit Navigation
are0h commented 2023-07-11 23:41:03 +02:00 (Migrated from koodu.ubiqueros.com)

The intention was to use role as a CSS anchor and describe what the block was being used for, so I got a bit carried away with it.

The HTML already relies on semantic HTML, so I can edit these files to remove most of the role tags and change the ones I do keep with valid values.

The intention was to use role as a CSS anchor and describe what the block was being used for, so I got a bit carried away with it. The HTML already relies on semantic HTML, so I can edit these files to remove most of the role tags and change the ones I do keep with valid values.
are0h commented 2023-07-27 22:59:57 +02:00 (Migrated from koodu.ubiqueros.com)

I'm going to start with this one because it's the most complex.

I have to edit every page style along with the JS that is referencing the role attribute specifically.

This will touch just about every system on the site, so I'm going to be methodical about getting this cleaned up so I don't send the codebase into a tailspin.

I'm going to start with this one because it's the most complex. I have to edit every page style along with the JS that is referencing the `role` attribute specifically. This will touch just about every system on the site, so I'm going to be methodical about getting this cleaned up so I don't send the codebase into a tailspin.
Moiety commented 2023-07-28 08:24:32 +02:00 (Migrated from koodu.ubiqueros.com)

Let me know if you want some help with this. We could do some pair programming if you want.

Let me know if you want some help with this. We could do some pair programming if you want.
ro added this to the Road to Version One project 2024-05-07 22:36:21 +02:00

All invalid role tags have been removed.

All invalid role tags have been removed.
ro self-assigned this 2024-06-05 21:39:31 +02:00
ro closed this issue 2024-11-11 20:41:38 +01:00
Sign in to join this conversation.
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: projects/fipamo#99
No description provided.