Skip to content

Commit

Permalink
Added skip report tools on Around page
Browse files Browse the repository at this point in the history
This commit should minimise the confusion for assistive device users. Currently, the way components are presented and the focus order is not linear; after they have travelled through the map, they are presented with the 'Can't use the map?' link and then go directly to the #key-tools component sitting at the bottom of the page, skipping the filters and report list. To respect the page sequence, the user should go directly to the report filter and then to the report list. The downside of this approach is that users who want to use the #key-tools would have to jump through a whole list of reports before being able to get to the #key-tools. The solution in this commit is to add a link right after the 'Can't use the map?' that gives the user some context about the mismatch between the reading sequence and also the opportunity to skip the #key-tools in case they just want to jump the next element which would be the report filters.
  • Loading branch information
lucascumsille committed Feb 5, 2025
1 parent ff65763 commit eb3bd29
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions templates/web/base/around/_skip_key_tools.html
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
<a href="#report-list-filters" class="skiplink" aria-label="Skip report tools at the bottom of the page and go to report filters">Skip report tools</a>
1 change: 1 addition & 0 deletions templates/web/base/around/_updates.html
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
[% INCLUDE 'around/_skip_key_tools.html' %]
<div class="shadow-wrap">
<ul id="key-tools">
<li>
Expand Down
2 changes: 1 addition & 1 deletion templates/web/base/reports/_list-filters.html
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
[% END %]
[% END %]

<div class="report-list-filters-wrapper govuk-fieldset-wrapper govuk-small">
<div id="report-list-filters" class="report-list-filters-wrapper govuk-fieldset-wrapper govuk-small">

[% IF use_form_wrapper %]
<form method="get" action="">
Expand Down

0 comments on commit eb3bd29

Please sign in to comment.