Bleach: Allow math attributes on <mn> elements

RESOLVED FIXED

Status

Mozilla Developer Network
Wiki pages
--
minor
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: fscholz, Unassigned)

Tracking

({in-triage})

Details

(Reporter)

Description

2 years ago
Discovered by our recent MathML documentation contributor "Draussenduscher".

<mn> elements can't have any attributes at the moment. 

Probably, this line should have been ALLOWED_ATTRIBUTES['mn'] instead of ALLOWED_ATTRIBUTES['mi']:

https://github.com/mozilla/kuma/blob/master/kuma/wiki/constants.py#L109

I can create a PR for this, if no one beats me to it.
(Reporter)

Comment 1

2 years ago
PR https://github.com/mozilla/kuma/pull/3756
(Reporter)

Updated

2 years ago
Severity: normal → minor
Keywords: in-triage

Comment 2

2 years ago
Commits pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/80910211b1254a62b976c47f80be167a480727e3
Fix bug 1239021: Correct ALLOWED_ATTRIBUTES for MathML <mn> element

https://github.com/mozilla/kuma/commit/d5e37fc656e5818d2e838ea5c685a6dd37e21fa5
Merge pull request #3756 from Elchi3/bug1239021

Fix bug 1239021: Correct ALLOWED_ATTRIBUTES for MathML <mn> element

Updated

2 years ago
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.