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

Restore behaviour of headings being able to be rendered inside example boxes #374

Merged
merged 2 commits into from
Dec 24, 2024

Conversation

callumknights
Copy link
Contributor

@callumknights callumknights commented Dec 20, 2024

This is technically against govspeak usage guidance, but was behaviour that worked before, and the changing of implementation for example blocks in #350 broke it.

Changing back to previous behaviour for now until more work is done on govspeak components and their interactions.

The strange behaviour with spacing appears to be related to this issue, credit to @brucebolt gettalong/kramdown#213

Trello: https://trello.com/c/UVePLiKZ/1481-re-enable-headings-inside-example-boxes-in-govspeak

⚠️ This repo is owned by the publishing platform team. Please let us know in #govuk-publishing-platform when you raise any PRs. ⚠️

@callumknights callumknights force-pushed the restore-example-headings branch 2 times, most recently from 3c8032d to fa35a4a Compare December 23, 2024 11:16
@callumknights callumknights force-pushed the restore-example-headings branch 5 times, most recently from 38c5866 to dbe164d Compare December 24, 2024 10:51
Headings used to work inside example boxes, that behaviour recently broke
and caused issues for publishers, this reinstates that behaviour.
@callumknights callumknights force-pushed the restore-example-headings branch from dbe164d to 3509809 Compare December 24, 2024 10:53
@callumknights callumknights merged commit 6242ac9 into main Dec 24, 2024
9 checks passed
@callumknights callumknights deleted the restore-example-headings branch December 24, 2024 11:11
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.

3 participants