summaryrefslogtreecommitdiff
path: root/src/components/AppHeader
diff options
context:
space:
mode:
authorjason westover <jwestover@nvidia.com>2025-11-04 05:58:59 +0300
committerJason Westover <jwestover@nvidia.com>2025-11-22 06:03:11 +0300
commitd36ac8a8be8636ddd0e64ce005d507b21bcdeb00 (patch)
tree09c1103bc6318971bd662ed8cfa44cbc758d4648 /src/components/AppHeader
parent8e38779a3623db60c47024510c479b68eb90e05b (diff)
downloadwebui-vue-d36ac8a8be8636ddd0e64ce005d507b21bcdeb00.tar.xz
Migrate to Bootstrap 5 and remove Vue compat plugin
Complete migration from Bootstrap 4 (bootstrap-vue) to Bootstrap 5 (bootstrap-vue-next) and remove the @vue/compat plugin to finalize the Vue 3 migration. Bundle size impact: - Before (Bootstrap 4 + bootstrap-vue): 535 KiB gzipped - After (Bootstrap 5 + bootstrap-vue-next): 511 KiB gzipped - Reduction: 24 KiB (4.5% smaller) Package updates: - Update bootstrap 4.6.2 -> 5.3.8 - Update bootstrap-vue 2.23.1 -> bootstrap-vue-next 0.40.8 - Remove @vue/compat plugin - Update vue 3.4.29 -> 3.5.24 and related packages - Add mitt 3.0.1 for global event bus - Add vue-demi 0.14.10 for library compatibility Bootstrap 5 CSS updates: - Replace directional classes: ml/mr/pl/pr -> ms/me/ps/pe - Replace text-left/right -> text-start/end - Replace sr-only -> visually-hidden / visually-hidden-focusable - Update media breakpoint xs -> sm (Bootstrap 5 removed xs) - Update color functions: gray("700") -> $gray-700 - Add form-switch border-radius for curved toggles - Update alert, table, toast, form, and button styles Bootstrap-Vue-Next API changes: - Use createBootstrap() for plugin registration - Update modal footer slots: #modal-footer -> #footer - Fix form select events: @change -> @update:model-value - Add v-model bindings to modals instead of manual show()/hide() - Update toast system with custom plugin wrapping useToast() - Register components and directives explicitly Vue 3 specific updates: - Replace $root.$emit with mitt event bus (eventBus.js) - Update render function from h(App) to createApp(App) - Add emits option to components - Use h() instead of $createElement in mixins - Add Vue 3 compile-time feature flags with documentation - Update event listeners: $on/$off to eventBus methods - Add beforeUnmount cleanup for event listeners New components and significant additions: - src/plugins/toast.js - Custom toast plugin wrapping useToast() for Options API compatibility - src/components/Global/ConfirmModal.vue - Global confirmation dialog shim to replace Bootstrap 4's removed bvModal.msgBoxConfirm - src/eventBus.js - mitt-based event bus with Vue 2-compatible API - Navigation state preservation on page refresh implemented Critical fixes: - Add global API interceptor to strip Vue reactivity from payloads - Preserve binary data (File, Blob, FormData) in API requests - Fix Generate CSR modal v-model binding for proper open/close - Remove debug logging and fix jest configuration - Fix responsive text visibility in AppHeader - Update BVTableSelectableMixin for proper row selection - Fix BVToastMixin VNode rendering for Vue 3 Vue 3 modal fixes (lazy-loaded components): - Add v-model support to network modals (ModalIpv4, ModalIpv6, ModalDns, ModalHostname, ModalMacAddress, ModalDefaultGateway) by adding modelValue prop, watcher on modelValue that triggers show(), and update:modelValue emit in resetForm - Remove lazy loading from TableIpv4, TableIpv6, TableDns to ensure modal component refs are available when v-model triggers - Fix modal title accessibility by adding title prop to modals (ModalAddDestination, ModalUser, ModalAddRoleGroup, etc.) i18n fixes (computed properties): - Fix computed properties using i18n translations in ModalAddRoleGroup, ModalUser, and ModalUploadCertificate - Move useI18n() call from data() to setup() and return i18n object - Use i18n.t() instead of $t in computed properties and templates - Prevents "this.$t is not a function" and "_ctx.$t is not a function" errors in Vue 3 Toast notification fixes: - Fix toast progress bar visibility by setting progressProps to undefined (documented way to opt-out) instead of false - Change modelValue prop to interval for auto-dismiss timing - Remove temporary CSS display:none hack from _toasts.scss Network settings fixes: - Fix checkbox @change event sending Vue reactive proxy object instead of boolean by casting with !! operator in changeDomainNameState and related methods in NetworkGlobalSettings.vue - Ensures API receives plain boolean values in PATCH requests Navigation fixes: - Fix nav-link styling for navigation items without children by replacing b-nav-item with router-link in AppNavigation.vue - Prevents blue font color from .nav-link CSS class Configuration updates: - Remove vue-compat webpack configuration - Add Vue 3 feature flags (__VUE_OPTIONS_API__, etc.) - Add .cursor to .gitignore Accessibility improvements: - Add autocomplete attributes to password and credential inputs - Add modal title props for screen reader support Build completes successfully and UI behavior matches pre-migration. Extracted features (to be submitted in follow-up PRs): The following features were removed from this migration PR to keep it focused on the Bootstrap 5 upgrade. They will be submitted separately: 1. UnresponsiveModal - Server connectivity watchdog with auto-retry 2. Auth token persistence - sessionStorage support for X-Auth-Token 3. Hardware store error handling - try/catch, dynamic discovery 4. Login page connecting indicator - Backend polling with spinner 5. Test updates - Jest setup and snapshot updates for Bootstrap-Vue-Next 6. Documentation updates - Vue 3 and Vue I18n v9+ API documentation 7. Enhanced ConfirmModal - Feature-rich confirmation dialog with custom actions Change-Id: Ib76a58f324b3c926cf536e6e4626e4271639de38 Signed-off-by: Jason Westover <jwestover@nvidia.com>
Diffstat (limited to 'src/components/AppHeader')
-rw-r--r--src/components/AppHeader/AppHeader.vue46
1 files changed, 26 insertions, 20 deletions
diff --git a/src/components/AppHeader/AppHeader.vue b/src/components/AppHeader/AppHeader.vue
index 89d561e81..5b0072b53 100644
--- a/src/components/AppHeader/AppHeader.vue
+++ b/src/components/AppHeader/AppHeader.vue
@@ -31,7 +31,7 @@
</b-button>
<b-navbar-nav>
<b-navbar-brand
- class="mr-0"
+ class="me-0"
to="/"
data-test-id="appHeader-container-overview"
>
@@ -42,15 +42,15 @@
:alt="altLogo"
/>
</b-navbar-brand>
- <div v-if="isNavTagPresent" :key="routerKey" class="pl-2 nav-tags">
+ <div v-if="isNavTagPresent" :key="routerKey" class="ps-2 nav-tags">
<span>|</span>
- <span class="pl-3 asset-tag">{{ assetTag }}</span>
- <span class="pl-3">{{ modelType }}</span>
- <span class="pl-3">{{ serialNumber }}</span>
+ <span class="ps-3 asset-tag">{{ assetTag }}</span>
+ <span class="ps-3">{{ modelType }}</span>
+ <span class="ps-3">{{ serialNumber }}</span>
</div>
</b-navbar-nav>
<!-- Right aligned nav items -->
- <b-navbar-nav class="ml-auto helper-menu">
+ <b-navbar-nav class="ms-auto helper-menu">
<b-nav-item
to="/logs/event-logs"
data-test-id="appHeader-container-health"
@@ -220,11 +220,17 @@ export default {
this.getEvents();
},
mounted() {
- this.$root.$on(
+ require('@/eventBus').default.$on(
'change-is-navigation-open',
(isNavigationOpen) => (this.isNavigationOpen = isNavigationOpen),
);
},
+ beforeUnmount() {
+ require('@/eventBus').default.$off(
+ 'change-is-navigation-open',
+ this.handleNavigationChange,
+ );
+ },
methods: {
getSystemInfo() {
this.$store.dispatch('global/getSystemInfo');
@@ -239,11 +245,11 @@ export default {
this.$store.dispatch('authentication/logout');
},
toggleNavigation() {
- this.$root.$emit('toggle-navigation');
+ require('@/eventBus').default.$emit('toggle-navigation');
},
setFocus(event) {
event.preventDefault();
- this.$root.$emit('skip-navigation');
+ require('@/eventBus').default.$emit('skip-navigation');
},
},
};
@@ -270,7 +276,7 @@ export default {
.navbar-text,
.nav-link,
.btn-link {
- color: color('white') !important;
+ color: $white !important;
fill: currentColor;
padding: 0.68rem 1rem !important;
@@ -299,13 +305,13 @@ export default {
.helper-menu {
@include media-breakpoint-down(sm) {
- background-color: gray('800');
+ background-color: $gray-800;
width: 100%;
justify-content: flex-end;
.nav-link,
.btn {
- padding: $spacer / 1.125 $spacer / 2;
+ padding: calc(#{$spacer} / 1.125) calc(#{$spacer} / 2);
}
.nav-link:focus,
@@ -315,8 +321,8 @@ export default {
}
.responsive-text {
- @include media-breakpoint-down(xs) {
- @include sr-only;
+ @include media-breakpoint-down(sm) {
+ @include visually-hidden;
}
}
}
@@ -334,12 +340,12 @@ export default {
}
.nav-tags {
color: theme-color-level(light, 3);
- @include media-breakpoint-down(xs) {
- @include sr-only;
+ @include media-breakpoint-down(sm) {
+ @include visually-hidden;
}
.asset-tag {
@include media-breakpoint-down($responsive-layout-bp) {
- @include sr-only;
+ @include visually-hidden;
}
}
}
@@ -364,7 +370,7 @@ export default {
}
&.open {
- background-color: gray('800');
+ background-color: $gray-800;
}
@include media-breakpoint-up($responsive-layout-bp) {
@@ -388,13 +394,13 @@ export default {
}
.navbar-brand {
- padding: $spacer/2;
+ padding: calc(#{$spacer} / 2);
height: $header-height;
line-height: 1;
&:focus {
box-shadow:
inset 0 0 0 3px $navbar-color,
- inset 0 0 0 5px color('white');
+ inset 0 0 0 5px $white;
outline: 0;
}
}