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

Upgrade to accessible-autocomplete v2.0.0 #60

Merged
merged 5 commits into from Sep 27, 2019
Merged

Upgrade to accessible-autocomplete v2.0.0 #60

merged 5 commits into from Sep 27, 2019

Conversation

colinrotherham
Copy link
Contributor

@colinrotherham colinrotherham commented Sep 27, 2019

This is a breaking change that now uses GOV.UK Design System CSS.

E.g. The focus state now meets the new WCAG 2.1 level AA requirements.

See Accessible Autocomplete v2.0.0 release notes:
https://github.com/alphagov/accessible-autocomplete/releases/tag/v2.0.0

Fixes #59 the Babel config issue by adding corejs version.

Signed-off-by: Colin Rotherham <work@colinr.com>
Signed-off-by: Colin Rotherham <work@colinr.com>
Signed-off-by: Colin Rotherham <work@colinr.com>
Not styled, but good for reference

Signed-off-by: Colin Rotherham <work@colinr.com>
Copy link
Contributor

@injms injms left a comment

Choose a reason for hiding this comment

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

🌍 🎉

Super - thanks @colinrotherham!

CHANGELOG.md Outdated
@@ -2,6 +2,12 @@

## Unreleased

## 1.0.0 - 2019-09-27
Copy link
Contributor

Choose a reason for hiding this comment

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

Only one thing - can you put this under 'Unreleased'? Then I'll a separate pull request specifically to bump the version number.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@injms Done 👍

@colinrotherham
Copy link
Contributor Author

Thanks @injms.

I've (assumed) this will need to be a breaking v1.0.0 release, but let me know if you need anything changing in the changelog etc.

Do you want me to run yarn version major and add the new commit/tag or shall I save that for you? Thanks again

Now using CSS from the GOV.UK Design System

Signed-off-by: Colin Rotherham <work@colinr.com>
Copy link
Contributor

@NickColley NickColley left a comment

Choose a reason for hiding this comment

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

Thanks a lot for helping with this Colin 👍

@injms the Design System team can release this if you are not able to or are busy.

@NickColley NickColley added this to Needs review in Design System Sprint Board via automation Sep 27, 2019
@NickColley NickColley added this to the 1.0.0 milestone Sep 27, 2019
@NickColley NickColley merged commit 7b5acee into alphagov:master Sep 27, 2019
Design System Sprint Board automation moved this from Needs review to Done Sep 27, 2019
@colinrotherham colinrotherham deleted the accessible-autocomplete-upgrade branch September 27, 2019 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Running the project locally fails due to corejs errors
3 participants