address sonarqube findings

src/components/IconComboContextMenu.vue

@@ -60,6 +60,12 @@ const groups = computed(() => {
                         option.action();
                         close();
                     "
+                    @keydown.enter="
+                        option.action();
+                        close();
+                    "
+                    tabindex="0"
+                    role="button"
                 >
                     <slot
                         v-if="$slots.item"

src/components/IconContextMenu.vue

@@ -137,7 +137,10 @@ onBeforeUnmount(() => {
     >
         <div
             class="activator"
+            role="button"
+            tabindex="0"
             @click.stop="toggle"
+            @keydown.enter="toggle"
         >
             <slot
                 name="activator"

src/components/MatchCard.vue

@@ -97,11 +97,14 @@
         @update:modelValue="emit('update:modelValue', $event)"
         @statusChanged="emitStatusChanged"
     />
     <div
         class="match card"
+        role="button"
+        tabindex="0"
         :class="{ readonly }"
         @click="openMatchEditor"
+        @keydown.enter="openMatchEditor"
     >
         <div
             class="team"
             :class="{

src/components/MatchRow.vue

@@ -91,11 +91,14 @@
         @update:modelValue="emit('update:modelValue', $event)"
         @statusChanged="emitStatusChanged"
     />
     <div
         class="match row"
+        role="button"
+        tabindex="0"
         :class="{ readonly }"
         @click="openMatchEditor"
+        @keydown.enter="openMatchEditor"
     >
         <div class="time-progress">
             <span
                 v-if="match.status === 'in-progress'"

src/components/StepProgress.vue

@@ -28,6 +28,8 @@ const jumpTo = (index: number) => {
                 <div class="step">
                     <div
                         class="circle"
+                        role="button"
+                        tabindex="0"
                         :class="{
                             done: index < modelValue,
                             active: index === modelValue,
@@ -37,6 +39,7 @@ const jumpTo = (index: number) => {
                                 (canGoBack && index < modelValue),
                         }"
                         @click="jumpTo(index)"
+                        @keydown.enter="jumpTo(index)"
                     ></div>
                     <span>{{ name }}</span>
                 </div>

src/components/TournamentMatches.vue

@@ -218,8 +218,11 @@ onMounted(() => {
                     :key="option"
                     :value="option"
                     class="chip-option"
+                    role="button"
+                    tabindex="0"
                     :class="{ selected: selectedGroupOption === option }"
                     @click="selectedGroupOption = option"
+                    @keydown.enter="selectedGroupOption = option"
                 >
                     by {{ getFriendlyOptionName(option) }}
                 </div>
@@ -239,8 +242,11 @@ onMounted(() => {
                 </div>
                 <div
                     class="filter"
+                    role="button"
+                    tabindex="0"
                     :class="{ open: matchFilter?.contextMenu?.isOpen }"
                     @click.stop.prevent="matchFilter?.contextMenu?.toggle()"
+                    @keydown.enter.prevent="matchFilter?.contextMenu?.toggle()"
                 >
                     <MatchFilter
                         ref="matchFilter"

src/components/modals/ShareFull/FoldableShareOption.vue

@@ -18,7 +18,10 @@ const expanded = ref(false);
     <div class="option">
         <div
             class="header"
+            role="button"
+            tabindex="0"
             @click="expanded = !expanded"
+            @keydown.enter="expanded = !expanded"
         >
             <ion-icon :name="expanded ? 'chevron-down' : 'chevron-forward'"></ion-icon>
             <h3>{{ title }}</h3>

src/components/notifications/ClosableNotification.vue

@@ -43,8 +43,11 @@ const click = () => {
             'cursor-pointer':
                 (notification.type !== 'redirect' && notification.redirect) || notification.onClick,
         }"
+        role="button"
+        tabindex="0"
         class="notification"
         @click.stop.prevent="click"
+        @keydown.enter="click"
     >
         <div class="message">
             <h4>

src/components/notifications/YesNoNotification.vue

@@ -46,6 +46,9 @@ const click = () => {
 <template>
     <div
         class="notification"
+        role="button"
+        tabindex="0"
+        @keydown.enter="click"
         @click.stop.prevent="click"
     >
         <div class="message">
@@ -62,13 +65,19 @@ const click = () => {
         <div class="yes-no">
             <div
                 class="yes option"
+                role="button"
+                tabindex="0"
                 @click.stop.prevent="yes"
+                @keydown.enter.stop.prevent="yes"
             >
                 <ion-icon name="checkmark-outline"></ion-icon>
             </div>
             <div
                 class="no option"
+                role="button"
+                tabindex="0"
                 @click.stop.prevent="no"
+                @keydown.enter.stop.prevent="no"
             >
                 <ion-icon name="close-outline"></ion-icon>
             </div>

src/components/share/GistConfig.vue

@@ -26,7 +26,7 @@ watch(
     async () => {
         if (!gistRemote.value) return false;
         const identifier = gistRemote.value.identifier;
-        canPush.value = await accounts.findShareAccount(identifier);
+        canPush.value = accounts.findShareAccount(identifier);
     },
     { immediate: true },
 );
@@ -101,9 +101,7 @@ const share = () => {
 onMounted(() => {
     const identifier = gistRemote.value?.identifier;
     if (!identifier) return false;
-    accounts.findShareAccount(identifier).then((account) => {
-        canPush.value = account;
-    });
+    canPush.value = accounts.findShareAccount(identifier);
 });
 </script>
 

src/helpers/common.ts

@@ -135,7 +135,7 @@ export const formatCurrentMatchTime = (match: Match, tournament: Tournament): st
 };
 
 export const copyToClipboard = (text: string) => {
-    navigator.clipboard.writeText(text);
+    void navigator.clipboard.writeText(text);
 };
 
 export const nextPowerOfTwo = (n: number): number => {

src/helpers/share/gist/gist.ts

@@ -188,7 +188,7 @@ const accessTokenToAccount = async (accessToken: string) => {
     }
 };
 
-const isMine = async (identifier: string, accounts: Account[]) => {
+const isMine = (identifier: string, accounts: Account[]) => {
     try {
         const { author } = GistClient.fromShare(identifier);
         return accounts.find((x) => x.displayName === author) ?? null;

src/helpers/share/gist/pullSync.ts

@@ -72,7 +72,7 @@ export const createPullSync: PullSyncFactory<IGistPullSync> = (tournament) => {
                 date: new Date(),
             };
         },
-        async stop() {
+        stop() {
             console.log("[Gist] Stopping pull sync");
             if (this._timer) {
                 clearInterval(this._timer);

src/helpers/share/index.ts

@@ -36,7 +36,7 @@ export interface IdentifierComponents {
     mode: ShareMode;
 }
 
-export type AccountResolver = (remote: IRemote) => Promise<Account | null>;
+export type AccountResolver = (remote: IRemote) => Account | null;
 
 export interface ISimpleShareClient {
     pull: (remote: IRemote) => Promise<Import | null>;

src/helpers/share/p2p/pullSync.ts

@@ -100,7 +100,7 @@ const connectPullPeer = (context: PullContext) => {
 interface IP2PPullSync extends IPullSync {
     _peer: Peer;
     _connection: DataConnection | null;
-    _onDiff: (diff: Change[]) => Promise<void>;
+    _onDiff: (diff: Change[]) => void;
 }
 
 export const createPullSync: PullSyncFactory<IP2PPullSync> = (tournament) => {
@@ -112,7 +112,7 @@ export const createPullSync: PullSyncFactory<IP2PPullSync> = (tournament) => {
             type: "live",
             lastUpdate: new Date(),
         }),
-        async _onDiff(diff: Change[]) {
+        _onDiff(diff: Change[]) {
             if (!tournament.value) {
                 return;
             }
@@ -156,7 +156,7 @@ export const createPullSync: PullSyncFactory<IP2PPullSync> = (tournament) => {
                 }
             });
         },
-        async stop() {
+        stop() {
             console.log("[P2P] Stopping pull sync");
             this._connection?.close();
             this._connection = null;

src/helpers/share/p2p/pushSync.ts

@@ -76,12 +76,12 @@ const attachHostConnection =
     (connection: DataConnection) => {
         connections.push(connection);
 
-        connection.on("open", () => {
+        connection.on("open", async () => {
             if (!tournament.value) {
                 return;
             }
 
-            connection.send({
+            await connection.send({
                 type: "full",
                 data: deepCopy(toRaw(tournament.value)),
             });
@@ -209,9 +209,9 @@ export const createPushSync = (tournament: Ref<Tournament | null>): IPushSync =>
             return;
         }
 
-        connections.forEach((connection) => {
+        connections.forEach(async (connection) => {
             if (connection.open) {
-                connection.send({
+                await connection.send({
                     type: "diff",
                     data: changes,
                 });

src/pages/settings/general/appearance.vue

@@ -12,8 +12,11 @@ const theme = useThemeStore();
         >
             <div
                 class="theme-preview"
+                role="button"
+                tabindex="0"
                 :class="availableTheme"
                 @click="theme.set(availableTheme)"
+                @keydown.enter="theme.set(availableTheme)"
             ></div>
             <span class="theme-name">
                 {{ availableTheme }}

src/stores/persistence/indexedDb.ts

@@ -1,7 +1,7 @@
 import localforage from "localforage";
 import type { ITournamentPersistor } from "./tournamentWatcher";
 import { tournamentFromJson } from "@/helpers";
-import type { AnyTournament, Tournament } from "@/types/tournament";
+import type { AnyTournament } from "@/types/tournament";
 import { toRaw } from "vue";
 
 const KEY_PREFIX = "tournament";
@@ -12,25 +12,27 @@ const store = localforage.createInstance({
 
 export const createIndexedDbStorage = (): ITournamentPersistor => {
     const key = (tournamentId: string) => `${KEY_PREFIX}.${tournamentId}`;
+    const getValue = async (key: string) => {
+        if (!key.startsWith(KEY_PREFIX)) return null;
+
+        const item = await store.getItem(key);
+        if (item) {
+            return tournamentFromJson(item as AnyTournament);
+        }
+        return null;
+    };
 
     return {
         load: async () => {
-            const storedTournaments: Tournament[] = [];
-            for (const key of await store.keys()) {
-                if (!key.startsWith(KEY_PREFIX)) continue;
-
-                const tournament = await store.getItem(key);
-                if (tournament) {
-                    storedTournaments.push(tournamentFromJson(tournament as AnyTournament));
-                }
-            }
-            return storedTournaments;
+            const keys = await store.keys();
+            const storedValues = await Promise.all(keys.map((k) => getValue(k)));
+            return storedValues.filter((v) => v != null);
         },
         onTournamentChange: (tournament) => {
-            store.setItem(key(tournament.id), toRaw(tournament));
+            void store.setItem(key(tournament.id), toRaw(tournament));
         },
         onTournamentDeleted: (tournament) => {
-            store.removeItem(key(tournament.id));
+            void store.removeItem(key(tournament.id));
         },
     };
 };

src/views/create/BasicsView.vue

@@ -72,8 +72,11 @@ const setSport = (sportKey: string) => {
                 v-for="(v, k) in DEFAULTS"
                 :key="k"
                 class="chip-option"
+                role="button"
+                tabindex="0"
                 :class="{ selected: tournament.config.sport === k }"
                 @click="setSport(k)"
+                @keydown.enter="setSport(k)"
                 :title="v.description"
             >
                 <ion-icon

src/views/create/TemplateView.vue

@@ -48,7 +48,10 @@ const useTemplate = (template: TournamentTemplate | null) => {
     <div class="template-list">
         <div
             class="template"
+            role="button"
+            tabindex="0"
             @click="useTemplate(null)"
+            @keydown.enter="useTemplate(null)"
         >
             <ion-icon
                 name="document-text-outline"
@@ -58,9 +61,12 @@ const useTemplate = (template: TournamentTemplate | null) => {
         </div>
         <div
             class="template"
+            role="button"
+            tabindex="0"
             v-for="template in templates"
             :key="template.name"
             @click="useTemplate(template)"
+            @keydown.enter="useTemplate(template)"
         >
             <img
                 :src="logoHref(template)"