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

Fix bill total wrapping when a credit #525

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

Cruikshanks
Copy link
Member

https://eaflood.atlassian.net/browse/WATER-4131

We have found when the value of a credit gets beyond a few pounds that the current view will wrap it even though there appears to be plenty of space.

So, in this change, we tweak the GDS layout class being used to give us a little extra space. The value will still wrap if it's really large. But in most cases now it should be on one line.

Before

Screenshot 2023-11-14 at 23 37 36

After

Screenshot 2023-11-14 at 23 37 58

https://eaflood.atlassian.net/browse/WATER-4131

We have found when the value of a credit gets beyond a few pounds that the current view will wrap it even though there appears to be plenty of space.

So, in this change we tweak the [GDS layout class](https://design-system.service.gov.uk/styles/layout/#common-layouts) being used to give us a little extra space. The value will still wrap if it's _really_ large. But in most cases now it should be on one line.
@Cruikshanks Cruikshanks added the bug Something isn't working label Nov 14, 2023
@Cruikshanks Cruikshanks self-assigned this Nov 14, 2023
@Cruikshanks Cruikshanks marked this pull request as ready for review November 14, 2023 23:49
@Cruikshanks Cruikshanks merged commit 58fb4d4 into main Nov 14, 2023
6 checks passed
@Cruikshanks Cruikshanks deleted the fix-display-of-bill-total-for-credits branch November 14, 2023 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant