-
Notifications
You must be signed in to change notification settings - Fork 19
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
Enable bold text in govspeak #463
Conversation
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 is going to need a lot of description in the PR, since the change is likely to be controversial and will be referenced a lot.
As for the code, we also need to remove the entire rich_govspeak
feature. That means removing the param, examples from the YAML file, and remove the param from any existing page that uses it.
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.
With the removal of node_modules
Certain types of guidance have tested positively with inline bold text (eg. https://www.gov.uk/guidance/connect-to-govwifi-using-an-android-device) Instead of enabling bold text as by exception we'd like to see the effect of enabling entirely. If nothing else it's good to talk through why this is a good or bad idea and what the effect of permitting bold text has and will be on content managed text.
Bold text has been enabled in govspeak so the rich-govspeak styles can be removed along with the component options.
https://trello.com/c/BUij6N3Y/361-turn-on-bold-for-whitehall
Do not merge because we'd like to see how this works in various formats in a preview app.
We've been asked to enable bold text for a specific format which tested favourably.
(eg. https://www.gov.uk/guidance/connect-to-govwifi-using-an-android-device)
Instead of enabling bold text by exception this PR enables it entirely for govspeak text containing
<strong>
tags.Component guide for this PR:
https://govuk-publishing-compon-pr-463.herokuapp.com/component-guide/