Skip to content

fix(charts): Adjust chart tooltip padding - #3347

Merged
mcoker merged 1 commit into
patternfly:masterfrom
dlabrecq:3346-tooltip-padding
Jul 30, 2020
Merged

mcoker merged 1 commit into
patternfly:masterfrom
dlabrecq:3346-tooltip-padding

Conversation

@dlabrecq

Copy link
Copy Markdown
Member

Victory changed the way padding is applied to chart tooltips. With the latest Victory packages, the tooltip appears too large.

Fixes #3346

Before
Screen Shot 2020-07-29 at 8 26 50 PM

After
Screen Shot 2020-07-29 at 8 27 21 PM

@dlabrecq
dlabrecq requested a review from mcoker July 30, 2020 01:18
@patternfly-build

patternfly-build commented Jul 30, 2020 •

Copy link
Copy Markdown
Collaborator

Preview: https://patternfly-pr-3347.surge.sh

A11y report: https://patternfly-pr-3347-coverage.surge.sh

CSS Size Report
NameCurrentPreviousDiff %
components/TabContent/tab-content.css202 B0 B100.00
components/Drawer/drawer.css21.4 kB19.6 kB8.43
components/Tabs/tabs.css24.7 kB24.2 kB2.09
docs/components/Tabs/examples/Tabs.css302 BNaN kBNaN
patternfly-no-reset.css748.2 kB745.7 kB0.34
patternfly.css750.1 kB747.6 kB0.34
patternfly.min.css661.0 kB658.8 kB0.33
patternfly-charts.css9.8 kB9.8 kB-0.01
site/docs/components/Tabs/examples/Tabs.css302 BNaN kBNaN

@mcoker
mcoker merged commit 5b82bfb into patternfly:master Jul 30, 2020
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