Skip to content
This repository has been archived by the owner on Nov 2, 2021. It is now read-only.

Update Search.js #2469

Closed
wants to merge 1 commit into from
Closed

Update Search.js #2469

wants to merge 1 commit into from

Conversation

ronakv
Copy link

@ronakv ronakv commented May 16, 2021

Added a label to the search form to improve accessibility

Description of PR

A brief description of what changes your PR introduces. The better this description, the quicker we can review, exchange feedback, and merge!

Relevant Issues
Fixes #1610

Checklist
Basic HTML label

Screenshots

Add relevant screenshots here

Added a label to a the search form to improve accessibility
@ronakv
Copy link
Author

ronakv commented May 16, 2021

Did not realise this would close the issue automatically

@shuklaayush
Copy link
Member

This doesn't look right
image

@shuklaayush shuklaayush closed this Jun 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get a 100 on lighthouse for accessibility
3 participants