-
Notifications
You must be signed in to change notification settings - Fork 193
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
Med review language proposal #82736
Conversation
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:
Thoughts @laurwill @kristenmcconnell ? @mistymg does mobile have similar UI that could support this? |
@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. |
@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 :) ) |
@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
@mistymg I updated the proposal based on this Slack thread:
I'll drop your examples in the linked Figma file, and get Check-in in there next. |
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.