tailwindcss-fluid icon indicating copy to clipboard operation
tailwindcss-fluid copied to clipboard

Classes for padding/margin not standard

Open pandroid opened this issue 6 years ago • 1 comments

Hey @bradlc, while further working I noticed something else:

The plugin doesn't generate tailwinds standard classes for padding/margin Tailwind: .p{side?}-{size}, .m{side?}-{size}

Plugin: .p-{side?}-{size}, .m-{side?}-{size}

and it doesn't output tailwinds standard classes for vertical / horizontal side .py-sm-fluid / .px-sm-fluid / .my-sm-fluid / .mx-sm-fluid

Maybe it's obsolet because classes for top, left, right, bottom are there. But its a little confusing because of tailwinds standards.

Maybe it would be a good alternative to make it optional: Option: classes for horizontal/vertical Option: classes for top/bottom/right/left

pandroid avatar Apr 16 '18 19:04 pandroid

Nice spot @pandroid! This oversight has happened because some class names do use a hyphen (e.g. rounded-t-sm) and some don’t (e.g. pr-3).

I agree that the plugin should generate vertical and horizontal class names as well.

Thanks again for posting these issues. I have only used the plugin for text sizing so far so these issues have gone unnoticed 👍

bradlc avatar Apr 17 '18 17:04 bradlc