mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-04-04 09:31:11 -04:00
Merge pull request #3694 from alphagov/remove-issue-from-accessibility-statement
Remove issue from accessibility statement
This commit is contained in:
@@ -38,7 +38,6 @@
|
||||
We know some parts of this website are not fully accessible:
|
||||
</p>
|
||||
<ul class="govuk-list govuk-list--bullet">
|
||||
<li>low vision users cannot access the table we use to show CSV data when viewed at high zoom</li>
|
||||
<li>screen reader users may find the process of moving templates and folders confusing</li>
|
||||
<li>one page links to a PDF document that is not fully accessible</li>
|
||||
<li>the Notify status page has several accessibility issues</li>
|
||||
@@ -96,10 +95,6 @@
|
||||
|
||||
<h3 class="heading-small">Non compliance with the accessibility regulations</h3>
|
||||
|
||||
<p class="govuk-body">
|
||||
Low vision users cannot access the table we use to show CSV data when viewed at high zoom. This fails <a class="govuk-link" href="https://www.w3.org/TR/WCAG21/#reflow">WCAG 1.4.10 success criteria (Reflow)</a>. We plan to fix this by the end of 2020.
|
||||
</p>
|
||||
|
||||
<p class="govuk-body">
|
||||
Screen reader users may find the process of moving templates and folders confusing. This fails <a class="govuk-link" href="https://www.w3.org/TR/WCAG21/#name-role-value">WCAG 4.1.2 success criteria (Name, role, value)</a>. We plan to fix this by November 2020.
|
||||
</p>
|
||||
@@ -168,6 +163,6 @@
|
||||
</p>
|
||||
|
||||
<p class="govuk-body">
|
||||
This statement was prepared on 23 September 2020. It was last reviewed on 9 October 2020.
|
||||
This statement was prepared on 23 September 2020. It was last reviewed on 22 October 2020.
|
||||
</p>
|
||||
{% endblock %}
|
||||
|
||||
24
tests/app/test_accessibility_statement.py
Normal file
24
tests/app/test_accessibility_statement.py
Normal file
@@ -0,0 +1,24 @@
|
||||
import re
|
||||
import subprocess
|
||||
from datetime import datetime
|
||||
|
||||
|
||||
def test_last_review_date():
|
||||
statement_file_path = "app/templates/views/accessibility_statement.html"
|
||||
|
||||
# test local changes against master for a full diff of what will be merged
|
||||
statement_diff = subprocess.run([f"git diff --exit-code origin/master -- {statement_file_path}"],
|
||||
stdout=subprocess.PIPE, shell=True)
|
||||
|
||||
# if statement has changed, test the review date was part of those changes
|
||||
if statement_diff.returncode == 1:
|
||||
raw_diff = statement_diff.stdout.decode('utf-8')
|
||||
today = datetime.now().strftime('%d %B %Y')
|
||||
with open(statement_file_path, 'r') as statement_file:
|
||||
current_review_date = re.search((r'This statement was prepared on 23 September 2020\. '
|
||||
r'It was last reviewed on (\d{1,2} [A-Z]{1}[a-z]+ \d{4})'),
|
||||
statement_file.read()).group(1)
|
||||
|
||||
# guard against changes that don't need to update the review date
|
||||
if current_review_date != today:
|
||||
assert 'This statement was prepared on 23 September 2020. It was last reviewed on' in raw_diff
|
||||
Reference in New Issue
Block a user