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

Remove accessible autocomplete component #1038

Merged
merged 2 commits into from
Aug 9, 2019

Conversation

alex-ju
Copy link
Contributor

@alex-ju alex-ju commented Aug 9, 2019

What

Remove the accessible-autocomplete component

Why

The accessible-autocomplete component was introduced in #621, was tested with users in finders-frontend and was replaced with an enhanced version of the option-select component.

The only application where this component is being used is content-data-admin, and we plan to move the component there.

The main reason for removing it:

  • it's only being used by one application and we don't foresee it being used in other apps in the near future
  • it requires a quite large chunk of JavaScript, that is currently being served to all users of frontend applications without using it. This affects performance as you can imagine.

Visual Changes

No visual changes

View Changes

https://govuk-publishing-compo-pr-1038.herokuapp.com/

@bevanloon bevanloon temporarily deployed to govuk-publishing-compo-pr-1038 August 9, 2019 11:15 Inactive
@alex-ju alex-ju force-pushed the remove-accessible-autocomplete-component branch from f120038 to 6380a49 Compare August 9, 2019 11:16
@alex-ju alex-ju merged commit d2b3ee5 into master Aug 9, 2019
@alex-ju alex-ju deleted the remove-accessible-autocomplete-component branch August 9, 2019 11:26
andysellick added a commit that referenced this pull request Nov 26, 2021
- this is the code for the previous version of the component, removed in #1038
- does not use the git branch or code for multiple selections, as this is not needed
- otherwise unchanged and apparently functional, but some functionality here is not required and tests are failing
andysellick added a commit that referenced this pull request Nov 26, 2021
- this is the code for the previous version of the component, removed in #1038
- does not use the git branch or code for multiple selections, as this is not needed
- otherwise unchanged and apparently functional, but some functionality here is not required and tests are failing
andysellick added a commit that referenced this pull request Nov 29, 2021
- this is the code for the previous version of the component, removed in #1038
- does not use the git branch or code for multiple selections, as this is not needed
- otherwise unchanged and apparently functional, but some functionality here is not required and tests are failing
MuriloDalRi added a commit to alphagov/content-data-admin that referenced this pull request Mar 29, 2023
This component was removed in version 18 of govuk_publishing_components alphagov/govuk_publishing_components#1038

We need to move the code into the app so that we can update govuk_publishing_components and other dependencies.

https://trello.com/c/egaF1kjo/3110-replace-accessible-autocomplete-for-content-data-admin
MuriloDalRi added a commit to alphagov/content-data-admin that referenced this pull request Apr 3, 2023
This component was removed in version 18 of govuk_publishing_components alphagov/govuk_publishing_components#1038

We need to move the code into the app so that we can update govuk_publishing_components and other dependencies.

https://trello.com/c/egaF1kjo/3110-replace-accessible-autocomplete-for-content-data-admin
MuriloDalRi added a commit to alphagov/content-data-admin that referenced this pull request Apr 3, 2023
This component was removed in version 18 of govuk_publishing_components alphagov/govuk_publishing_components#1038

We need to move the code into the app so that we can update govuk_publishing_components and other dependencies.

https://trello.com/c/egaF1kjo/3110-replace-accessible-autocomplete-for-content-data-admin
MuriloDalRi added a commit to alphagov/content-data-admin that referenced this pull request Apr 3, 2023
This component was removed in version 18 of govuk_publishing_components alphagov/govuk_publishing_components#1038

We need to move the code into the app so that we can update govuk_publishing_components and other dependencies.

https://trello.com/c/egaF1kjo/3110-replace-accessible-autocomplete-for-content-data-admin
MuriloDalRi added a commit to alphagov/content-data-admin that referenced this pull request Apr 3, 2023
This component was removed in version 18 of govuk_publishing_components alphagov/govuk_publishing_components#1038

We need to move the code into the app so that we can update govuk_publishing_components and other dependencies.

https://trello.com/c/egaF1kjo/3110-replace-accessible-autocomplete-for-content-data-admin
MuriloDalRi added a commit to alphagov/content-data-admin that referenced this pull request Apr 3, 2023
This component was removed in version 18 of govuk_publishing_components alphagov/govuk_publishing_components#1038

We need to move the code into the app so that we can update govuk_publishing_components and other dependencies.

https://trello.com/c/egaF1kjo/3110-replace-accessible-autocomplete-for-content-data-admin
robinjam pushed a commit to alphagov/content-data-admin that referenced this pull request Apr 25, 2023
This component was removed in version 18 of govuk_publishing_components alphagov/govuk_publishing_components#1038

We need to move the code into the app so that we can update govuk_publishing_components and other dependencies.

https://trello.com/c/egaF1kjo/3110-replace-accessible-autocomplete-for-content-data-admin
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