Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[Vue][Bug] Long v-if condition break the code after reformat

See original GitHub issue

Prettier 1.15.2 Vue 2.5.17 Playground link

# Options (if any):
--parser vue

Input: A v-if is used on a template element which is also declared as a slot.

<template>
  <div>
    <template 
      slot="row_expander"
      slot-scope="{row}"
      v-if="longCondition1 && anotherLongCondition === 'my awesome result' && anotherWhichBreakTheLineLength">
      <something />
      <something/>
    </template>
  </div>
</template>

Output:

<template>
  <div>
    <template
      slot="row_expander"
      slot-scope="{
        row
      }"
      v-if="
        longCondition1 &&
          anotherLongCondition === 'my awesome result' &&
          anotherWhichBreakTheLineLength
      "
    >
      <something /> <something />
    </template>
  </div>
</template>

In this particular case (usage on template element which is a slot renderer), the resulting code will break.

Expected behavior: The code must be encapsulated in parenthesis to properly work. (And indentation may be better imho.)

<template>
  <div>
    <template
      slot="row_expander"
      slot-scope="{
        row
      }"
      v-if="(
        longCondition1 &&
        anotherLongCondition === 'my awesome result' &&
        anotherWhichBreakTheLineLength
     ) "
    >
      <something /> <something />
    </template>
  </div>
</template>

Issue Analytics

State:
Created 5 years ago
Comments:5 (4 by maintainers)

Top GitHub Comments

1reaction

ikatyangcommented, Nov 27, 2018

As a workaround, you could use :

Prettier 1.15.2 Playground link

--parser vue

Input:

<template>
  <div>
    <!-- prettier-ignore-attribute v-if -->
    <template 
      slot="row_expander"
      slot-scope="{row}"
      v-if="longCondition1 && anotherLongCondition === 'my awesome result' && anotherWhichBreakTheLineLength">
      <something />
      <something/>
    </template>
  </div>
</template>

Output:

<template>
  <div>
    <!-- prettier-ignore-attribute v-if -->
    <template
      slot="row_expander"
      slot-scope="{
        row
      }"
      v-if="longCondition1 && anotherLongCondition === 'my awesome result' && anotherWhichBreakTheLineLength"
    >
      <something /> <something />
    </template>
  </div>
</template>

0reactions

ikatyangcommented, Dec 1, 2018

Fixed in vuejs/vue#9119.