Skip to content

Conversation

@syedszeeshan
Copy link
Collaborator

@syedszeeshan syedszeeshan commented May 7, 2025

Docs PR: GovAlta/ui-components-docs#362

Before (the change)

After (the change)

Make sure that you've checked the boxes below before you submit the PR

  • I have read and followed the setup steps
  • I have created necessary unit tests
  • I have tested the functionality in both React and Angular.

Steps needed to test

@syedszeeshan syedszeeshan changed the title NOT READY - fix(#2408): form-stepper incomplete rendered as partially complete fix(#2408): form-stepper incomplete rendered as partially complete May 7, 2025
@vanessatran-ddi vanessatran-ddi linked an issue May 7, 2025 that may be closed by this pull request
@vanessatran-ddi
Copy link
Collaborator

#2408 (comment)
QA passed.

Copy link
Collaborator

@vanessatran-ddi vanessatran-ddi left a comment

Choose a reason for hiding this comment

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

Hello @syedszeeshan can you add unit test or browser test for status not-started and the label has been changed for incomplete from Partially Complete to Incomplete which I think we should add a test to cover that.

@syedszeeshan syedszeeshan force-pushed the SyedZeeshan/2408-UI branch 2 times, most recently from 497e7f8 to 521ca93 Compare May 8, 2025 02:06
@syedszeeshan
Copy link
Collaborator Author

@vanessatran-ddi
I have added tests as requested.

vanessatran-ddi
vanessatran-ddi previously approved these changes May 8, 2025
@ArakTaiRoth
Copy link
Collaborator

Getting some weird things happening in NVDA.
Are these things being said with VoiceOver @vanessatran-ddi @twjeffery ?

image

The main issues are:

  • check box not checked - after every single step click
  • It also seems to repeat the step name, but for step 3 and step 4 it added something, "incomplete" and "4", no idea why
  • If I click the Previous and/or Next button nothing about the statuses is read (is this okay @twjeffery?)

@vanessatran-ddi
Copy link
Collaborator

Getting some weird things happening in NVDA. Are these things being said with VoiceOver @vanessatran-ddi @twjeffery ?

image

The main issues are:

  • check box not checked - after every single step click
  • It also seems to repeat the step name, but for step 3 and step 4 it added something, "incomplete" and "4", no idea why
  • If I click the Previous and/or Next button nothing about the statuses is read (is this okay @twjeffery?)

@ArakTaiRoth Yes for VoiceOver it is similar to NVDA
image

It is a good catch! I didn't notice it when I tested. However, on our production (design system website) it is behaving like that too
image

Included the "check box not checked" and "click the Previous and/or Next button" so I don't know if it is in the scope of this ticket.

@twjeffery
Copy link
Collaborator

twjeffery commented May 22, 2025

@syedszeeshan Preferred text read out:

Step 2 of 4, Step name: Employment history, Status: Complete

@syedszeeshan syedszeeshan changed the title fix(#2408): form-stepper incomplete rendered as partially complete NOT READY - fix(#2408): form-stepper incomplete rendered as partially complete May 22, 2025
@syedszeeshan syedszeeshan marked this pull request as draft May 22, 2025 16:15
@syedszeeshan
Copy link
Collaborator Author

syedszeeshan commented May 22, 2025

I have tried following, in FormStep.svelte

For input
-role=presentation OR aria-hidden=true
-aria-disabled = true (hardcoded)
-move aria-label from label to input

For label
-removed for=text
-removed tabindex
-move aria-label from label to input

and a few combinations of above.

No matter what, the part "checkbox checked/not checked" is always pronounced by the screen reader...probably b/c of the type checkbox

UPDATE:
After discussion with Chris, replacing input type="checkbox" with button and adjust the rest of the code to keep the same behavior, and adjust the tests also.

@syedszeeshan syedszeeshan force-pushed the SyedZeeshan/2408-UI branch 2 times, most recently from 72cc49e to c558e58 Compare May 23, 2025 19:11
@syedszeeshan
Copy link
Collaborator Author

image

@syedszeeshan syedszeeshan force-pushed the SyedZeeshan/2408-UI branch from 588621f to e9850a9 Compare May 23, 2025 19:45
@syedszeeshan syedszeeshan changed the title NOT READY - fix(#2408): form-stepper incomplete rendered as partially complete fix(#2408): form-stepper incomplete rendered as partially complete May 26, 2025
@syedszeeshan syedszeeshan marked this pull request as ready for review May 26, 2025 18:27
@vanessatran-ddi
Copy link
Collaborator

QA on 27th May 2025:
There is 1 case for Next/Previous button, not work for NVDA and VoiceOver.

  • Focus on the Next button using a keyboard
  • Press Enter
  • Step is changed but no screen reader announcing
  • Focus on the Previous button by pressing Shift + Tab
  • Enter
  • Screen reader announce for the previous button
    Sometimes it announces by using a keyboard and sometimes it doesn't, especially when the button just newly focus and first time press enter.

@syedszeeshan syedszeeshan force-pushed the SyedZeeshan/2408-UI branch from 24884a0 to a223529 Compare May 27, 2025 20:49
@syedszeeshan syedszeeshan force-pushed the SyedZeeshan/2408-UI branch from b0dee1f to e8e6592 Compare June 1, 2025 15:16
ArakTaiRoth
ArakTaiRoth previously approved these changes Jun 4, 2025
vanessatran-ddi
vanessatran-ddi previously approved these changes Jun 9, 2025
@ArakTaiRoth
Copy link
Collaborator

@chrisolsen This PR is ready to be merged

@chrisolsen chrisolsen merged commit dfed581 into alpha Jun 10, 2025
4 checks passed
@chrisolsen chrisolsen deleted the SyedZeeshan/2408-UI branch June 10, 2025 16:22
@tzuge
Copy link
Collaborator

tzuge commented Jun 10, 2025

🎉 This PR is included in version 1.5.0-alpha.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge
Copy link
Collaborator

tzuge commented Jun 10, 2025

🎉 This PR is included in version 4.5.0-alpha.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge tzuge added the released on @alpha Released into alpha. label Jun 10, 2025
@tzuge
Copy link
Collaborator

tzuge commented Jun 10, 2025

🎉 This PR is included in version 1.35.0-alpha.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge
Copy link
Collaborator

tzuge commented Jun 10, 2025

🎉 This PR is included in version 6.5.0-alpha.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge
Copy link
Collaborator

tzuge commented Jun 16, 2025

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge
Copy link
Collaborator

tzuge commented Jun 16, 2025

🎉 This PR is included in version 1.35.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge
Copy link
Collaborator

tzuge commented Jun 16, 2025

🎉 This PR is included in version 4.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge tzuge added the released Released into production. label Jun 16, 2025
@tzuge
Copy link
Collaborator

tzuge commented Jun 16, 2025

🎉 This PR is included in version 6.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released on @alpha Released into alpha. released Released into production.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Form Stepper: Incomplete being rendered as Partially complete

7 participants