Closed bbertucc closed 1 month ago
working on it
complete ✅
On JAWS and confirm this is still an issue. For example when I delete a report or property I only know the reports or properties page has loaded. After I verify and sign up, I don't know that I've been taken to the reports page. I press the send to scan button I don't get any information other than the page has loaded--it seems to not do much of anything from a screen reader perspective.
Looks like this hasn't been deployed yet @kevinandrews1
@wilsuriel03 - as I wrote in Slack, please use the Radix Toast notifications. Those Radix notifications are in spec of W3C accessibility. They don't work all the time on Apple VoiceOver, but that's a problem with how Apple VoiceOver works - not Radix. @azdak and @alexstine also tested the radix example and it works fine.
Let me know when it's up on staging, then @kevinandrews1 or @alexstine can test one last time and I can push to production.
@bbertucc this is ready for testing
When testing getting page not found when trying to create report or property - @wilsuriel03 I'm logged into @bbertucc's on dev. For the record the success notifcation when logging in is good. I'll keep this open for now but I can retest later today if this is fixed in the interim. I'm thinking the toast is ok but want to confirm before closing.
@kevinandrews1 can you try this again on our production? This time, can you go through our full list of tests and record what is or isn't creating notifications? (A screen recording of you going through and just saying "this worked" and "this didn't" might be best.
Here is the full list of tests in order that you should do it (using H3s to divide up the tests):
Start URL: https://dashboard.equalify.app/signup Test:
Note: When testing on a screen reader, the signup page and email verification page should be read out in addition to all form verification notifications.
Start URL: https://dashboard.equalify.app/signup Test:
Note: When testing on a screen reader, the signup page and email verification page should be read out in addition to all form verification notifications.
Start URL: https://dashboard.equalify.app/reports Test:
Start URL: https://dashboard.equalify.app/reports
Start URL: https://dashboard.equalify.app/reports
Start URL: https://dashboard.equalify.app/reports
Start URL: https://dashboard.equalify.app/reports
Start URL: https://dashboard.equalify.app/reports Test:
Start URL: https://dashboard.equalify.app/reports Test:
Start URL: https://dashboard.equalify.app/reports Test:
Start URL: https://dashboard.equalify.app/reports Test:
@wilsuriel03 -
Yay! looks like most of the notifications are working perfectly! A few notes:
Video 1
Video 2
@bbertucc @kevinandrews1 I did not see a visible page message that an account was successfully deleted. If a message is not visible on the page, should we announce it?
@alexstine we might actually just need to create that message.
Moving this to Version 1.1, since we decided it wasn't mission cirtical.
@azdak updating this for you. Deleting an account is the only toast notification that was left.
So due to how stuff is architected, there's a little bit of an edge case/race condition thing happening with the account deletion toast, the user flow being:
...and all this happens before there's time to push a toast notification.
I've tweaked things slightly on staging so that following user confirmation of account deletion, a toast is pushed that says "Deleting account...", and then the delete function is called, and the user is taken to the login screen (a separate toast will appear if the deletion fails). I think this more or less conveys what's happening to the user, but open to it if folks think we need more- could also try to figure out another solution that gets around this limitation, like a system to inject notifications into the login screen. Thoughts?
@azdak After the redirect to the login page, can you display the toast at that point?
This is a tricky order of operations issue.
@alexstine Ok so I was probably overthinking this haha- everything I was saying above is true, but this being an SPA, code context is largely kept intact between views- so while I was tinkering, I tried just wrapping the toast call in a timeout, and it's now showing up both visually and in NVDA for me on the login screen after account deletion lol. Not the most elegant solution, but gift horses, mouths, etc. Just pushed to staging for testing-
@azdak On staging, the announcement is working now but the toast is not visible in the DOM on the login page. I get the alert that my account has been deleted and then login page loaded.
On submitting a form or doing an action, there are no notifications that the action was done.
Here is what I expected: