max-attributes-per-line defaults to "singleline: 1", should be 3 according to docs

Tell us about your environment

• ESLint Version: any
• eslint-plugin-vue Version: master
• Node Version: any

What did you do? Please include the actual source code causing the issue.

I use the strongly-recommended rule that enables max-attributes-per-line. It should default to 3 attributes in the singleline case, according to the docs:

There is a configurable number of attributes that are acceptable in one-line case (default 3), as well as how many attributes are acceptable per line in multi-line case (default 1).

This should be correct:

    <div v-for="item of items" :key="item.id">
      {{ item.name }}
    </div>

There’s also a docs inconsistency, as this code is given as incorrect in the docs:

<MyComponent lorem="1" ipsum="2"/>

What did you expect to happen?

Don’t get a lint error.

What actually happened?

Got a lint error.

Also noticed by @michalsnik in https://github.com/vuejs/eslint-plugin-vue/issues/77#issuecomment-347014998

[chenxeed]

I agree with @rzaharenkov as well.

Having 2-3 attributes is still considered common and also short so it doesn’t have to be strict to be in new line. For example of these cases:

<ul class="lists" v-for="item" v-key="item.id">

<img src="" alt="title" />

<div id="appBox" class="app-wrapper">

<button class="btn btn-submit" @click="submitForm">

Our developers would like to follow the full VueJS recommended standards too, but this rules seems too strict and our HTML template doesn’t look really well after most of these elements needs to be in multi-line.

I understand that we can just override the rules with custom max-attribute-per-lines lint rules in the .eslintrc, but again we would prefer to follow the official rules set, just like @tiltec said.

Please reconsider, thank you!

[michalsnik]

Hey @tiltec, thanks for posting this. Now this topic will have it’s proper attention 😃

What do you think about this @chrisvfritz? You probably missed my proposition in bunch of other messages in the original issue 😄 If we’re going to change anything here, it has to start with the StyleGuide, so we can align with our source of truth 😃