Skip to content

Commit

Permalink
Fix/update terms headings (#1874)
Browse files Browse the repository at this point in the history
* fix: change the headings to h2's so there is no heading jump

* fix: move the `/terms` page from the GCA tests to the app pages tests
  • Loading branch information
andrewleith committed Jun 20, 2024
1 parent 7f61fdf commit 5481402
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 10 deletions.
18 changes: 9 additions & 9 deletions app/templates/components/terms.html
Original file line number Diff line number Diff line change
Expand Up @@ -25,22 +25,22 @@
{% endmacro %}

{% macro terms_of_use() %}
<h3 class="heading-medium">{{ _('Your responsibilities') }}</h3>
<h2 class="heading-medium">{{ _('Your responsibilities') }}</h2>
<p>
<em>For CDS responsibilities, read GC Notify’s </em
><a href="https://notification.canada.ca/service-level-agreement"
><em>service level agreement</em></a
><em>.</em>
</p>

<h3 class="heading-medium">{{ _(headings[0].title.format(headings[0].link)) }}</h3>
<h2 class="heading-medium">{{ _(headings[0].title.format(headings[0].link)) }}</h2>
<p>
We may suspend your account or service if you break these terms or misuse GC
Notify. Misuse includes creating duplicate services or services incompatible
with your original purpose.
</p>

<h3 class="heading-medium">{{ _(headings[1].title) }}</h3>
<h2 class="heading-medium">{{ _(headings[1].title) }}</h2>
<p>
We apply limits to maintain our
<a href="https://notification.canada.ca/service-level-objectives"
Expand Down Expand Up @@ -75,7 +75,7 @@ <h3 class="heading-medium">{{ _(headings[1].title) }}</h3>
</div>
</div>

<h3 class="heading-medium">{{ _(headings[2].title) }}</h3>
<h2 class="heading-medium">{{ _(headings[2].title) }}</h2>
<p>Texts are:</p>
<ul class="list list-bullet ml-10">
<li>
Expand All @@ -94,7 +94,7 @@ <h3 class="heading-medium">{{ _(headings[2].title) }}</h3>
</li>
</ul>

<h3 class="heading-medium">{{ _(headings[3].title) }}</h3>
<h2 class="heading-medium">{{ _(headings[3].title) }}</h2>
<p>
GC Notify holds information temporarily but your organization is responsible
for that information.&nbsp;
Expand Down Expand Up @@ -126,7 +126,7 @@ <h3 class="heading-medium">{{ _(headings[3].title) }}</h3>
</li>
</ul>

<h3 class="heading-medium">{{ _(headings[4].title) }}</h3>
<h2 class="heading-medium">{{ _(headings[4].title) }}</h2>
<ul class="list list-bullet ml-10">
<li>
Follow your organization’s policies and check messages are:<br />(a) Within
Expand All @@ -146,7 +146,7 @@ <h3 class="heading-medium">{{ _(headings[4].title) }}</h3>
</li>
</ul>

<h3 class="heading-medium">{{ _(headings[5].title) }}</h3>
<h2 class="heading-medium">{{ _(headings[5].title) }}</h2>
<ul class="list list-bullet ml-10">
<li>
Use<strong> </strong>accurate, in-use email addresses. We may suspend your
Expand All @@ -156,7 +156,7 @@ <h3 class="heading-medium">{{ _(headings[5].title) }}</h3>
<li>Use accurate, in-use mobile phone numbers. Do not send to landlines.</li>
</ul>

<h3 class="heading-medium">{{ _(headings[6].title) }}</h3>
<h2 class="heading-medium">{{ _(headings[6].title) }}</h2>
<ul class="list list-bullet ml-10">
<li>
Assess context to decide what
Expand All @@ -176,7 +176,7 @@ <h3 class="heading-medium">{{ _(headings[6].title) }}</h3>
</li>
</ul>

<h3 class="heading-medium">{{ _(headings[7].title) }}</h3>
<h2 class="heading-medium">{{ _(headings[7].title) }}</h2>
<ul class="list list-bullet ml-10">
<li>
Use a valid government email and a mobile device with a Canadian phone
Expand Down
1 change: 1 addition & 0 deletions tests_cypress/cypress/e2e/admin/a11y/app_pages.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ const pages = [
{ name: "Contact us", route: "/contact" },
{ name: "Create an account", route: "/register" },
{ name: "Sign in", route: "/sign-in" },
{ name: "Terms of use", route: "/terms" },
];

describe(`A11Y - App pages [${config.CONFIG_NAME}]`, () => {
Expand Down
1 change: 0 additions & 1 deletion tests_cypress/cypress/e2e/admin/a11y/gca_pages.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ const fullPageList = [
{ en: "/features", fr: "/fonctionnalites" },
{ en: "/formatting-emails", fr: "/guide-mise-en-forme" },
{ en: "/service-level-agreement", fr: "/accord-niveaux-de-service" },
{ en: "/terms", fr: "/conditions-dutilisation" },
{ en: "/guidance", fr: "/guides-reference" },
{ en: "/home", fr: "/accueil" },
{ en: "/message-delivery-status", fr: "/etat-livraison-messages" },
Expand Down

0 comments on commit 5481402

Please sign in to comment.