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

fix: GH issue 8669 NVD3 tooltip overflow #278

Merged
merged 5 commits into from
Dec 3, 2019

Conversation

rusackas
Copy link
Member

@rusackas rusackas commented Dec 2, 2019

🐛 Bug Fix

This addresses Superset issue 8669:
apache/superset#8669

It also makes the headers bold, which hopefully looks better to folks.

Before:
image

After:
image

@rusackas rusackas requested a review from a team as a code owner December 2, 2019 22:03
@netlify
Copy link

netlify bot commented Dec 3, 2019

Deploy preview for superset-ui-plugins ready!

Built with commit 1c4a3bc

https://deploy-preview-278--superset-ui-plugins.netlify.com

Copy link
Contributor

@etr2460 etr2460 left a comment

Choose a reason for hiding this comment

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

lgtm with one nit

.nvtooltip tbody tr:not(:first-child) td:nth-child(2){
word-break: break-word;
}
/* END tooltip styles */
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: add a new line

Copy link
Member Author

Choose a reason for hiding this comment

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

Committed. I wish people didn't object to using Husky to add pre-commit hooks running Prettier

Copy link
Contributor

@williaster williaster Dec 3, 2019

Choose a reason for hiding this comment

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

who objects to husky pre-commit hooks? I support that 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

on second thought, I'm not so sure. we've had these in other repos and the major downside of pre-commit hooks is that they can take longer and longer to run, especially as the repo grows.

I think you can setup your ide to auto-fix eslint (which calls prettier) changes on save.

@codecov
Copy link

codecov bot commented Dec 3, 2019

Codecov Report

Merging #278 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #278   +/-   ##
======================================
  Coverage    36.9%   36.9%           
======================================
  Files          12      12           
  Lines         233     233           
  Branches       24      24           
======================================
  Hits           86      86           
  Misses        135     135           
  Partials       12      12

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9a4ceff...1c4a3bc. Read the comment docs.

@etr2460 etr2460 merged commit 8586cc9 into apache-superset:master Dec 3, 2019
@rusackas rusackas deleted the nvd3-tooltip-overflow-fix branch December 4, 2019 20:14
nytai pushed a commit to preset-io/superset-ui-plugins that referenced this pull request Apr 27, 2020
* fix: nvd3 tooltip overflow

* fix: fix build

* Revert "fix: fix build"

This reverts commit 44532e9.

* build: updating build-config package

* fix: adds newline to css file
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants