Flo Edelmann

Results 458 comments of Flo Edelmann

Ah, I reckoned there was a similar rule, but couldn't find it either. I also support to deprecate `new-line-between-multi-line-property` and add a new rule with options for different scopes. My...

> We cannot add an option to specify the number of rows in the empty lines. That suggestion was based on the current options of `new-line-between-multi-line-property`, where we already allow...

> The number specified in the options of the `new-line-between-multi-line-property` rule is the minimum number of lines to be judged as multiple property. It's not the number of empty lines....

@barthy-koeln's [suggestion](https://github.com/vuejs/eslint-plugin-vue/issues/1817#issuecomment-1072268655) goes in a similar direction as [the one that I rejected during writing](https://github.com/vuejs/eslint-plugin-vue/issues/1817#issuecomment-1072267662). So maybe it's worth to go that route. Combining [my original suggestion](https://github.com/vuejs/eslint-plugin-vue/issues/1817#issuecomment-1072218413) with @barthy-koeln's: 1\....

@ota-meshi @barthy-koeln What do you think about [my last suggestion](https://github.com/vuejs/eslint-plugin-vue/issues/1817#issuecomment-1072292779)? I think if you both give it a thumbs-up, then it's ready to be implemented. @inker Thanks for linking #1390....

I don't have any further notes, the suggestions looks fine.

@przemyslawjanpietrzak Thanks for sharing! Feel free to submit a pull request to add this to the official eslint-plugin-vue!

> I think it would be great if could check them out with a plugin 👍. The sample code you provided is very hard to read. Sorry, I don't understand....

For this code: ```vue ``` * Volar complains about the second ``: `v-else/v-else-if has no adjacent v-if or v-else-if.` * eslint-plugin-vue ([vue/attributes-order](https://eslint.vuejs.org/rules/attributes-order.html)) complains about the second ``: `Attribute "v-for" should...

Oh sorry, the code above is indeed from a Vue 2 environment.