Skip to content

Conversation

ileasile
Copy link
Collaborator

This commit fixes the following bug.

Steps to reproduce:

  1. Execute %use dataframe
  2. Calculate and render some dataframe-like table which is rendered with small arrows. It will be rendered correctly
  3. Re-execute cell with %use
  4. Arrows in the table now become giant

Reason:
When you re-execute %use command, it effectively clears all the output including CSS-s. The disappearing of styles leads to different visual problems including this one.

Fix:
Include CSS to the output for every dataframe-like object

Possible problems:
toHTML is a public function, and its API is broken with this commit. If it's unacceptable, consider another solution

This commit fixes the following bug.

Steps to reproduce:
1. Execute %use dataframe
2. Calculate and render some dataframe-like table which is rendered with small arrows. It will be rendered correctly
3. Re-execute cell with %use
4. Arrows in the table now become giant

Reason:
When you re-execute %use command, it effectively clears all the output including CSS-s. Disappearing of styles leads to different visual problems including this one.

Fix:
Include CSS to the output for every dataframe-like object
@ileasile ileasile requested a review from koperagen September 13, 2022 16:51
@koperagen koperagen merged commit 34d8388 into master Sep 16, 2022
@Jolanrensen Jolanrensen added this to the 0.9.0 milestone Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants