maproulette3 icon indicating copy to clipboard operation
maproulette3 copied to clipboard

Upgrade to Tailwind v3

Open jschwarz2030 opened this issue 2 years ago • 1 comments

We are currently on version 2 and need to upgrade to version 3, as well as upgrade all of the associated tailwind libraries.

We should also remove the mr- prefix, which I believe was incorporated so that we didn’t have css naming overlap with other styling libraries.

jschwarz2030 avatar Nov 20 '23 14:11 jschwarz2030

After looking into this, I think it would be nice to remove the mr- prefix but there is also a valid argument for keeping it, not to mention that it won't be trivial to assess any existing naming conflicts, remove it and edit all the relevant Tailwind classes. This may warrant more discussion.

As far as upgrading Tailwind: we have a number of legacy plugins and dependencies that need to be assessed for compatibility. Some will pretty clearly work without much issue after some simple config changes, while others may or may not be able to get along with Tailwind 3 depending on whether the classes they generate interfere. Still others, like the tailwind-grid plugin, might better be replaced by standard Tailwind grid utilities, depending on how much they are used throughout the app.

I think I'll probably split these two tasks into separate tickets or at least separate PRs, with the prefix-removal taking priority.

AndrewPhilbin avatar Jan 30 '24 00:01 AndrewPhilbin