Nested pseudo classes within pseudo elements about stylex HOT 5 OPEN

This is a valid issue. You can suppress the ESLint error for now.

from stylex.

@necolas My bad. I assumed it was a bug in the Babel plugin because it used to be a bug at one point. You're right, the best solution here is to just suppress the ESLint error.

from stylex.

Surely the better workaround is to ignore the eslint-plugin's warning? Isn't the correct syntax the one in the OP, not the one in the workaround above? I think pseudo-elements shouldn't be allowed within property object-values - that doesn't really make sense to me.

The role of the eslint-plugin should be limited to stylistic things like "sort keys", or other opinionated preferences like "don't use px values for font-size", not for validating syntax or allowed properties.

from stylex.

should this be removed from the plugin?

there is a TODO comment that reminds to remove this suggestion as well

from stylex.

should this be removed from the plugin?

With some extra checks to disallow it for anything but pseudo elements.

from stylex.