-
-
Notifications
You must be signed in to change notification settings - Fork 78.5k
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
Fix JS components console error "Error: <Component> is transitioning" #21743
Conversation
Nice work. |
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.
Great work @vanduynslagerp thank you 👍
Will it be merged? |
Good time to merge! ;) |
@PlugIN73 too bad Savage is taking so long. |
js/src/modal.js
Outdated
@@ -162,16 +161,17 @@ const Modal = (($) => { | |||
} | |||
|
|||
if (this._isTransitioning) { |
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.
@vanduynslagerp I think you should add a !this._isShown
check here. If you don't and hide()
is called when the modal is already hidden _isTransitioning
is set to true
and never reset to false
.
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.
Good catch ! Modification done. Thanks
@Johann-S @bardiharborow Is this PR good to merge over #21613? Seems this PR fixes the tooltip in modal issue (#21607), and other related transition issues as mentioned in #21727. |
the some problem: github |
Would be nice to have popover fixed in the beta release, please. |
…d the tooltip suddenly disappearing and reappearing when hovering on and off
afdb14e
to
02c63f5
Compare
@vanduynslagerp I rebased your branch and now your branch is up to date. I'll review your code asap |
It seems I cannot update your repository anymore, so can you update your branch and I'll merge because everything is fine @vanduynslagerp |
Does this apply to npm? I just downloaded it with npm and have this error. |
You'll have this fix on the next release of BS4 @Turqueso not Alpha6 |
@Turqueso And here is the progress for the next release just in case if you (like me) are eagerly waiting. |
I pulled the current v4-dev branch into a project in which I'm receiving a "collapse is transitioning" error (understrap/understrap-child#46) and the problem persists, even with this merge.
|
still getting this error. Any idea? |
This fix will be in our next release |
I was able to resolve this issue by removing |
@Johann-S when is your next release scheduled? We're still using static shows instead of transitions/animations. We can merge and pull from this brand but we'd rather wait for a release from your end. |
We shipped a release 4 days ago see : https://github.com/twbs/bootstrap/releases/tag/v4.0.0-beta or our blog |
Don't know how I missed that! I'll start implementation in the morning! Tx. |
beta fixed it for me |
Fixes #21727