Nikurasu/elk
1
0
Fork 0
forked from Mirrors/elk
Code Pull requests Releases 2 Packages Activity

fix(ui): wrong tabindex usage (#2615)

Browse source
This commit is contained in:
Joaquín Sánchez 2024-02-24 19:13:12 +01:00 committed by GitHub
parent a3c5272e07
commit 913e2892f7
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
components/common/CommonRouteTabs.vue
View file

@ -33,7 +33,7 @@ useCommands(() => command
:to="option.to" :to="option.to"
:replace="replace" :replace="replace"
relative flex flex-auto cursor-pointer sm:px6 px2 rounded transition-all relative flex flex-auto cursor-pointer sm:px6 px2 rounded transition-all
tabindex="1" tabindex="0"
hover:bg-active transition-100 hover:bg-active transition-100
exact-active-class="children:(text-secondary !border-primary !op100 !text-base)" exact-active-class="children:(text-secondary !border-primary !op100 !text-base)"
@click="!preventScrollTop && $scrollToTop()" @click="!preventScrollTop && $scrollToTop()"
@ -46,7 +46,7 @@ useCommands(() => command
</template> </template>
<template v-if="isHydrated && moreOptions?.options?.length"> <template v-if="isHydrated && moreOptions?.options?.length">
<CommonDropdown placement="bottom" flex cursor-pointer mx-1.25rem> <CommonDropdown placement="bottom" flex cursor-pointer mx-1.25rem>
<CommonTooltip placement="top" :content="moreOptions.tooltip || t('action.more')"> <CommonTooltip placement="top" no-auto-focus :content="moreOptions.tooltip || t('action.more')">
<button <button
cursor-pointer cursor-pointer
flex flex

2
components/common/CommonTabs.vue
View file

@ -49,7 +49,7 @@ useCommands(() => command
><label ><label
flex flex-auto cursor-pointer px3 m1 rounded transition-all flex flex-auto cursor-pointer px3 m1 rounded transition-all
:for="`tab-${toValidName(option.name)}`" :for="`tab-${toValidName(option.name)}`"
tabindex="1" tabindex="0"
hover:bg-active transition-100 hover:bg-active transition-100
@keypress.enter="modelValue = option.name" @keypress.enter="modelValue = option.name"
><span ><span