feat(ktabs): add before change prop #2644
Conversation
portikM
requested review from
adamdehaven,
jillztom,
Justineo and
a team
as code owners
✅ Deploy Preview for kongponents-sandbox ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for kongponents ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
|
||
| Prop that takes a function return value of which determines whether tab change should be skipped. Useful when some information needs to me brought to user's attention before they leave the content in the currently displayed tab. | ||
|
|
||
| The function receives new tab value as an argument and returns a boolean value, `false` will prevent tab change. Defaults to `() => true`. |
There was a problem hiding this comment.
I would suggest we also add support for async operations:
async function onBeforeTabchange() {
return await checkSomeCondition()
}Or when we add programatical use for modals:
async function onBeforeTabchange() {
return await showConfirmModal()
}There was a problem hiding this comment.
Do we have a use case for that? The use case we're trying to accommodate here is to give a way to communicate to the user that they will loose unsaved progress when switching away from the form.
There was a problem hiding this comment.
I think it's reasonable to assume the callback might need to be asynchronous
# [9.25.0](v9.24.3...v9.25.0) (2025-03-24) ### Features * **ktabs:** add before change prop ([#2644](#2644)) ([becd5af](becd5af))
|
🎉 This PR is included in version 9.25.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Summary
Adds
beforeChangeprop in KTabsContext: sometimes (especially in views that have forms) we want to prompt user and get their confirmation before they switch to a different tab and currently KTabs does not provide an easy way to do that