Skip to content

LG-5213: Hide "Start Over" and "Cancel" in capture tips#5553

Merged
aduth merged 3 commits intomainfrom
aduth-lg-5213-start-over-cancel
Oct 29, 2021
Merged

LG-5213: Hide "Start Over" and "Cancel" in capture tips#5553
aduth merged 3 commits intomainfrom
aduth-lg-5213-start-over-cancel

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Oct 28, 2021

Follow-up from #5534

Why: Per design, the "Start Over" and "Cancel" links are not intended to be shown in the troubleshooting tips. Because this was rendered outside the React application, the approach to hiding them is to absorb the logic of the links into the React application, rendering them only where applicable.

Screenshots:

Before After
localhost_3000_verify_doc_auth_document_capture(iPhone X) localhost_3000_verify_doc_auth_document_capture(iPhone X) (1)

Follow-up from #5534

**Why**: Per design, the "Start Over" and "Cancel" links are not intended to be shown in the troubleshooting tips. Because this was rendered outside the React application, the approach to hiding them is to absorb the logic of the links into the React application, rendering them only where applicable.
Portal-ing is non-obvious
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@anniehirshman-gsa anniehirshman-gsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@aduth aduth merged commit 72d34a0 into main Oct 29, 2021
@aduth aduth deleted the aduth-lg-5213-start-over-cancel branch October 29, 2021 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants