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
281 stars 197 forks source link

[Design] 10-10CG - Improvements: Content Cleanup post-July update #89939

Closed hdjustice closed 2 weeks ago

hdjustice commented 1 month ago

Background

Now that the July form updates have been released, we want to review content and make some adjustments to better align with VADS guidelines. These changes include removing "please" from some error messages, updating the confirmation page Caregiver address, and a few others outlined in the design file and list below.

Resources

Tasks

Acceptance Criteria

jessicastump commented 1 month ago

Setup/started a staging and ideal state view in Figma here; and updated Mural source of truth with notes

jessicastump commented 1 month ago

hey @hdjustice -- The figma file side-by-sdie is ready to review/walk through together.

I noticed a couple other minor updates and spacing issues noted those too in the Figma file summary here:

hdjustice commented 1 month ago

i love the side by side. I know that was a lot of work, but i think it will help when we start working on it.

@longmd - when you have a moment, check out the Figma file and see if there is anything that stands out to you. We can also review in a call if you would like to.

I plan to break this down, maybe by page/section? Let me know what you think would be easiest. That way you could put up PR for one thing and move on to the next.

longmd commented 1 month ago

@hdjustice / @jessicastump , I agree. Love the breakout and description of what each change needs to be. Great work on this! 🙃

jessicastump commented 3 weeks ago

@hdjustice - With the various FE tickets opened, is this good to close or should we wait until we hear back from CAIA on the "please" convo?

Slack thread convo from Aliyah:

The CAIA team has been working on updating error messages, including removing "please", and getting that guidance in the design system. I'm not sure if that overlaps at all with the forms library and what Platform asked you all to do. But in alignment with our content guidance, where possible, we recommend that teams remove "Please" in their content. As mentioned in previous convos, if the "please" is locked in to the error message component (i.e. it's not customizable) it's fine to leave that in for now. And once the design system is updated, CAIA can circle back about the error messages. I'm not entirely sure why Platform recommended a bug ticket for this, since CAIA is working on error message updates. Laura Willwerth I know you've been working with one of our copy editors on these error message updates. Do you recommend that Jessica and Matt hold on opening a bug ticket with the design system since you're working on getting error message updates in the design system?

longmd commented 3 weeks ago

RE: the "Please" stuff... I'm of the opinion now, I'll just go in to the forms code and update everything to remove it. Then, I'll link to the guidance you gathered in the PR and use that to push it through. I don't think we need to do a bug ticket or hold off on the update.

hdjustice commented 2 weeks ago

I am good with that direction, thank you both! @jessicastump - If you have a few minutes today, I want to go through the content list really quick and make sure we have what we need there.

jessicastump commented 2 weeks ago

Closing this ticket and adding a note - there are some spacing discrepancies within the address blocks and checkboxes used but they are not consistent. We will review and open up bug tickets (our team or Platform) that come up during QA but the design part of the content clean up is done :) cc @hdjustice