- 🇺🇸United States luke.leber Pennsylvania
I wonder if a different, somewhat lighter (and safer), approach might be to ship default SVG assets as
data://
encoded CSS background images within thesocial_media.css
file. This would also let us improve the CLS (Content Layout Shift) for these assets by also providing an explicit height in the CSS file as well.There's already the option for module users to specify custom libraries, which could add CSS to replace the background images with custom ones.
Thoughts?
- last update
about 2 years ago 1 fail The last submitted patch, 8: 2944785-using-inline-svg-tags-instead-of-files-8.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Needs work
about 1 month ago 8:35am 3 July 2025 - 🇫🇮Finland leipomalla
Thank you for this feature! It should at least in theory make the styling of the buttons easier.
I am just wondering why I still have the svg as a file even though I installed the latest patch and have added the svg:s inline. There is not any field for the image file even but somehow it still manifests.
Any ideas?