vuetifyjs / eslint-plugin-vuetify

An eslint plugin for Vuetify

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

border migration util matches to much

adrianjost opened this issue · comments

The border migration regex matches too many classes and is therefor always attempting to migrate our custom defined class border-right-default with border-eight-default.

[/^border-([rl])(.*)$/, ([side, rest]) => {

commented

I can't actually find these in v2 at all, this rule shouldn't exist in the first place.

I think this rule referred to the following change and might still be necessary?

left and right have been replaced by start and end respectively. This applies to utility classes too, for example .border-r is now .border-e.

https://vuetifyjs.com/en/getting-started/upgrade-guide/#styles-and-utility-classes:~:text=left%20and%20right%20have%20been%20replaced%20by%20start%20and%20end%20respectively.%20This%20applies%20to%20utility%20classes%20too%2C%20for%20example%20.border%2Dr%20is%20now%20.border%2De.

commented

That's wrong too I guess:
Screenshot_20240416_154747