Skip to content
This repository was archived by the owner on Dec 30, 2022. It is now read-only.

Commit fdbf56a

Browse files
committed
perf: use unique keys in all v-for loops
1 parent 271a113 commit fdbf56a

File tree

6 files changed

+9
-9
lines changed

6 files changed

+9
-9
lines changed

src/components/Pagination.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
<slot name="previous">&lt;</slot>
1111
</a>
1212
</li>
13-
<li v-for="(item, key) in pages" :key="key" :class="[bem('item'), page === item ? bem('item', 'active') : '']" >
13+
<li v-for="item in pages" :key="item" :class="[bem('item'), page === item ? bem('item', 'active') : '']" >
1414
<a href="#" @click.prevent="goToPage(item)">
1515
<slot :value="item" :active="item === page">
1616
{{ item }}

src/components/Rating.vue

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@
1313
:max="max"
1414
:count="facet.count"
1515
>
16-
<template v-for="(n, key) in max">
17-
<span v-if="n <= facet.value" :class="bem('star')" :key="key">&#9733;</span>
18-
<span v-else :class="bem('star', 'empty')" :key="key">&#9734;</span>
16+
<template v-for="n in max">
17+
<span v-if="n <= facet.value" :class="bem('star')" :key="n">&#9733;</span>
18+
<span v-else :class="bem('star', 'empty')" :key="n">&#9734;</span>
1919
</template>
2020
&nbsp;&amp; up
2121
<span :class="bem('count')">{{facet.count}}</span>

src/components/RefinementList.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
<slot name="header"></slot>
55

6-
<div v-for="(facet, key) in facetValues" :key="key" :class="[bem('item'), facet.isRefined ? bem('item', 'active') : '']">
6+
<div v-for="facet in facetValues" :key="facet.name" :class="[bem('item'), facet.isRefined ? bem('item', 'active') : '']">
77
<label :class="bem('label')">
88
<input type="checkbox"
99
:class="bem('checkbox')"

src/components/Results.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
<slot name="header"></slot>
55

6-
<slot v-for="(result, key) in results" :result="result" :key="key">
6+
<slot v-for="result in results" :result="result" :key="result.objectID">
77
Result 'objectID': {{ result.objectID }}
88
</slot>
99

src/components/ResultsPerPageSelector.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<template>
22
<select :class="bem()" v-model="resultsPerPage">
3-
<template v-for="(option, key) in options">
4-
<option :value="option" :key="key"><slot :option="option">{{ option }}</slot></option>
3+
<template v-for="option in options">
4+
<option :value="option" :key="option"><slot :option="option">{{ option }}</slot></option>
55
</template>
66
</select>
77
</template>

src/components/SortBySelector.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<template>
22
<select :class="bem()" v-model="indexName">
33
<slot v-for="index in indices" :indexName="index.name" :label="index.label">
4-
<option :value="index.name">
4+
<option :value="index.name" :key="index.name">
55
{{ index.label }}
66
</option>
77
</slot>

0 commit comments

Comments
 (0)