New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix legend size #4272
Fix legend size #4272
Conversation
…e chart got scaled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution! This seems to be a great solution. Could you please add a test to make sure we do not break this feature again?
Hi Nikolasrieble, I tried to add unit test but found it hard to propose a reasonable to test it. The main reason is that the |
Hello @zhonglin94 please see here for examples how to mock |
Thank you so much for help. I add unit test to cover my changes. |
@PavelVanecek this change is targeting |
Well we need another PR for 3.x. There will be conflicts. This is unfortunately the price for having two live branches. |
Ack, got it. I can port this to 3.x |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@zhonglin94 if you want to make the same PR to the 3.x branch feel free! Otherwise I'll get around to it eventually |
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [recharts](https://togithub.com/recharts/recharts) | [`2.12.2` -> `2.12.3`](https://renovatebot.com/diffs/npm/recharts/2.12.2/2.12.3) | [![age](https://developer.mend.io/api/mc/badges/age/npm/recharts/2.12.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/recharts/2.12.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/recharts/2.12.2/2.12.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/recharts/2.12.2/2.12.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>recharts/recharts (recharts)</summary> ### [`v2.12.3`](https://togithub.com/recharts/recharts/releases/tag/v2.12.3) [Compare Source](https://togithub.com/recharts/recharts/compare/v2.12.2...v2.12.3) Some more small changes/fixes while working on 3.x #### What's Changed ##### Fix - `Legend`: fix issue where Legend was not taken into account when scaling the chart container by [@​zhonglin94](https://togithub.com/zhonglin94) in [recharts/recharts#4272 closes [recharts/recharts#4246 - `Area`: fixed a bug where className was not assigned to areaDot by [@​108yen](https://togithub.com/108yen) in [recharts/recharts#4294 closes [recharts/recharts#4290 - `RadialBar`: address regression where radial bar and its background were off from eachother because of rounding by [@​ckifer](https://togithub.com/ckifer) in [recharts/recharts#4295 closes [recharts/recharts#4264 - `ErrorBar`: do not count `null` as 0 in error bar domain by [@​rinkstiekema](https://togithub.com/rinkstiekema) in [recharts/recharts#4297 #### New Contributors - [@​zhonglin94](https://togithub.com/zhonglin94) made their first contribution in [recharts/recharts#4272 **Full Changelog**: recharts/recharts@v2.12.2...v2.12.3 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/SAP/ui5-webcomponents-react). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNDUuMCIsInVwZGF0ZWRJblZlciI6IjM3LjI0NS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Description
The chart shrinks when I apply a scale on the chart container
Related Issue
The issue
Motivation and Context
The legend size(height and width) provided by the
getBoundingClientRect
is changed if it's parent container is transformed using scale. If the container scale is 3x, then the legend height is significantly larger than usual, which makes the chart content visibly smaller than expected.How Has This Been Tested?
Firstly, I tested locally using the Recharts demo
Then I build a package and test in a third party application that facing the same problem
Screenshots (if appropriate):
Types of changes
Checklist: