dcastil/tailwind-merge

Add support for Tailwind 3.4's `size-*`

ahkhanjani opened this issue · 5 comments

ahkhanjani commented

size-* should override both h-* and w-* classes.

dcastil commented

I'll add support for all Tailwind CSS v3.4 features in the coming days.

  • ❤️5
github-actions commented

This was addressed in release v2.2.0.

imekachi commented

@dcastil Could you take a look at this test case?

expect(twMerge('w-5 size-10 w-12 h-3')).toBe('w-12 h-3')
image

Since w and h override every property that size provides, should size be removed?
For some reason, it's not.

dcastil commented

Hey @imekachi! 👋

This is intentional and shouldn't cause any issues: w-12 and h-3 would get applied in CSS although the size-10 class is present. I wrote an explanation to a similar question in #156 (comment) which also applies here.

  • 👍1
imekachi commented

Hey @imekachi! 👋

This is intentional and shouldn't cause any issues: w-12 and h-3 would get applied in CSS although the size-10 class is present. I wrote an explanation to a similar question in #156 (comment) which also applies here.

I see. It makes sense now. Thanks for the explanation.

  • ❤️1