Page MenuHomePhabricator

New message inclusion syntax strips <span> with more than one class
Closed, ResolvedPublic

Description

We are now using <span> elements to do some of the message formatting in fundraising banners, as this was agreed to be safe. A span with a single class works fine. However adding any more classes causes the span tag to be stripped out when the message is included in another banner.

For example:

We can probably work around this if needed, but I don't see why multiple classes shouldn't be allowed.

Event Timeline

Change 494404 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[mediawiki/extensions/CentralNotice@master] Allow multiple classes on a tag in a banner message

https://gerrit.wikimedia.org/r/494404

Change 494404 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@master] Allow multiple classes on a tag in a banner message

https://gerrit.wikimedia.org/r/494404

@Pcoombe, this should be live on production now... Can you confirm that it's working? Thanks much!!!!

Looks good, thanks Andy!