Dany Castillo
Dany Castillo
Possible solution: Add optional `priority` number property to validator functions which will be used to insert validators in `classMap`. https://github.com/dcastil/tailwind-merge/blob/47155f0ebe17188991c8cafc9fe2ae034b18265c/src/lib/class-group-utils.ts#L145-L148 Instead of pushing to the validators array, I search for...
This would be a breaking change for users of tailwind-merge's validators since their execution order might change. I can only do this in a new major version.
Hey @faiz9068! Do you have a reproducible example of this? Just tried this out in https://codesandbox.io/p/sandbox/ymtvyy?file=%2Fsrc%2Findex.ts%3A17%2C28 but tailwind-merge seems to correctly remove `aspect-6/1`.
@davidhellmann sorry for my late reply! The reason for tailwind-merge not to dedup these classes is that `aspect-landscape` doesn't exist in the default Tailwind config. If you added that class...
Closing this issue as the original issue is not reproducible. Let me know in case I should reopen.
Hey @toppsdown! 👋 Sorry for the long wait, looks like I missed a bunch of notifications. From the usage example link I infer that you're using Tailwind CSS v2. The...
Ah yeah that's because tailwind-merge is configured to treat all `ring-*` classes that don't belong to another group (that includes all unrecognized classes) as a ring color class. So `ring-red-500`...
Closing this issue as resolved. Let me know if I should reopen!