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

[CST] [ENG] Navigate to the files upload section within the files tab from notice details page #94061

Closed uxgary closed 2 weeks ago

uxgary commented 2 months ago

Value Statement

As a Veteran I want to directed straight to the file upload section after clicking a CTA to upload evidence So that I'm not confused by landing on a page that doesn't immediately display the upload options.


Background Context

The research study, revealed that Veterans were momentarily confused when clicking the 'upload evidence' link from the notice details page, as they were directed to the top of the files page. It took them time to reorient themselves and locate the upload section.

We aim to link them directly to the 'additional evidence' section of the files page, allowing them to quickly understand where they are and take the necessary steps to upload their evidence.

Below is an example of the confirmation message in the files page.

Bug

New Feature

Outcome, Success Measure, KPI(S), and Tracking Link

Design

Enablement team (if needed)

@member of team

Engineering

Out of scope

Open questions

Tasks

Definition of Done

Acceptance Criteria

skylerschain commented 1 month ago

This one is more of an engineering ticket as I think it would just involve updating the link. Pointed it at a 1 but happy to defer if Eng feels differently.

pmclaren19 commented 4 weeks ago

Hey @harshil1793 heres the ticket I was referring to that I worked where we removed the scrolling logic from AdditionalEvidencePage.jsxand added it to the parent component FilesPage.jsx. It is using setPageFocus() and setUpPage() to set the page focus. I'm pretty sure that your going to want to change the code in FilesPage.jsx, lines 42 - 46 where setUpPage() is being called since the va-breadcrumbs is the focus that we want to override.

jacobworrell commented 3 weeks ago

@harshil1793 Hello, what's the status on this ticket? Is there a PR?

pmclaren19 commented 2 weeks ago

Tested in Staging on user 19, 1st claim and it looks good! @skylerschain @jacobworrell please review and close if all looks good!

skylerschain commented 2 weeks ago

Design QA: LGTM 👍