The previous method was not intuitive since there was nothing in the <a> tag,
and required group-xxx classes for every element.
The new pseudo element method fixes those problems, see:
https://kittygiraudel.com/2022/04/02/accessible-cards/
The `.Params.images` and `.Site.Params.images` have existed since the creation of the files
but have never been referenced in the documentation since their initial commit (3cc1569fc1)
Also, Meta tags are not valid inside card elements
- This removes an inline event handler
- This also adds an aria-label
Using javascript instead of just using <a></a> is bad practice regarding accessibility and automated processing of web pages. Even the browser doesn't show a hint if hovering badges when using the old implementation. This in my opinion is a good way to replace inline event handlers in that code.
This greatly reduces the number of inline styles in the project and enhances maintainability by avoiding a lot of duplicate code.
This also reduces the number of hashes required for a CSP since all previous background-image inline styles are now in main.css.