Re-allow title attribute in <abbr> (#2254)

* Re-allow title attribute in <abbr>

This was accidentally removed in a6363c3a2a

Signed-off-by: Plastikmensch <plastikmensch@users.noreply.github.com>

* Add test

Add a new test to check that title attribute on <abbr> is kept.

Signed-off-by: Plastikmensch <plastikmensch@users.noreply.github.com>

---------

Signed-off-by: Plastikmensch <plastikmensch@users.noreply.github.com>
th-downstream
Plastikmensch 1 year ago committed by GitHub
parent 58b0a61705
commit 02ff7c5f3d

@ -74,6 +74,7 @@ class Sanitize
attributes: {
'a' => %w(href rel class title),
'abbr' => %w(title),
'span' => %w(class),
'blockquote' => %w(cite),
'ol' => %w(start reversed),

@ -43,6 +43,10 @@ describe Sanitize::Config do
it 'keeps a with supported scheme and no host' do
expect(Sanitize.fragment('<a href="dweb:/a/foo">Test</a>', subject)).to eq '<a href="dweb:/a/foo" rel="nofollow noopener noreferrer" target="_blank">Test</a>'
end
it 'keeps title in abbr' do
expect(Sanitize.fragment('<abbr title="HyperText Markup Language">HTML</abbr>', subject)).to eq '<abbr title="HyperText Markup Language">HTML</abbr>'
end
end
describe '::MASTODON_OUTGOING' do

Loading…
Cancel
Save