Skip to content

Tag Page <p> styling#14270

Merged
abeddow91 merged 3 commits intomainfrom
ab/tag-page-formatting
Jul 24, 2025
Merged

Tag Page <p> styling#14270
abeddow91 merged 3 commits intomainfrom
ab/tag-page-formatting

Conversation

@abeddow91
Copy link
Contributor

@abeddow91 abeddow91 commented Jul 23, 2025

What does this change?

Add 4px margin at the bottom of <p> tags and adds styling to some tags that are nested within paragraphs such as bold, strong and italic tags.

This is achieved by extending the element tree builder so that it explicitly recognises P tags and styling to them. It resolves #11930 but it is not an exhaustive list of tags so it is possible this will need to be extended further if additional nodes are nested. We can address these as and when they arise.

Why?

To create a visual distinction in the description of a tag page header.

Screenshots

Before After
before after

@github-actions
Copy link

github-actions bot commented Jul 23, 2025

@abeddow91 abeddow91 marked this pull request as ready for review July 23, 2025 14:59
@abeddow91 abeddow91 self-assigned this Jul 23, 2025
@github-actions
Copy link

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@github-actions
Copy link

github-actions bot commented Jul 23, 2025

@abeddow91 abeddow91 force-pushed the ab/tag-page-formatting branch from c0fadec to c1d794d Compare July 23, 2025 15:27
@abeddow91 abeddow91 force-pushed the ab/tag-page-formatting branch from c1d794d to 8672ea9 Compare July 23, 2025 15:30
@domlander domlander added the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
@domlander
Copy link
Contributor

I'm not sure how easy it is to find, but what elements does frontend support? If DCR achieves parity with frontend, then I think we can close #11930

@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
@abeddow91 abeddow91 merged commit 9264f5f into main Jul 24, 2025
43 of 44 checks passed
@abeddow91 abeddow91 deleted the ab/tag-page-formatting branch July 24, 2025 10:46
@prout-bot
Copy link

Seen on PROD (merged by @abeddow91 11 minutes and 29 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing Styling In Tag Page Description

3 participants