Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Med review language proposal #82736

Merged
merged 9 commits into from
May 29, 2024
Merged

Conversation

outerpress
Copy link
Contributor

@outerpress outerpress commented May 9, 2024

Please review the language proposed in this file on this branch. You can edit the content directly, or add comments to this PR and I'll make the updates.

@outerpress outerpress added design appointments-fe-ux VAOS Product & Design team tag. Do not use for engineering work. labels May 9, 2024
@outerpress outerpress self-assigned this May 9, 2024
@outerpress outerpress marked this pull request as ready for review May 9, 2024 14:31
@outerpress
Copy link
Contributor Author

I'm thinking this content could be accessed as needed in an additional info component. It's not something everyone will have to read every time. Something like:

{h2} Prepare for your visit

Bring a list of your medications to review with your provider.

{Addl Info Link} More information

> Final med review content

{End Addl Info Content}

Thoughts @laurwill @kristenmcconnell ?

@mistymg does mobile have similar UI that could support this?

@mistymg
Copy link
Contributor

mistymg commented May 13, 2024

@outerpress Yes, we have a link component that when tapped opens a large panel (a modal). My only concern though is that "More information" is not really helpful content for link copy. Wondering if @laurwill or @DanielleThierryUSDSVA agree that "more information" needs to changed. Maybe something more like "Learn what medication information to share with your provider" or "Find out what medication information to share with your provider"? This is where seeing it in a design mock up would eventually help to see how the copy and UI are playing with one another.

@outerpress
Copy link
Contributor Author

@mistymg I did a mock of the proposed language in our content library if it helps to see the placement with the rest of the content. (With the caveat that this is all WIP so all comments are welcome :) )

@mistymg
Copy link
Contributor

mistymg commented May 13, 2024

@outerpress @laurwill @DanielleThierryUSDSVA If we use the link component that opens to a modal, this is what it would look like in mobile app. Some concerns:

Updated add'l info trigger and added Figma example
@outerpress
Copy link
Contributor Author

@mistymg I updated the proposal based on this Slack thread:

  • Tweaked the additional info trigger language
  • Added an example showing the language in web

I'll drop your examples in the linked Figma file, and get Check-in in there next.

@outerpress outerpress merged commit ed9b4ad into master May 29, 2024
5 checks passed
@outerpress outerpress deleted the 05-24-med-review-lang-proposal branch May 29, 2024 13:35
@outerpress
Copy link
Contributor Author

No further changes to the Medication Review language so merging this into main. Next step is review with NAPMIS and other stakeholders.

  • Content source of truth is here
  • Examples in context are here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
appointments-fe-ux VAOS Product & Design team tag. Do not use for engineering work. design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants