tremor icon indicating copy to clipboard operation
tremor copied to clipboard

feat: Dynamic y axis width

Open mbauchet opened this issue 1 year ago • 1 comments

Description yAxisWidth prop can now take the "auto" value, so that the y axis width is calculated base on the widest tick label (with max value to 100px)

Related issue(s) #646

What kind of change does this PR introduce? (check at least one)

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New Feature (non-breaking change which adds functionality)
  • [ ] Refactor
  • [ ] Build-related changes
  • [ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

  • [ ] Yes
  • [x] No

If yes, please describe the impact and migration path for existing applications:

How has This been tested? Storybook

Screenshots (if appropriate): Capture d’écran 2023-09-17 à 08 25 35 Capture d’écran 2023-09-17 à 08 25 23 Capture d’écran 2023-09-17 à 09 24 15

The PR fulfills these requirements:

  • [x] It's submitted to the main branch
  • [x] When resolving a specific issue, it's referenced in the related issue section above
  • [x] My change requires a change to the documentation. (Managed by Tremor Team)
  • [x] I have added tests to cover my changes
  • [x] Check the "Allow edits from maintainers" option while creating your PR.
  • [x] Add refs #XXX or fixes #XXX to the related issue section if your PR refers to or fixes an issue.

mbauchet avatar Sep 17 '23 06:09 mbauchet

Any update on this?

nsantos16 avatar Oct 20 '23 15:10 nsantos16

Hi! We are not moving forward with this. Too much interference with padding/margin + font type issues. Thanks for the contribution though!

severinlandolt avatar May 15 '24 16:05 severinlandolt