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
Adjust code to start to use required fields logic that ffb added. #235
Conversation
aria-describedby=${hasHelpText ? inputName + '-help-text' : ''}, | ||
aria-labelledby=${hasAriaLabel ? ariaLabel : ''}, | ||
aria-invalid=${hasError}" | ||
th:value="${hasValue ? value : fieldData.getOrDefault(inputName, '')}"> |
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.
This spot allows for an override of the data filled into the field. I don't think we need this in MD's code. But maybe we should pull this logic into the ffb?
1220fd7
to
08a53cd
Compare
@@ -640,10 +640,10 @@ income-delete-confirmation.title=Delete household member income | |||
income-delete-confirmation.yes=Yes, delete it | |||
household-income-total.add-by-job=Do you want to add your monthly household income by job instead? This can give your caseworker more details to better help with your application. | |||
household-income-total.no=No, continue | |||
household-income-total.yes=Yes, add income by job | |||
household-income-total.yes=Yes, add income by job3 x |
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.
Typo?
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.
I'll take this out.
<th:block | ||
th:replace="~{fragments/form :: form(action=${formAction}, content=~{::formContent})}"> | ||
<th:block th:ref="formContent"> | ||
<div class="form-card__content"> | ||
<th:block | ||
th:replace="~{fragments/inputs/money :: money(inputName='householdIncomeLast30Days', ariaLabel='header', placeholder='0.00')}"/> | ||
th:replace="'fragments/inputs/money' :: money(inputName='householdIncomeLast30Days', ariaLabel='header header-help-message', placeholder='0.00')"/> | ||
</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.
Hmmm... not sure if my rebase reversed this, but we do want the ~{fragement}
syntax instead of the single quotes.
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.
nm, I'm taking this out as it's a duplicate field on this page now.
Helps address: https://www.pivotaltracker.com/story/show/186994206
text.html
template from this code base and lets us default to the one in ffl (and therefore get required field functionality)selectHelpNeeded.html
to move some text into the card header.