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

UHF-5527 Small accessibility improvements for unit searches and custom searches #902

Merged
merged 14 commits into from
Feb 9, 2024

Conversation

teroelonen
Copy link
Contributor

@teroelonen teroelonen commented Feb 2, 2024

UHF-5527

What was done

  • Added role="search" to form element in unit searches and custom searches.
  • Added type="search" to inputs that were previously type text in forms in unit searches and custom searches.
  • Removed magnifying glass icon from the search inputs in unit searches and custom searches.

How to install

  • Make sure your instance is up and running on latest dev branch. I recommend testing this with all instances that are listed below in the testing instructions.
  • For KASKO and STRATEGIA there is feature-branch that needs to be used as well.
    • git pull origin dev
    • make fresh
      OR
    • git fetch
    • git checkout UHF-5527
    • make fresh
  • Update the HDBT theme
    • composer require drupal/hdbt:dev-UHF-5527
  • Run make drush-cim drush-cr

How to test

Make sure that for each of these listed searches the form element has attribute role="search" and that if there is text-input field on the filters it has the type="search" attribute. Also the input field should no longer have the magnifying glass icon on any of the searches. ATTENTION: the ploughing schedule search uses HDS-SearchInput component and that component doesn't support type-attribute so it doesn't have it.

KASKO:

STRATEGIA:

SOTE:

KUVA:

REKRY:

KYMP:

ETUSIVU:

Designers review

  • This PR does not need designers review

Other PRs

@teroelonen teroelonen added the recreate-reference-images Use this label to re-create reference images used by Visual regression tests label Feb 7, 2024
Copy link

github-actions bot commented Feb 9, 2024

✅ Tests passed! You can check the output here: https://city-of-helsinki.github.io/drupal-hdbt/pull/902/html_report/

@teroelonen teroelonen merged commit 478c584 into main Feb 9, 2024
2 checks passed
@teroelonen teroelonen deleted the UHF-5527 branch February 9, 2024 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
recreate-reference-images Use this label to re-create reference images used by Visual regression tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants