Skip to content

Show hint text on nurse-facing health answers#146

Open
paulrobertlloyd wants to merge 1 commit intomainfrom
hint-text-for-health-answers
Open

Show hint text on nurse-facing health answers#146
paulrobertlloyd wants to merge 1 commit intomainfrom
hint-text-for-health-answers

Conversation

@paulrobertlloyd
Copy link
Collaborator

Gets a bit busy (which is why I’ve made the left border on responses blue to pull them out a bit more). Too much?

Screenshot of health answers showing hint text.

@paulrobertlloyd paulrobertlloyd force-pushed the hint-text-for-health-answers branch from 2d1b9a4 to bcc38ca Compare September 3, 2025 16:45
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-hint-text--57wm1v September 3, 2025 16:45 Inactive
@paulrobertlloyd paulrobertlloyd force-pushed the main branch 7 times, most recently from 17cc046 to 4a172c2 Compare November 17, 2025 16:41
@paulrobertlloyd paulrobertlloyd force-pushed the main branch 4 times, most recently from d1be53d to e0b32ac Compare November 26, 2025 20:35
@paulrobertlloyd paulrobertlloyd force-pushed the hint-text-for-health-answers branch from bcc38ca to e311f5e Compare December 22, 2025 12:22
html += formatHealthAnswer(healthAnswer)
}

let keyText =
Copy link

Choose a reason for hiding this comment

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

Given keyText is now immediately overwritten and its initial value not used, we can remove lines 247 and 248 and move its declaration to line 250. Unless, that is, it was a mistake to replace keyText with healthQuestions[key].label in those later lines?

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.

2 participants