Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

fix: remove sticky tooltip when query returns no data in explore view #42

Merged
merged 1 commit into from
Apr 8, 2019

Conversation

xtinec
Copy link
Contributor

@xtinec xtinec commented Apr 7, 2019

🐛 Bug Fix

Tap into the newly added willUnmount hook in apache-superset/superset-ui#131 to clean up sticky tooltip that nvd3 adds to body directly. This removes such tooltips that were left behind in the explore view with the new query returns 'no data'.
image

Tap into the newly added `willUnmount` hook to clean up sticky tooltip that nvd3 adds to `body` directly. This removes such tooltips that were left behind in the explore view with the new query returns 'no data'.
@xtinec xtinec requested a review from a team as a code owner April 7, 2019 06:26
@xtinec xtinec added #bug Something isn't working reviewable labels Apr 7, 2019
@xtinec xtinec requested review from kristw and williaster April 7, 2019 06:52
Copy link
Contributor

@williaster williaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Is there any difference between hide vs remove? Or hide does remove?

@xtinec xtinec merged commit 789596f into master Apr 8, 2019
@delete-merged-branch delete-merged-branch bot deleted the xtinec--add-willUnmountCallback-nvd3 branch April 8, 2019 17:26
@xtinec
Copy link
Contributor Author

xtinec commented Apr 8, 2019

It does either. The parameter shouldRemove determines whether to hide or to remove. 🙂

export function hideTooltips(shouldRemove) {

nytai pushed a commit to preset-io/superset-ui-plugins that referenced this pull request Apr 27, 2020
feat: Create factory functions for creating D3 formatters.
BREAKING CHANGE: Remove D3NumberFormatter class
nytai pushed a commit to preset-io/superset-ui-plugins that referenced this pull request Apr 27, 2020
…apache-superset#42)

Tap into the newly added `willUnmount` hook to clean up sticky tooltip that nvd3 adds to `body` directly. This removes such tooltips that were left behind in the explore view with the new query returns 'no data'.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
#bug Something isn't working reviewable size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants