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

HTML format #1797

Closed
Jerryzhuht opened this issue Aug 8, 2017 · 6 comments
Closed

HTML format #1797

Jerryzhuht opened this issue Aug 8, 2017 · 6 comments

Comments

@Jerryzhuht
Copy link

@Jerryzhuht Jerryzhuht commented Aug 8, 2017

Hi. Under the new release, there's still a minor problem of the HTML formatting. When rendering from Jupyter to HTML, there's an extra ending tag </div> in the tag of container which contains hv.extension('Bokeh'). The outcome would be the rest of content cannot be inside the container.
Just a minor one.
capture

@jlstevens
Copy link
Contributor

@jlstevens jlstevens commented Aug 8, 2017

Are you using version 1.8.2 which was released Friday last week?

Here is what I am currently seeing:

image

I recently removed the divs to fix HTML export (see #1769) so I think if you upgrade using:

conda install -c conda-forge holoviews

it should work properly. Note, you can also pip install if you wish.

@Jerryzhuht
Copy link
Author

@Jerryzhuht Jerryzhuht commented Aug 8, 2017

@jlstevens Thanks for your reply.
I already updated to the latest 1.8.2. It looks nice in Jupyter notebook, but when using File -- Download as --HTML(.html), the problems shows up in the html file(text outside container).
I believe using nbconvert gives the same outcome.

@jlstevens
Copy link
Contributor

@jlstevens jlstevens commented Aug 8, 2017

Nightmare! I thought I had fixed this. :-(

@jlstevens
Copy link
Contributor

@jlstevens jlstevens commented Aug 8, 2017

Ah, I just verified this is only an issue when logo=False so it isn't as bad as I first thought! Thanks for reporting this and it shouldn't be too hard to fix in the end...

@Jerryzhuht
Copy link
Author

@Jerryzhuht Jerryzhuht commented Aug 8, 2017

Exactly! Thanks.

@philippjfr
Copy link
Member

@philippjfr philippjfr commented Aug 25, 2017

This was fixed and has now made it into the latest release. Closing.

@philippjfr philippjfr closed this Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants