Opened 5 years ago
Last modified 2 weeks ago
#53611 assigned defect (bug)
Font size inconsistency in the data erasure inline notice for 'additionalMessages' list items
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 7.1 | Priority: | normal |
| Severity: | minor | Version: | 5.7.2 |
| Component: | Privacy | Keywords: | has-patch |
| Focuses: | ui, css | Cc: |
Description
This in reference to the unordered list items added by plugins or other code as 'additionalMessages' with a personal data erasure admin process.
The inline notice has a <p> tag with text about the completed request. Plugins can add messages using the wp_privacy_personal_data_erasers filter. Additional messages are displayed using the unordered list markup.
Right now, the <p> tag in this notice has a smaller font size and line height, so the added items are not uniform with the default message's appearance.
I'm working on an update that will add CSS to adjust the font-size and line-height of these list items to match the paragraph.
Attachments (1)
Change History (4)
This ticket was mentioned in PR #12021 on WordPress/wordpress-develop by @masteradhoc.
2 weeks ago
#1
- Keywords has-patch added; needs-patch removed
#2
@
2 weeks ago
Hey @kimannwall
Thanks for the issue and the good documentation on where this is coming from.
I drafted a PR to fix this rather small issue for WP.
---
Steps to reproduce:
- Install WordPress with WooCommerce and the WooCommerce Smooth Generator plugin
- Use Smooth Generator to generate products and orders
- Go to Tools → Erase Personal Data
- Add a data erasure request for one of the generated customer email addresses
- Confirm the request and use Force erase personal data to process it
- Once erasure completes, observe the inline notice that appears below the row — it contains a
<p>with a summary message followed by a<ul>with WooCommerce-specific retention messages (e.g. "Personal data within order X has been retained.") - The
<li>text renders at 14px while the<p>text renders at 13px, causing a visible size inconsistency
After applying the fix it should look like this: https://snipboard.io/T4IAlj.jpg
Fix font size inconsistency in the data erasure inline notice for list items added via
additionalMessages.The inline notice rendered inside the personal data erasure list table contains a
<p>element followed by a<ul>with plugin-provided messages. The<p>correctly inheritsfont-size: 13pxfrom.notice p, but the<li>elements were inheritingfont-size: 14pxfrom the.widefat thrule, since the notice sits inside a<th colspan="5">rather than a<td>.This patch scopes the fix to notice list items only, avoiding unintended side effects on other lists within widefat tables:
.widefat td .notice ul li, .widefat th .notice ul li { font-size: 13px; line-height: 1.5em; }Trac ticket: https://core.trac.wordpress.org/ticket/53611
## Use of AI Tools
AI assistance: Used for PR description