Skip to content
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

Regression in Panel Exports with INLINE resources. #2790

Closed
hokieg3n1us opened this issue Sep 28, 2021 · 3 comments · Fixed by #2794
Closed

Regression in Panel Exports with INLINE resources. #2790

hokieg3n1us opened this issue Sep 28, 2021 · 3 comments · Fixed by #2794
Labels
type: bug Something isn't correct or isn't working
Milestone

Comments

@hokieg3n1us
Copy link

Upgrading from 0.11.3 to 0.12.X, the CSS styling for many Panel components is not being exported with the INLINE resources for the HTML. This is preventing my team from being able to upgrade, since we export and serve static self-contained reports.

Minimal Code Example:

import panel as pn
from bokeh.resources import INLINE

pn.extension()

column = pn.Column()
column.append(pn.pane.Alert("WARNING", alert_type="warning"))
column.append(pn.Card(title="CARD", width=400, height=400, collapsible=False))

column.save('panel-css.html', resources=INLINE)

In 0.11.3, HTML shown below:

image

In 0.12.1 and 0.12.4:

image

Additionally, a cause for concern that I noticed, the exported HTML in 0.12.4 doubled in size.
image

@philippjfr philippjfr added the type: bug Something isn't correct or isn't working label Sep 30, 2021
@philippjfr philippjfr added this to the v0.12.5 milestone Sep 30, 2021
@philippjfr
Copy link
Member

Resolved the bug in #2794. As for the file size that's down to Bokeh now including a 1.8 MB bokeh-mathjax.js bundle. We'll have to to some work to ensure that Panel only includes the bundles that are actually used in which case the file size will probably shrink.

@philippjfr
Copy link
Member

Opened an issue about the file size issue or more specifically bokeh bundling behavior here: #2796

@hokieg3n1us
Copy link
Author

That was quick work. Thanks for that @philippjfr, and for opening a ticket for the file size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't correct or isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants