equalify icon indicating copy to clipboard operation
equalify copied to clipboard

Lack of Feedback When Creating Reports or Properties

Open kevinandrews1 opened this issue 1 year ago • 8 comments

There is no feedback to inform users when actions such as creating a report or adding a property are successfully completed. For example, after adding a property, there is no communication indicating that the property has been successfully created. Clear feedback is crucial for users to know when their actions have been successfully completed. Without it, users may be unsure if their action was successful or if they need to retry. To address this, programmatically provide success messages using aria-live regions to announce updates to screen readers. For instance, after a report is created or a property is added, an aria-live="assertive" region can announce “Report successfully created” or “Property successfully added.” Additionally, move the focus to the success message once the action is completed to ensure users relying on screen readers are immediately notified of the success.

kevinandrews1 avatar Jul 18 '24 09:07 kevinandrews1

complete ✅

wilsuriel03 avatar Jul 18 '24 21:07 wilsuriel03

Hey @wilsuriel03 I get a 404 when trying to create a report so I'm unable to test that part. When I add a property now I know the properties page has loaded but it doesn't give any sort of indication the property was successfully added either through an on-screen message or solely through something like an ARIA alert to the screen reader.

kevinandrews1 avatar Jul 19 '24 12:07 kevinandrews1

I get a 404 when trying to create a report

I created a separate issue, #361, with that note

bbertucc avatar Jul 19 '24 20:07 bbertucc

both have been completed ✅

wilsuriel03 avatar Jul 19 '24 22:07 wilsuriel03

Creating a report, creating a property, and deleting a report don't communicate programmatically to the user that the action was successful--those are the three I tried @wilsuriel03.

kevinandrews1 avatar Jul 20 '24 14:07 kevinandrews1

@kevinandrews1 it eventually says "report deleted successfully" but it does take a while

@wilsuriel03 there might want to move up the component to read out sooner or use aria-live="assertive"

bbertucc avatar Jul 22 '24 16:07 bbertucc

@wilsuriel03 here is more documentation: https://www.radix-ui.com/primitives/docs/components/toast

bbertucc avatar Jul 22 '24 16:07 bbertucc

This makes way more sense, @bbertucc. I won't say rip out the toast because time. So, instead, @wilsuriel03, read this piece on toasts by our good friend/mentor, Adrian, please.

kevinandrews1 avatar Jul 22 '24 20:07 kevinandrews1

@kevinandrews1 this should work now. I understand your concern about toast. I will see what I can come up for V2. For now this issue has been fix

wilsuriel03 avatar Jul 23 '24 00:07 wilsuriel03

Assigning @kevinandrews1 this to test.

bbertucc avatar Jul 23 '24 10:07 bbertucc

On my call with @wilsuriel03 this afternoon I was not aware, for example, the report was deleted when we tested. @wilsuriel03 is going to do some more work--actually they mentioned making sure it merged with the latest branch.

kevinandrews1 avatar Jul 23 '24 21:07 kevinandrews1

Kicking this back to @wilsuriel03

kevinandrews1 avatar Jul 23 '24 21:07 kevinandrews1

Hey I jsut saw, in retesting #351 I didn't know when the account was deleted but saw I was back on login screen @wilsuriel03 not sure if it's related to the toasts issue.

kevinandrews1 avatar Jul 23 '24 21:07 kevinandrews1

@kevinandrews1 , I cant seem to recreate this issue. I have downloaded a screen reader and tested and I can hear the notification of the successful action.

wilsuriel03 avatar Jul 26 '24 03:07 wilsuriel03

@kevinandrews1 would you mind adding a video to confirm?

And @wilsuriel03 feel free to switch the assignee to Kevin or anyone else if you need something from them.

bbertucc avatar Jul 26 '24 10:07 bbertucc

@wilsuriel03 I just tested creating and deleting reports and properties and it works but the issue boils down to the screen reader user not having an indication that action was successful. If there's a message on screen either about the asset being created or deleted, programmatically there's no communication of that message to the user. even on screen I check and manually and either it's quickly appearing and disappearing or it never appears. I would have something on screen for everyone regardless but also alert the user about the action. Here's the video you requested. Leaving this open for now: https://drive.google.com/file/d/1QkNcwdeOtgXSGYhCwEOW1JaMAFVnA1Wm/view?usp=sharing

kevinandrews1 avatar Jul 30 '24 01:07 kevinandrews1

I pushed another version, attempting to fix it. not sure why jaw screen reader isnt capturing it. @kevinandrews1

wilsuriel03 avatar Jul 30 '24 01:07 wilsuriel03

NO change @wilsuriel03 also on incognito here. Maybe give me a quick breakdown of what happens when a report and property is either created or deleted, what the implementation is, etc. I can do some research on this and try to figure out why JAWS isn't speaking it.

kevinandrews1 avatar Jul 30 '24 02:07 kevinandrews1

@wilsuriel03 and @kevinandrews1 I'm closing this issue in favor of #390 and #388 that should address this issue. Focusing on more discrete issues will allow us to better fix the problem.

bbertucc avatar Jul 30 '24 11:07 bbertucc