Skip to content

HDPI-4205 Ensure Titles and H1s are correct across journey#1027

Open
ayub-macalim wants to merge 3 commits intomasterfrom
HDPI-4205-h1-consistency
Open

HDPI-4205 Ensure Titles and H1s are correct across journey#1027
ayub-macalim wants to merge 3 commits intomasterfrom
HDPI-4205-h1-consistency

Conversation

@ayub-macalim
Copy link
Copy Markdown
Contributor

@ayub-macalim ayub-macalim commented Apr 2, 2026

HDPI-4205

Change description

  • Updated pageTitle values in locale files (EN + CY) to match revised page titles from Column D of the spreadsheet, including: "Respond to a claim", "Free legal advice", "Your postal
    address", "Rental agreement", "Rental agreement start", "Disputing the claim", "Payments or agreements", "Payments made", "Agreements made", "Instalments"
    • Fixed contactPreferences H1/title capitalisation: "Contact Preferences" → "Contact preference"
    • Added separate heading keys where the H1 and page title now differ (tenancy date steps, non-rent arrears dispute, repayments agreed)
    • Updated nonRentArrearsDispute.njk to use correct {% block pageTitle %} format and render {{ heading }} as H1
    • Added {% block pageTitle %} blocks to paymentInterstitial.njk and 13 placeholder step templates
    • Removed "(placeholder)" text from H1s in placeholder step templates and corrected capitalisation to match UCD spec
    • Added isPageHeading: true to repayments-made radio field so the question renders as "h1"
    • Updated translationKeys in affected step index files to expose new heading/pageTitle keys to templates

Testing done

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@ayub-macalim ayub-macalim requested a review from a team as a code owner April 2, 2026 08:22
"includesBullet1": "{{claimantName}}s grounds for possession (their reasons for making the claim)",
"includesBullet2": "any documents theyve uploaded to support their claim",
"includesBullet3": "any other information theyve given as part of their claim",
"includesBullet1": "{{claimantName}}'s grounds for possession (their reasons for making the claim)",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the correct apostrophe? Also there are some names that shouldn't have an 's added on the end - I have feeling there is a helper in the code that will pluralise a name correctly.

{% from "macros/stepButtons.njk" import stepButtons %}
{% from "govuk/components/error-summary/macro.njk" import govukErrorSummary %}
{% from "macros/csrf.njk" import csrfProtection %}
{% block pageTitle %}Your circumstances - HM Courts & Tribunals Service – GOV.UK{% endblock %}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This first dash can be escaped as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants