-
Notifications
You must be signed in to change notification settings - Fork 1
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
Minor content changes for 2023 #139480
Minor content changes for 2023 #139480
Conversation
This one I could go either way on, if only due to timing. My guess is the trailing colon will be fine. It's worth running by David, though. |
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.
We want to hide the 'optional' hint, right?
Otherwise changes look good. I verified the trailing colon and non-breaking space don't have any issues when read with VoiceOver (It just says "Part 1" and moves on)
Code Climate has analyzed commit a5addc8 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (90% is the threshold). This pull request will bring the total coverage in the repository to 61.9% (0.2% change). View more on Code Climate. |
Description
title
. I don't have any text to add for the title but I can add an invisible truthy string: U+00A0, the non-breaking space. This causes the headers to render asPart 1:
andPart 2:
. If the trailing colon is not desirable, then this request will require a broader change."typeLabel": "HSI Program"
, and an apparent bug: it seems like it was always intended for this text to pass through to the button text. Additionally, it seems like it was always intended to pass this through to the "do you have another" question text, as well as the "delete last" button text. I've "fixed" it here, but THIS MAY NOT BE A GOOD IDEA, as it will affect all repeatables throughout the report. This includes all six repeatables in Section 4, which will now ask "Do you have another Goal in this list?" with button text "Add another Goal".hideOptionalHint
flag through the question.Related ticket(s)
MDCT-3035
How to test
Important updates
n/a
Author checklist