WordPress.org

Make WordPress Core

Opened 3 months ago

Last modified 2 weeks ago

#42729 assigned enhancement

Permit use of 'text-transform' in safecss_filter_attr()

Reported by: danielbachhuber Owned by: juiiee8487
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Editor Keywords: needs-unit-tests good-first-bug has-patch
Focuses: Cc:

Description

As a WordPress user, I'd expect that whitelisting use of the style attribute for <span> elements would permit use of text-transform:lowercase; within the attribute.

However, doing so also requires filtering safe_style_css. Given text-transform is a text formatting property like font-style and text-decoration, it makes sense to also include it in safecss_filter_attr()'s default property set.

Attachments (1)

42729.diff (330 bytes) - added by juiiee8487 3 months ago.
I have created patch to whitelist text-transform CSS property.

Download all attachments as: .zip

Change History (5)

#1 @danielbachhuber
3 months ago

  • Keywords good-first-bug added

@juiiee8487
3 months ago

I have created patch to whitelist text-transform CSS property.

#2 @juiiee8487
3 months ago

  • Keywords has-patch added; needs-patch removed

I have created patch 42729.diff to whitelist text-transform CSS property.

#3 @DrewAPicture
2 weeks ago

  • Owner set to juiiee8487
  • Status changed from new to assigned

Hi @juiiee8487, thanks for the patch! I've assigned this good-first-bug ticket to you to mark it as "claimed" and somebody should be along to review your patch and give you some feedback on next steps. Sorry for the delay in response.

#4 @juiiee8487
2 weeks ago

Hi @DrewAPicture, No problem! and Thank you! I am happy to own first ticket :)

Note: See TracTickets for help on using tickets.