-
Notifications
You must be signed in to change notification settings - Fork 327
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
Update release notes for removing deprecated $govuk-border-width-form-element-error
variable
#2380
Conversation
0c7c9bb
to
0749dde
Compare
@EoinShaughnessy I've made some changes following Ollie's comment - would you be ok to have another look at the changes? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
@vanitabarrett @36degrees Do we need to change the title of both this pull request and #1963 to include the word 'variable'? |
$govuk-border-width-form-element-error
variable
0414199
to
0ef1b73
Compare
@36degrees Are you happy to approve this PR? |
0ef1b73
to
816b32a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @vanitabarrett – looks good 👍🏻
816b32a
to
4b1451b
Compare
Expand on the release notes for pull request 1963: Remove deprecated
$govuk-border-width-form-element-error
setting so it's clearer what users need to do as a result of this change.cc @EoinShaughnessy for visibility - this has already been drafted in the v4.0.0 draft release notes document