I don't view this as a problem that needs fixing as much as a subjective user preference.

Oh, I assumed it was a typo. I've never seen semicolons in HTML classes.

Isn't more common not to use them?

Oh, I assumed it was a typo.

No, not a typo, but I honestly don't have strong feelings either way. It seemed like it was stated as a problem, but I don't think it really matters.

Isn't more common not to use them?

It wouldn't surprise me if one was somewhat more popular than others. I know people that have definite preferences for or against them at the end.

I don't know what the split on preference is. I think at some point it used to be required, but I think these days most browsers are more forgiving. If you go online, you'll often see it taught to always add the ; probably because it used to be needed, but yeah, I know there are people who don't add them and I've probably left them off at times myself. This is probably the most thought I've put into the significance of trailing semicolons in inline styles 🙃.

I guess my point is I don't really care it's there or not, but I guess I'm not against removing it either 🤷🏻.

This is probably the most thought I've put into the significance of trailing semicolons in inline styles 🙃.

I get why you have implemented that way in the style attribute, but I'm talking about setting the element class attribute, not style.

<ol class="fancylists-lower-alpha;" type="a">
  <li>List item</li>
</ol>

With the current implementation, CSS selector should be .fancylists-lower-alpha\; instead of .fancylists-lower-alpha.

Example:

.fancylists-lower-alpha\; li::marker {
    content: counter(list-item, lower-alpha) ") ";
}
.fancylists-upper-alpha\; li::marker {
    content: counter(list-item, upper-alpha) ") ";
}

Instead of:

.fancylists-lower-alpha li::marker {
    content: counter(list-item, lower-alpha) ") ";
}
.fancylists-upper-alpha li::marker {
    content: counter(list-item, upper-alpha) ") ";
}

Oh, a semicolon in the class?! Oh, that is definitely a typo.

I completely missed that

@gir-bot lgtm

Sorry for the misunderstanding. I should have written a better PR text.

Thanks!

I should have looked over the review closer 😅

Released: https://github.com/facelessuser/pymdown-extensions/releases/tag/10.10.1