Skip to content

Commit da6f29f

Browse files
committed
fix(ui): hide DevicesDropdown if user doesn't have namespaces
1 parent 0a53ba4 commit da6f29f

File tree

3 files changed

+5
-11
lines changed

3 files changed

+5
-11
lines changed

ui/src/components/AppBar/DevicesDropdown.vue

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
<template>
22
<v-badge
3+
v-if="hasNamespaces"
34
:model-value="pendingDevicesCount > 0"
45
:content="pendingDevicesCount"
56
offset-y="-5"
@@ -346,6 +347,7 @@ const stats = computed(() => statsStore.stats);
346347
const offlineDevices = computed(
347348
() => stats.value.registered_devices - stats.value.online_devices,
348349
);
350+
const hasNamespaces = computed(() => namespacesStore.namespaceList.length > 0);
349351
const toggleDrawer = () => {
350352
isDrawerOpen.value = !isDrawerOpen.value;
351353
};
@@ -392,7 +394,7 @@ const fetchRecentDevices = async () => {
392394
};
393395
394396
onBeforeMount(async () => {
395-
if (namespacesStore.namespaceList.length === 0) return;
397+
if (!hasNamespaces.value) return;
396398
await fetchStats();
397399
await fetchPendingDevices();
398400
await fetchRecentDevices();

ui/tests/components/AppBar/__snapshots__/AppBar.spec.ts.snap

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,7 @@ exports[`AppBar Component > Renders the component 1`] = `
4343
</button>
4444
<!--teleport start-->
4545
<!--teleport end-->
46-
<div class="v-badge" size="x-small" data-test="device-dropdown-badge">
47-
<div class="v-badge__wrapper"><i class="mdi-developer-board mdi v-icon notranslate v-theme--light v-icon--size-default text-primary v-icon--clickable" role="button" aria-hidden="false" tabindex="0" aria-label="Open devices menu" data-test="devices-icon"></i>
48-
<transition-stub name="scale-rotate-transition" appear="false" persisted="false" css="true"><span class="v-badge__badge v-theme--light bg-success" style="bottom: calc(100% - 7px); left: calc(100% - 12px); display: none;" aria-atomic="true" aria-label="Badge" aria-live="polite" role="status">0</span></transition-stub>
49-
</div>
50-
</div>
46+
<!--v-if-->
5147
<!--teleport start-->
5248
<!--teleport end--><button type="button" class="v-btn v-btn--icon v-theme--light text-primary v-btn--density-default v-btn--variant-text" aria-haspopup="menu" aria-expanded="false" aria-controls="v-menu-v-14" aria-owns="v-menu-v-14" data-test="user-menu-btn"><span class="v-btn__overlay"></span><span class="v-btn__underlay"></span>
5349
<!----><span class="v-btn__content" data-no-activator=""><div data-v-09753bb1="" class="v-avatar v-theme--light bg-primary v-avatar--density-default v-avatar--variant-flat border" style="width: 1.5rem; height: 1.5rem;" email="[email protected]" data-test="user-icon"><div data-v-09753bb1="" class="v-responsive v-img v-img--booting" data-test="gravatar-image"><div class="v-responsive__sizer"></div><!----><transition-stub name="fade-transition" appear="false" persisted="false" css="true"><!----></transition-stub><!----><!----><!----><!----></div><!----><span class="v-avatar__underlay"></span>

ui/tests/layouts/__snapshots__/AppLayout.spec.ts.snap

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -308,11 +308,7 @@ exports[`App Layout Component > Renders the component 1`] = `
308308
</button>
309309
<!--teleport start-->
310310
<!--teleport end-->
311-
<div class="v-badge" size="x-small" data-test="device-dropdown-badge">
312-
<div class="v-badge__wrapper"><i class="mdi-developer-board mdi v-icon notranslate v-theme--light v-icon--size-default text-primary v-icon--clickable" role="button" aria-hidden="false" tabindex="0" aria-label="Open devices menu" data-test="devices-icon"></i>
313-
<transition-stub name="scale-rotate-transition" appear="false" persisted="false" css="true"><span class="v-badge__badge v-theme--light bg-success" style="bottom: calc(100% - 7px); left: calc(100% - 12px); display: none;" aria-atomic="true" aria-label="Badge" aria-live="polite" role="status">0</span></transition-stub>
314-
</div>
315-
</div>
311+
<!--v-if-->
316312
<!--teleport start-->
317313
<!--teleport end--><button type="button" class="v-btn v-btn--icon v-theme--light text-primary v-btn--density-default v-btn--variant-text" aria-haspopup="menu" aria-expanded="false" aria-controls="v-menu-v-20" aria-owns="v-menu-v-20" data-test="user-menu-btn"><span class="v-btn__overlay"></span><span class="v-btn__underlay"></span>
318314
<!----><span class="v-btn__content" data-no-activator=""><div data-v-09753bb1="" class="v-avatar v-theme--light bg-primary v-avatar--density-default v-avatar--variant-flat border" style="width: 1.5rem; height: 1.5rem;" email="" data-test="user-icon"><i data-v-09753bb1="" class="mdi-account mdi v-icon notranslate v-theme--light v-icon--size-default text-surface" aria-hidden="true" data-test="gravatar-placeholder"></i><!----><span class="v-avatar__underlay"></span>

0 commit comments

Comments
 (0)