Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ADJUST1-588 Redesign - Select offence on Add and Edit Remand pages #299

Conversation

pavilionsahota
Copy link
Contributor

No description provided.

@pavilionsahota pavilionsahota requested a review from a team as a code owner April 12, 2024 15:48
<span class="govuk-hint">Select all that apply</span>
</p>
{% for offence in offences %}
<div class="govuk-checkboxes__item">
Copy link
Contributor

Choose a reason for hiding this comment

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

have you considered using https://design-system.service.gov.uk/components/checkboxes/ component for this? I think using the nunjucks function for the component gives then benefit if they change the component in the future you'd get the change for free upgrading, using straight html doesn't give that benefit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's not really feasible in the context of a macro because of the amount of dynamic content in the checkbox label.
Whilst I understand your concern that using the Nunjucks function offers the benefit of automatically updating the component, GDS updates are typically designed to be backwards compatible, and they take into account the needs of various governmental departments to ensure that updates don't disrupt service. Plus, their focus on accessibility means they're always aiming to enhance, not hinder, the usability of components across different devices and user needs.
This priority on stability, accessibility, and compatibility across devices makes It is highly unlikely that they'd roll out an update that completely wrecks existing HTML implementations, especially since many government apps depend on the robustness of these implementations.

Copy link
Contributor

@carloveo-moj carloveo-moj left a comment

Choose a reason for hiding this comment

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

LGTM

@carloveo-moj carloveo-moj enabled auto-merge (squash) April 15, 2024 14:06
@carloveo-moj carloveo-moj merged commit f0c0c79 into main Apr 15, 2024
4 checks passed
@carloveo-moj carloveo-moj deleted the ADJUST1-588-Redesign-Select-offence-on-Add-and-Edit-Remand-pages branch April 15, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants