-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[16.0][FIX] Fix rendering when height/width is zero #2793
base: 16.0
Are you sure you want to change the base?
[16.0][FIX] Fix rendering when height/width is zero #2793
Conversation
Hi @ChrisOForgeFlow, @LoisRForgeFlow, |
e35ecd9
to
21382db
Compare
Hi @robyf70 We don't update the library version once merged in an odoo stable version, as it affect existing usages of the widget. The migration to v17 or v18 will be the oportunity to do this update. Sorrry for the inconvenience. |
Hi @LoisRForgeFlow, I agree 100% with you, however what about this case if the library is breaking the navigation and crash? |
@robyf70 Do you have a use case that you can share? Maybe you can point to the bokeh chart computation and/or share a video of how it crashes. |
Sure please see this video Peek.2024-04-09.17-29.mp4 |
As you can see I'm just getting back to the tree view using the breadcrumb and then crashes |
and forgot to mention the Bokeh PR to fix the issue which is on branch-3.2, see https://github.com/bokeh/bokeh/pull/13140/files |
@LoisRForgeFlow How to we proceed with this PR? |
@robyf70 I still think we should stick to the stable version rules and keep it as is in 16.0. However I will make sure that the bokeh version is updated in 17.0. I already pinged the author of the v17 migration asking to update bokeh version there. In your case, you can use this PR in your instalations to have the newer version and avoid the bug that you are facing. I know is not the ideal solution for you, but there is a risk of impacting many current users of the module if we do this update. I hope you understand and sorry for the inconveniences 🙏 |
The bokeh v3.1.1 has a issue when the canvas has width/height at zero. This happen when moving from a view form to tree view and the Bokeh widget is going to be destroyed. Javascript console report issue below. The fix has been introduced in v3.2.x of the library.