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

[Outreachy Task Submission] Fix Missing Search Button Value Text #902

Conversation

Chiemezuo
Copy link
Contributor

@Chiemezuo Chiemezuo commented Mar 16, 2024

Introduction

The search form component in both the Data and Map view has a problem. The search button has only an icon and has no value text. While this is just a stylistic choice, it raises an issue with any accessibility checking tool. It does this because a button having no value text is a violation of WCAG Success Criterion 1.1.1 (Level A) - Non-text content. See. See a screenshot of the pertinent component:

image

Note the search icon

How to Test the PR

Manually Testing

  1. Open your browser and log into your running Ushahidi deployment (from localhost).
  2. Navigate to either the data or map section.
  3. Try right-clicking on the search form at the top of the screen and selecting 'inspect'.
  4. View the properties, and look for a span element with a class of mzima-button__text. See screenshot below:
    image
  5. You will not see any text in it.
  6. Stop the running Ushahidi deployment in your terminal.
  7. Checkout to this PR branch.
  8. Re-run your Ushahidi deployment.
  9. Repeat steps 1-4.
  10. You will see a value in the span element this time around, and this text will be visually hidden.

Using an Accessibility Checker

If you use a tool like WAVE evaluation tool, it's as simple as following steps 1 and 2 above, and then turning on the browser extension. It will flag the issue. Checkout to the PR and do this again, you will notice the issue is no longer flagged.

Dependency

It has a dependency on a previous PR

Fixes

It partially fixes #4805

@Angamanga Angamanga self-requested a review March 20, 2024 19:04
Copy link
Contributor

@Angamanga Angamanga left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution 🙏

@Angamanga
Copy link
Contributor

@Chiemezuo I already approved this but please go ahead and remove the !important here as well 🙏

@Chiemezuo
Copy link
Contributor Author

Alright! @Angamanga
Consider that done.
Thank you! 🥳

…iemezuo/ushahidi-platform-client-mzima into fix/search-form-component-button-text
@Angamanga Angamanga closed this Oct 7, 2024
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.

[Outreachy Task Submission] Buttons do not have an accessible name
2 participants