-
Notifications
You must be signed in to change notification settings - Fork 208
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
Bump bootstrap from 5.1.3 to 5.2.0 #729
Conversation
6e0ee5e
to
a5e7f53
Compare
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.
The behavior in 5.2.1 looks like different from 5.1.x while it is different from 5.2.0 😀 Pushing back to draft...
a5e7f53
to
4574b68
Compare
4574b68
to
3d8bda4
Compare
@tnir I fixed bootstrap 5.2.0 compatibility. Issue was introduced by twbs/bootstrap#35679, and I fixed it by forcing a disposal of the popper so that it's properly regenerated. 5.2.1 has a different issue, but I think this is fine to merge. |
3d8bda4
to
66c1f98
Compare
66c1f98
to
ff38ba3
Compare
Thanks @deivid-rodriguez for merging this 🎉 |
Bootstarap 5.2.0 (GA) and 5.2.1 was released: https://blog.getbootstrap.com/2022/07/19/bootstrap-5-2-0/ and https://blog.getbootstrap.com/2022/09/07/bootstrap-5-2-1/
Bootstrap 5.2.1 fixed the regression introduced as below in 5.2.0.
As of 2022-07-25, Bootstrap 5.2.0 had a regression as below, so we cannot upgrade to 5.2.0. Hopefully we can go with 5.2.1.getInstance
on.popover
element twbs/bootstrap#36837In 5.2.0-beta1, the popover is placed incorrectly on the second attempt twbs/bootstrap#36518Signed-off-by: Takuya Noguchi [email protected]