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

Update v3.9.0 (unreleased) changelog for style #1948

Merged
merged 3 commits into from
Sep 10, 2020

Conversation

m-green
Copy link
Contributor

@m-green m-green commented Sep 8, 2020

This updates the v3.9.0 (unreleased) changelog for style and readability.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1948 September 8, 2020 13:48 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1948 September 8, 2020 13:49 Inactive
menuButtonLabel: "Custom menu label"
}) }}
```
### Fixes
Copy link
Contributor

Choose a reason for hiding this comment

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

This section appears to be duplicated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think (?) I just accidentally fixed this when I rebased... let me know if not though!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah wait no sorry, it's still there... I'll fix

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be fixed now - thanks for spotting!

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1948 September 8, 2020 13:55 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1948 September 8, 2020 13:58 Inactive
@m-green m-green linked an issue Sep 8, 2020 that may be closed by this pull request
3 tasks
Copy link
Member

@hannalaakso hannalaakso left a comment

Choose a reason for hiding this comment

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

Nice tidy up @m-green 👍 Left a couple of small comments.

CHANGELOG.md Outdated
You can now add a prefix or suffix element to the text input to help users enter things like currencies and measurements.
You can now [use prefixes and suffixes in the text input component](https://design-system.service.gov.uk/components/text-input/#prefixes-and-suffixes) to help users enter things like currencies and measurements.

This was added in [pull request #1269: Add input prefix and suffix](https://github.com/alphagov/govuk-design-system/pull/1269/files). Thanks to [@simonwhatley](https://github.com/simonwhatley) and the GOV.UK Coronavirus Services Team.
Copy link
Member

Choose a reason for hiding this comment

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

The PR should be #1816.

CHANGELOG.md Outdated

This was added in [pull request #1918: Add new brand colour for FCDO](https://github.com/alphagov/govuk-frontend/pull/1918).
- [#1905: Set navigation and mobile menu labels of the header component with new options](https://github.com/alphagov/govuk-frontend/pull/1905)
- [#1943: Change header menu button label](https://github.com/alphagov/govuk-frontend/pull/1943)
Copy link
Member

Choose a reason for hiding this comment

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

This could be

CHANGELOG.md Outdated

#### Set navigation and mobile menu labels of the header with new options
#### Test if your HTML matches GOV.UK Frontend
Copy link
Member

Choose a reason for hiding this comment

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

This is just a suggestion but I would be tempted to position this before the header features as it feels like the more important feature.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1948 September 8, 2020 14:46 Inactive
@m-green
Copy link
Contributor Author

m-green commented Sep 10, 2020

@hannalaakso This has been 2i'd now - no changes.

@hannalaakso hannalaakso merged commit 56edb17 into master Sep 10, 2020
@hannalaakso hannalaakso deleted the edit-3.9.0-release-note branch September 10, 2020 16:25
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.

Finalize release notes for v3.9.0
4 participants