-
Don't suppose anyone has noticed vuetify's usage of v-slot modifiers is invalid according to eslint-plugin-vue's valid-v-slot rule? Vuetify documentation example -> https://codepen.io/pen/?&editable=true&editors=101=https%3A%2F%2Fvuetifyjs.com%2Fen%2Fcomponents%2Fdata-tables%2F Also it isn't happy about the below either, but this i'm not too fussed about: So who is right and who is wrong, or are you both right and I shouldn't bother creating an issue about it on either of your github repositories? Discuss... 😉 |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments
-
Your codepen link is blank, but I'm assuming it's about So eslint-plugin-vue is actually wrong here, because this convention does work as expected. |
Beta Was this translation helpful? Give feedback.
-
Bugger would help if I saved the codepen after pushing it out of the documentation! But anyways, you were right about what I was getting at. I'll log the issue with eslint-plugin-vue then. See if they have a counter argument and send me back your way 😅 EDIT: issue has been logged vuejs/eslint-plugin-vue#1165 |
Beta Was this translation helpful? Give feedback.
-
I see @KaelWD is taking this up with the Vue.js team directly now, so I consider this discussion complete as far as Vue 2 is concerned. We shall see how things play out for Vue 3. 🤞 |
Beta Was this translation helpful? Give feedback.
Your codepen link is blank, but I'm assuming it's about
item.foo
slots. eslint-plugin-vue is technically correct, but v-slot is a compiler directive and actually ignores modifiers -v-slot:foo.bar
is compiled toscopedSlots: { 'foo.bar' () { ...
instead ofdirectives: [{ name: 'slot', arg: 'foo', modifiers: { bar: true } }]
So eslint-plugin-vue is actually wrong here, because this convention does work as expected.