diff options
| author | jason westover <jwestover@nvidia.com> | 2025-11-04 05:58:59 +0300 |
|---|---|---|
| committer | Jason Westover <jwestover@nvidia.com> | 2025-11-22 06:03:11 +0300 |
| commit | d36ac8a8be8636ddd0e64ce005d507b21bcdeb00 (patch) | |
| tree | 09c1103bc6318971bd662ed8cfa44cbc758d4648 /src/components/AppHeader | |
| parent | 8e38779a3623db60c47024510c479b68eb90e05b (diff) | |
| download | webui-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.vue | 46 |
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; } } |
