ENH Update AttributesHTML to output alt attribute even if it's empty

silverstripe / silverstripe-framework

Silverstripe Framework, the MVC framework that powers Silverstripe CMS

https://www.silverstripe.org

BSD 3-Clause "New" or "Revised" License

720 stars 820 forks source link

Closed maxime-rainville closed 1 month ago

maxime-rainville commented 1 month ago

AttributeHTML want to suppress attributes with falsy values.

When rendering Images and the alt attribute contains a blank string, the attribute should be rendered anyway.

Arguably this should be on ImageManipulation instead, but it seems like an sensible exception to have across the board. I haven't decided yet.

maxime-rainville commented 1 month ago

Fixed the linting issue