Skip to content

made changes to InputMask component and input html to fix missing and…#2611

Merged
Tony-Villa merged 2 commits into
developfrom
2572-wave-accessibility-projectspage-formlabels
Aug 28, 2025
Merged

made changes to InputMask component and input html to fix missing and…#2611
Tony-Villa merged 2 commits into
developfrom
2572-wave-accessibility-projectspage-formlabels

Conversation

@heejung-hong

Copy link
Copy Markdown
Member

What changes did you make?

  • Added the selectProps to the InputMask component and hardcoded the props of name and id in the input element.

Why did you make the changes (we will use this info to test)?

  • The changes were made to fix the missing and orphaned label warning from the Wave extension on Chrome.

Issue-Specific User Account

The tdm@hackforla.org account was used for testing.

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied Screenshot 2025-08-18 at 11 02 33 PM Screenshot 2025-08-18 at 11 03 01 PM
Visuals after changes are applied Screenshot 2025-08-18 at 11 01 38 PM Screenshot 2025-08-18 at 11 06 37 PM

@eburdekin
eburdekin self-requested a review August 19, 2025 20:55
Comment thread client/src/components/ProjectWizard/RuleInput/MultiInput.jsx Outdated
@heejung-hong
heejung-hong requested a review from eburdekin August 20, 2025 19:08

@eburdekin eburdekin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Don't see the error anymore either. Looks good!

@Tony-Villa
Tony-Villa merged commit 9484651 into develop Aug 28, 2025
4 checks passed
@Tony-Villa
Tony-Villa deleted the 2572-wave-accessibility-projectspage-formlabels branch August 28, 2025 01:29
@geolunalg geolunalg mentioned this pull request Nov 13, 2025
20 tasks
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.

Dev: WAVE Accessibility: Create Project Pages, Form Labels

3 participants