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

10427 story v2 intermediate branch to test 1725995155 #5353

Conversation

En-8
Copy link
Contributor

@En-8 En-8 commented Sep 10, 2024

Allow text wrapping for InlineLinks, and a couple very minor tweaks to force inline links onto their own lines in some spots.

I just removed the text wrapping configurability on the component entirely since it's not needed at the moment -- if we get to a point where it is needed we can pretty easily add it back in.

@En-8 En-8 merged commit ff3cea5 into ustaxcourt:test Sep 11, 2024
44 checks passed
@En-8 En-8 deleted the 10427-story-v2-intermediate-branch-to-test-1725995155 branch September 11, 2024 13:53
@@ -51,6 +51,7 @@ export const FilingFeeOptions = function () {
<br />
<br />
<strong>Can’t afford to pay the filing fee?</strong>
<br />
<InlineLink
Copy link
Contributor

Choose a reason for hiding this comment

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

Should the prop be removed here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah yes, good catch. Sorry I didn't see this before I merged! I've made the corresponding change in the staging PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

👍 Didnt realize it was merged already!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants