Skip to content

Commit f8267f2

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

File tree

1 file changed

+23
-21
lines changed

1 file changed

+23
-21
lines changed

app/components/Package/Card.vue

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -160,30 +160,32 @@ 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>
180-
<span
181-
v-if="result.package.keywords.length > 5"
182-
class="text-fg-subtle text-xs pointer-events-auto"
183-
:title="result.package.keywords.slice(5).join(', ')"
184-
>
185-
+{{ numberFormatter.format(result.package.keywords.length - 5) }}
186-
</span>
187-
</div>
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>
180+
<li>
181+
<span
182+
v-if="result.package.keywords.length > 5"
183+
class="text-fg-subtle text-xs pointer-events-auto"
184+
:title="result.package.keywords.slice(5).join(', ')"
185+
>
186+
+{{ numberFormatter.format(result.package.keywords.length - 5) }}
187+
</span>
188+
</li>
189+
</ul>
188190
</BaseCard>
189191
</template>

0 commit comments

Comments
 (0)