-
-
Notifications
You must be signed in to change notification settings - Fork 174
feat: Add keyboard navigation support to BButtonToolbar component #2837
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
|
Co-authored-by: VividLemon <[email protected]>
VividLemon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot Similar to the btable component implement home/end button support.
- Added CODE_HOME and CODE_END imports - Implemented Home key to focus first enabled button - Implemented End key to focus last enabled button - Added comprehensive test coverage for Home/End key functionality - Maintains backward compatibility and existing behavior Co-authored-by: VividLemon <[email protected]>
Added Home/End key support to BButtonToolbar component. Home key jumps to the first enabled button, End key jumps to the last enabled button, following the same pattern as other keyboard navigation components in the library. The implementation maintains backward compatibility and includes comprehensive test coverage. (commit ad59d75) |
|
@copilot You are missing the required changes in .data.ts as it will not build without it |
Co-authored-by: VividLemon <[email protected]>
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
commit: |
* upstream/main: (32 commits) chore: release main (bootstrap-vue-next#2838) feat(BButtonToolbar): Add keyboard navigation support (bootstrap-vue-next#2837) fix(BFormSelect): modelValue not being applied during SSR (bootstrap-vue-next#2835) feat(BTable): Implement keyboard navigation (bootstrap-vue-next#2834) fix: BOffcanvas responsive SSR compatibility - defer breakpoint evaluation to client-side (bootstrap-vue-next#2833) chore: release main (bootstrap-vue-next#2831) fix(BDropdown): transition broken by inline display styles (bootstrap-vue-next#2832) feat: change useColorMode selector from 'body' to 'html' (bootstrap-vue-next#2830) chore: release main (bootstrap-vue-next#2817) feat: Add headerAttrs prop to BOffcanvas and BModal components docs: Add useScrollspy documentation fix: Remove lazy-modifier warning on FormInput docs (bootstrap-vue-next#2825) feat(BSort)!: Implement initial-sort-direction and move compare from sort-by to fields (bootstrap-vue-next#2777) docs(table): fix incorrect list item (bootstrap-vue-next#2823) docs: Fix grammar in documentation data files (bootstrap-vue-next#2816) doc(BTooltip): Parity pass (bootstrap-vue-next#2762) docs(BOverlay): Parity pass (bootstrap-vue-next#2767) feat: implement BFormRating disabled state (bootstrap-vue-next#2753) (bootstrap-vue-next#2793) docs: fix otp for index and reference pages (bootstrap-vue-next#2819) docs(BApp): add some additional details around legacy plugin use (bootstrap-vue-next#2820) ...
This PR implements keyboard navigation for the
BButtonToolbarcomponent to achieve parity with the original bootstrap-vue library.What's Changed
Key Features
keyNavprop defaults tofalse, so existing implementations are unaffectedNavigation Behavior
When
keyNavis enabled, users can navigate using:Usage
Implementation Details
The implementation follows the same patterns used in other components like
BPaginationandBTabs, ensuring consistency across the library. It uses:nextTick()for proper DOM updatesTesting
Fixes #2141.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.