department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
284 stars 206 forks source link

508-defect-3 [COGNITION]: "Finish this request later" button should be styled as a button and positioned higher in the page #25111

Open joshkimux opened 3 years ago

joshkimux commented 3 years ago

508-defect-3

Feedback framework

Definition of done

  1. Review and acknowledge feedback.
  2. Fix and/or document decisions made.
  3. Accessibility specialist will close ticket after reviewing documented decisions / validating fix.

Point of Contact

VFS Point of Contact: Josh Kim

Details

The "Finish this request later" button is styled as a link which is materially dishonest causing several issues:

For more on this long-tail effort, check out our buttons vs. links Mural board and this Slack thread discussing why material honesty is so important.

"A few years at an accessibility event, a blind man was telling a story about how he was on the phone with someone who was trying to help him sign up for a service. The person on the phone told him to click the “sign up button” and he couldn’t find it because it was a link. He told the story with a lot more gravity than I can, but that was the point where I realized how important the human-human communication part of accessibility is, it’s not just machine to human." - Tim Wright

Solution

Our solution may change depending on what approach we take and our constraints.

Following this thread with @thehastingsp and @davidakennedy , I'm curious if we might be able to:

Environment

WCAG or Vendor Guidance (optional)

Screenshots or Trace Logs

Screen Shot 2021-05-24 at 7 27 31 AM
caw310 commented 3 years ago

@joshkimux , this issue was discussed today in the DSC meeting. Here is the follow.

Mottie commented 3 years ago

Temporary fix that moves the "button" above the back & continue buttons has been merged. Still pending is:

three changes might help:

  1. Changing this to a Link from react-router-dom to preserve Redux state when navigating to a new URL
  2. Make this link only navigate to a new page (not call a function which both saves and navigates)
  3. Make the new page do the saving (which is why we needed to preserve the Redux state)
    • This will require a design change (which is why I mentioned the scope creep :sweat_smile:)

from this comment

Mottie commented 3 years ago

I've merged in another change that renders the "Finish this request later" link above the submit button on the review & submit page.

Mottie commented 2 years ago

"Finish this request later" button is styled as a button change to be addressed by the Design system team

SarahKay8 commented 2 years ago

@caw310 @caw310 Hi Carol, per Josh's recommendations, "Finish this request later" link has successfully been placed above "Continue + Back" buttons however "Finish this request later" link is still styled as a button

Screenshot for reference: Image 7-25-22 at 11.00 PM.jpg

joshkimux commented 2 years ago

@SarahKay8 and @caw310 , this is awesome work to bump the finish this request later button above the back and continue buttons. Will help a ton with screen reader and mobile discoverability. That being said, I acknowledge that styling this as a button may present some visual design challenges given our absence of a tertiary style for buttons.

Instead of focusing too much on redesigning its style, it may be easier to address this by asking questions a little more upstream:

k80bowman commented 2 years ago

Thank you @SarahKay8 and @joshkimux. As this is a forms library issue, I've added the forms label. @taylorkaren @micahchiang how would you like to handle this?

taylorkaren commented 2 years ago

Remaining work to do: The "Finish this request later" button is styled as a link but should be implemented as a button.

obliviga commented 2 years ago

Hi @joshkimux, should the "Finish this request later" button be styled as a primary or secondary button? Thanks!

Reference: https://design.va.gov/components/button/#examples

joshkimux commented 2 years ago

@obliviga this is tricky, because styling this as a primary or secondary button may negatively impact the sighted user experience for mobile users or folks with cognitive disabilities. I think it would be helpful to have a more upstream conversation about the questions I raised earlier. It may be easier to keep this as a defect-level-3 in the old (current) form system and attempt to resolve this officially in the new form system:

joshkimux commented 2 years ago

Hey folks, quick update here. Following this thread with @thehastingsp and @davidakennedy , I'm curious if we might be able to:

(@DanielleThierryUSDSVA for awareness, just in case, 'cuz she's awesome 😄 )

joshkimux commented 1 year ago

@jeana-adhoc I'm not sure if we may have talked about this, but bringing this up just in case it's relevant to your work on the forms team. The last comment has our most recent context on the issue.

rsmithadhoc commented 3 months ago

@caw310 I believe this one (and https://github.com/department-of-veterans-affairs/vets-design-system-documentation/issues/1766) would be up to the forms team to address, as this isn't in the design system.

caw310 commented 3 months ago

@tbaker1026 , would the VFFT be able to pick this up?

caw310 commented 2 months ago

@tbaker1026 Would you add this to your backlog since it is a form issue?

jeana-adhoc commented 2 months ago

@caw310 - This might be a question for @humancompanion-usds. We just tested moving this even lower on a page, and it tested fine.

How do you handle competing old tickets with new information on the DST?

caw310 commented 2 months ago

@jeana-adhoc We are actually closing old tickets and asking people to open a new ticket if the issue persists with the new VADS components.

jeana-adhoc commented 2 months ago

Thanks, @caw310 - We might do the same here. But I need to get my head around what the remaining issue is, and find out if its still an issue.

humancompanion-usds commented 2 months ago

@jeana-adhoc and @tbaker1026 - At your leisure, move this over to the forms team repo, or open a new issue. Jeana - Sounds like the remaining work is for us to update our forms templates in Figma with the new location, update it in code, and then socialize that change?