Skip to content

Commit bdcb297

Browse files
fix(a11y): use semantic html for keyword list
1 parent 4db33e9 commit bdcb297

File tree

1 file changed

+14
-14
lines changed

1 file changed

+14
-14
lines changed

app/components/Package/Card.vue

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -160,30 +160,30 @@ const numberFormatter = useNumberFormatter()
160160
</div>
161161
</div>
162162

163-
<div
163+
<ul
164164
v-if="result.package.keywords?.length"
165165
:aria-label="$t('package.card.keywords')"
166166
class="relative z-10 flex flex-wrap gap-1.5 mt-3 pt-3 border-t border-border list-none m-0 p-0 pointer-events-none items-center"
167167
>
168-
<ButtonBase
169-
v-for="keyword in result.package.keywords.slice(0, 5)"
170-
class="pointer-events-auto"
171-
size="small"
172-
:key="keyword"
173-
:aria-pressed="props.filters?.keywords.includes(keyword)"
174-
:title="`Filter by ${keyword}`"
175-
:data-result-index="index"
176-
@click.stop="emit('clickKeyword', keyword)"
177-
>
178-
{{ keyword }}
179-
</ButtonBase>
168+
<li v-for="keyword in result.package.keywords.slice(0, 5)" :key="keyword">
169+
<ButtonBase
170+
class="pointer-events-auto"
171+
size="small"
172+
:aria-pressed="props.filters?.keywords.includes(keyword)"
173+
:title="`Filter by ${keyword}`"
174+
:data-result-index="index"
175+
@click.stop="emit('clickKeyword', keyword)"
176+
>
177+
{{ keyword }}
178+
</ButtonBase>
179+
</li>
180180
<span
181181
v-if="result.package.keywords.length > 5"
182182
class="text-fg-subtle text-xs pointer-events-auto"
183183
:title="result.package.keywords.slice(5).join(', ')"
184184
>
185185
+{{ numberFormatter.format(result.package.keywords.length - 5) }}
186186
</span>
187-
</div>
187+
</ul>
188188
</BaseCard>
189189
</template>

0 commit comments

Comments
 (0)