search

localgovdrupal / localgov_subsites

Replaces LocalGov Campaigns with a new name and extended functionality to support subsites within LocalGov Drupal installations.
GNU General Public License v2.0
4 stars 2 forks source link

59 current page should be text #69

Closed cjstevens78 closed 2 years ago

cjstevens78 commented 2 years ago

This fixes the issue raised here: https://github.com/localgovdrupal/localgov_subsites/issues/59

finnlewis commented 2 years ago

Just discussing this at our 'merge monday' meeting. Andy has tested the functionality which looks sensible, so he's approved it. Thanks @andybroomfield !

I'm just wondering if the motivation for this is an accessibility issue or just general best practice in user experience.

If anyone has any evidence we can point to as to why this is good, it would be lovely to document it with links to supporting evidence.

@msayoung any thoughts for the accessibility perspective?

cjstevens78 commented 2 years ago

@finnlewis I think its probably a bit of both really, its best practice from a usability perspective not to have a redundant link to the page you are currently viewing and having the 'active' link in the navigation styled differently gives an instant verification to the user which of the pages in the navigation they are viewing. From an accessibility point of view I should imagine it could be rather confusing to have the page you are on linked to because for example if a user tabs to the link and the screen reader reads out that it is a link to ‘xyz’ page and the user knows they are already on that page - it could cause confusion.