charts

helm lint only catches if a chart name is missing. You can see that in the following snippet...

https://github.com/helm/helm/blob/292368a64d6c9794b4573afa5a8dc6e940a21045/pkg/lint/rules/chartfile.go#L106-L111

The chartutil package has the ability to validate names in some forms. We could provide better hints (like no capital letters) via lint.

Output of helm version: 3.7

Output o

• Victory version 36.1.0
• React version 17.0.2

I'm trying to add some data-testid to Victory components in order to do some tests with Cypress. I have tried with several components: VictoryChart, VictoryBar, VictoryContainer. They never pass down the data-testid even if this prop is accepted by TypeScript.

Could be very useful to have this feature, or at list to be able to passe down

it's becoming more time-consuming and error-prone to manually re-test all the demos following internal refactorings and API adjustments.

now that the API is fleshed out a bit, it's possible to test a large amount of code (non-granularly) without having to simulate all interactions via Puppeteer or similar.

a lot of code can already be regression-tested by simply running all the demos and val

.rickshaw_graph .detail .item.active from detail.css would benefit by adding 'width: auto; height:auto;' to auto-size the detail tooltip on graph mouse hover. In it's current form, the tooltip doesn't get auto-sized when using a relative/percentage based layout.

I use these charts on a webpage with other content. As people scroll down the page, if the mouse goes over the chart, the chart stops the page from scrolling and instead starts zooming the chart.

I know I can disable the zooming on the chart, but instead I'd like to change the key binding for zooming to 'cmd + mousewheel'.

That way a user has to explicitly hold down the cmd key when they wan

Hello and Thanks for AMAZING package.
It would be great if we could add some border styling for tooltip. until you guys handle tooltip with widget.
Thanks a lot.