Tailwind CSS class names are removed `hover:`

[docapotamus]

Tailwind CSS class names are removed `hover:`

[a-h]

Sorry about the issue, it looks like it was an oversight during the addition of optional CSS class rendering functionality. The tests all had CSS class names that didn't trigger the CSS safety features, hence missing it.

I've done a PR, does that look like what you were expecting?

For background, CSS class names are sanitised by default, since templ can't tell whether they came from a user controlled variable (unsafe), or a programmer controlled constant (safe).

It looks like you spotted that you have to mark Go variables that you've set yourself (i.e. programmer controlled) as being safe by converting them to a templ.ConstantCSSClass type using the templ.SafeClass function.

But... I missed the code to handle that specific KV generic type.

Thanks for the great bug report and the nice words! 😁

[docapotamus]

@a-h glad the issue with using SafeClass in KV is resolved. Looking at the PR, I realised that I should have been using SafeClass rather than SafeCSS in the string element and that resolved my issue in that case too.

Thanks for the help. Once I've got a bit further with the conversion to templ I'll try and add some extra docs for CSS,

[docapotamus]

@a-h thanks for quick response and the PR. It looks good to me, I'll check out that version in the code base later this morning and report back.