Skip to content

Conversation

@Themitchell
Copy link
Contributor

What is the change?

Align the height page following UCD review

  • Fix incorrect styling for headers and labels
  • Add suffix labels to fields
  • Reword switch to imperial / metric to swith to centimeters / feet and inches
  • Add 1 decimal place validation errors
  • handle no value for feet / inches

Why are we making this change?

Designs have moved on since initial development, this brings the height page inline with the designs

<input class="nhsuk-input" id="id_field" name="field" type="number" value="1">
<div class="nhsuk-input__suffix" aria-hidden="true">cm</div>
</div>
</>
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't seem to be correct HTML syntax.

@Themitchell Themitchell force-pushed the PPHA-262-height-page-updates branch from d233d67 to a4fa254 Compare November 26, 2025 11:24
@sonarqubecloud
Copy link

@Themitchell Themitchell merged commit aff9f32 into main Nov 26, 2025
23 checks passed
@Themitchell Themitchell deleted the PPHA-262-height-page-updates branch November 26, 2025 11:31
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.

3 participants