Revised English Translation of the Accessibility Statement #1765
Conversation
| <p class="govuk-body">{{ websiteAvailable | safe}}</p> | ||
| <p class="govuk-body">{{ websiteRanBy }}</p> | ||
| <p class="govuk-body">{{ asManyAsPossible }}</p> | ||
| {% set paragraphs = [websiteAvailable | safe, websiteRanBy, asManyAsPossible] %} |
There was a problem hiding this comment.
what is the reason for creating array in njk file?
| {% for paragraph in paragraphs %} | ||
| <p class="govuk-body">{{ paragraph }}</p> | ||
| {% endfor %} | ||
| {% set accessibilityOptions = [asManyAsPossibleColours, asManyAsPossibleZoom, asManyAsPossibleKeyboard, asManyAsPossibleSpeech, asManyAsPossibleListen] %} |
There was a problem hiding this comment.
what is the reason for creating array in njk file?
| <p class="govuk-body">{{ somePartsNot }}</p> | ||
| {% set notAccessibleParts = [somePartsNotDescriptiveness, somePartsNotText, somePartsNotPDF, somePartsNotWelsh, somePartsNotError] %} |
There was a problem hiding this comment.
what is the reason for creating array in njk file?
| <li>{{ email | safe }}</li> | ||
| <li>{{ accessibilityPhoneNumber }}</li> | ||
| </ul> | ||
| {% set improvements = [lookingToImprove | safe, improveAccessibility | safe] %} |
There was a problem hiding this comment.
what is the reason for creating array in njk file?
There was a problem hiding this comment.
Thank you for your feedback Debolina,
In this update I have created the arrays shown in the njk file because the original template displayed multiple instances where paragraph and unordered list HTML tags were repeatedly written out with different content. Using the arrays enabled me to group the related content together that were within those tags and use the Nunjucks' {% for %} loop to iterate over each array, dynamically generate the required HTML.
This approach helps shorten the template and enables quicker content updates by simply modifying the arrays. Therefore, I was under the impression that this approach would be ideal.
However, I appreciate your insights and I am open to reverting the template to how it was originally or exploring any other suggestions you might have to improve the implementation.
There was a problem hiding this comment.
why not in content.ts?
There was a problem hiding this comment.
Given that the data on this page is static and used solely for display purposes, I found it more streamlined to reference the variables defined in the .ts file directly within the Nunjucks template also using arrays. This approach efficiently aligns the data presentation with the template’s layout without the complexities of manipulating data within the .ts file itself. I’m definitely open to suggestions if there's a more efficient method we could consider.
| <li>{{ email | safe }}</li> | ||
| <li>{{ accessibilityPhoneNumber }}</li> | ||
| </ul> | ||
| {% set improvements = [lookingToImprove | safe, improveAccessibility | safe] %} |
There was a problem hiding this comment.
why not in content.ts?
hmcts-jenkins-j-to-z
bot
added
ns:private-law
prd:prl
rel:prl-citizen-frontend-pr-1765
labels
…slations and contact details
…slations and contact details
…slations and contact details
…slations and contact details
…slations and contact details
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PRL-6915
Change description
English translation of accessibility page has been updated to align with the current requirements.
Does this PR introduce a breaking change? (check one with "x")