Skip to content

Commit 3608b04

Browse files
authored
refactor: fix eslint warnings in the cli-ui codebase (#6189)
1 parent b711a05 commit 3608b04

File tree

8 files changed

+43
-30
lines changed

8 files changed

+43
-30
lines changed

packages/@vue/cli-ui/src/components/content/TerminalView.vue

+1-1
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,7 @@ export default {
213213
if (emptySelection) {
214214
this.$_terminal.selectAll()
215215
}
216-
var selection = this.$_terminal.getSelection()
216+
const selection = this.$_terminal.getSelection()
217217
textarea.value = selection
218218
textarea.select()
219219
document.execCommand('copy')

packages/@vue/cli-ui/src/components/dashboard/Widget.vue

+5-2
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,7 @@
2424
<!-- Custom actions -->
2525
<template v-if="widget.configured">
2626
<VueButton
27-
v-for="action of headerActions"
28-
v-if="!action.hidden"
27+
v-for="action of visibleHeaderActions"
2928
:key="action.id"
3029
:icon-left="action.icon"
3130
:disabled="action.disabled"
@@ -296,6 +295,10 @@ export default {
296295
return this.widget.definition.detailsComponent
297296
}
298297
return this.widget.definition.component
298+
},
299+
300+
visibleHeaderActions () {
301+
return this.headerActions.filter(action => !action.hidden)
299302
}
300303
},
301304

packages/@vue/cli-ui/src/components/dashboard/WidgetAddPane.vue

+7-6
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,13 @@
3434
/>
3535

3636
<template v-else-if="data">
37-
<WidgetAddItem
38-
v-for="definition of data.widgetDefinitions.filter(filterDefinition)"
39-
v-if="definition.canAddMore"
40-
:key="definition.id"
41-
:definition="definition"
42-
/>
37+
<template v-for="definition of data.widgetDefinitions.filter(filterDefinition)">
38+
<WidgetAddItem
39+
v-if="definition.canAddMore"
40+
:key="definition.id"
41+
:definition="definition"
42+
/>
43+
</template>
4344
</template>
4445
</template>
4546
</ApolloQuery>

packages/@vue/cli-ui/src/components/file-diff/FileDiffChange.vue

+4
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@ export default {
4545
} else if (this.change.type === 'del') {
4646
return this.change.ln
4747
}
48+
49+
return ''
4850
},
4951
5052
ln2 () {
@@ -53,6 +55,8 @@ export default {
5355
} else if (this.change.type === 'add') {
5456
return this.change.ln
5557
}
58+
59+
return ''
5660
}
5761
},
5862

packages/@vue/cli-ui/src/components/folder/FolderExplorer.vue

+8-7
Original file line numberDiff line numberDiff line change
@@ -146,13 +146,14 @@
146146
/>
147147
</transition>
148148
<template v-if="folderCurrent && folderCurrent.children">
149-
<FolderExplorerItem
150-
v-for="folder of folderCurrent.children"
151-
v-if="showHidden || !folder.hidden"
152-
:key="folder.name"
153-
:folder="folder"
154-
@select="openFolder(folder.path)"
155-
/>
149+
<template v-for="folder of folderCurrent.children">
150+
<FolderExplorerItem
151+
v-if="showHidden || !folder.hidden"
152+
:key="folder.name"
153+
:folder="folder"
154+
@select="openFolder(folder.path)"
155+
/>
156+
</template>
156157
</template>
157158
</div>
158159

packages/@vue/cli-ui/src/components/prompt/PromptsList.vue

+9-8
Original file line numberDiff line numberDiff line change
@@ -8,14 +8,15 @@
88
>
99
<div v-if="group.id" class="group-name">{{ $t(group.id) }}</div>
1010

11-
<component
12-
v-for="prompt of group.prompts"
13-
v-if="prompt.visible"
14-
:key="prompt.id"
15-
:is="getModule(prompt)"
16-
:prompt="prompt"
17-
@answer="value => $emit('answer', { prompt, value })"
18-
/>
11+
<template v-for="prompt of group.prompts">
12+
<component
13+
v-if="prompt.visible"
14+
:key="prompt.id"
15+
:is="getModule(prompt)"
16+
:prompt="prompt"
17+
@answer="value => $emit('answer', { prompt, value })"
18+
/>
19+
</template>
1920
</div>
2021

2122
<div v-if="!prompts.length" class="vue-ui-empty">

packages/@vue/cli-ui/src/components/view/ViewNav.vue

+7-6
Original file line numberDiff line numberDiff line change
@@ -15,12 +15,13 @@
1515
class="vertical small-indicator left-indicator primary"
1616
indicator
1717
>
18-
<ViewNavButton
19-
v-for="view of views"
20-
v-if="hasProjectType(view)"
21-
:key="view.id"
22-
:view="view"
23-
/>
18+
<template v-for="view of views">
19+
<ViewNavButton
20+
v-if="hasProjectType(view)"
21+
:key="view.id"
22+
:view="view"
23+
/>
24+
</template>
2425
</VueGroup>
2526

2627
<ViewNavMore/>

packages/@vue/cli-ui/src/components/view/ViewNavButton.vue

+2
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,8 @@ export default {
6666
if (this.view.badges && this.view.badges.length) {
6767
return this.view.badges.slice().sort((a, b) => b.priority - a.priority)
6868
}
69+
70+
return []
6971
},
7072
7173
firstNotHiddenBadge () {

0 commit comments

Comments
 (0)