-
Notifications
You must be signed in to change notification settings - Fork 126
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
Update primary action link & remove useless prop #29848
Update primary action link & remove useless prop #29848
Conversation
<div className="arrow" style={{ maxWidth: '75%' }}> | ||
<div className="vads-u-background-color--primary vads-u-padding--1"> | ||
<a | ||
className="vads-c-action-link--white" | ||
href={`/form-upload/${formNumber}`} | ||
> | ||
Start uploading your form | ||
</a> | ||
</div> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given that we're using this exact same component in multiple places now, I'd love it if this could be defined in a common space and instantiated where necessary. Do we have such a spot we can park this component?
.arrow { | ||
position: relative; | ||
} | ||
|
||
.arrow::after { | ||
content: ""; | ||
position: absolute; | ||
top: 0; | ||
right: -24px; | ||
width: 0; | ||
height: 0; | ||
border-top: 24px solid transparent; | ||
border-bottom: 24px solid transparent; | ||
border-left: 24px solid var(--vads-color-primary); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same with this css file!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, this is straightforward and looks good! I'd really like to eliminate the code duplication and more importantly reduce points of potential divergence between the multiple iterations of this component so I'd like to see them combined into one definition if at all possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes lgtm, one suggestion
Summary
This PR updates the primary action link on the form upload flow. It also removes a prop that we don't use on the
content-build
side so it is useless. We'll switch on whether a form has the form upload flow purely on thecontent-build
side, not here.Related issue(s)
https://app.zenhub.com/workspaces/vagov-product-team-forms-634853151f5c6000165942bc/issues/gh/department-of-veterans-affairs/va.gov-team-forms/1280
Screenshots