AntiSamy sanitizer - add support for "ol[start]"

Václav Suchánek, modified 1 Year ago. Junior Member Posts: 26 Join Date: 8/15/18 Recent Posts

Hello,

It would be nice to add support for "start" attribute within "ol" tag:
https://www.w3schools.com/tags/tag_ol.asp

There is support only for "type" attribute:
https://github.com/liferay/liferay-portal/blob/master/modules/apps/portal-security/portal-security-antisamy/src/main/resources/META-INF/resources/sanitizer-configuration.xml#L815

Now we have to make an override for "com.liferay.portal.security.antisamy.jar", but if it is a standard HTML5 attribute - why it is not added by default?

Thanks,

Best Regards,

Václav

thumbnail
Václav Suchánek, modified 1 Year ago. Liferay Master Posts: 764 Join Date: 1/5/10 Recent Posts

Hi Václav,

I can only guess. I think it may have been left out because the start attribute of the ol tag was being deprecated for a while before it was brought back.

Watch https://liferay.atlassian.net/browse/LPD-21192.

Regards,
Zsigmond

Václav Suchánek, modified 1 Year ago. Junior Member Posts: 26 Join Date: 8/15/18 Recent Posts

Thank you, Zsigmond.

thumbnail
Zsigmond Rab, modified 1 Year ago. Liferay Master Posts: 764 Join Date: 1/5/10 Recent Posts

Hi Václav,

In the meantime, we decided to handle this as a bug: https://liferay.atlassian.net/browse/LPD-21547

Regards,
Zsigmond

Václav Suchánek, modified 1 Year ago. Junior Member Posts: 26 Join Date: 8/15/18 Recent Posts

It is great. Thank you for your fast response.