feat(ColumnChart): add alignLabels property

[skythefire]

Fixes: #3421

Thank you for your contribution! 👏

To get it merged faster, kindly review the checklist below:

Pull Request Checklist

• Reviewed the Contributing Guidelines
  • Especially the How to Contribute section
• Correct commit message style

[cla-assistant]

All committers have signed the CLA.

[Lukas742]

Hi @skythefire

thanks a lot for the contribution! Please see my review below.

---

• BarChart hardcodes 'insideRight' the same way. Both are independent wrappers around recharts' BarChart (no shared base). Please add alignLabels to BarChart as well (default 'insideRight').
• BulletChart and ComposedChart also use the ChartDataLabel component. Due to the type change a typescript error is now thrown. Please adjust the types for labelPosition in these components as well. You can check if there are build errors locally by running yarn build.

[skythefire]

@Lukas742 Thanks for the review, I've addressed your comments accordingly

[Lukas742]

Looks good to me! 🚢

Please note that I adjusted the ChartDataLabel to filter out non-Label props. I hope that's alright :)

We will merge this PR soon and include it in our next minor release.

[skythefire]

Looks good to me! 🚢

Please note that I adjusted the ChartDataLabel to filter out non-Label props. I hope that's alright :)

We will merge this PR soon and include it in our next minor release.

Looks great, I realise that I misunderstood your previous comments, so all's good!

[coveralls]

Coverage Report for CI Build 25482025905

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+0.003%) to 84.141%

Details

• Coverage increased (+0.003%) from the base build.
• Patch coverage: 3 of 3 lines across 3 files are fully covered (100%).
• No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.

---

Coverage Stats

Relevant Lines:	5841
Covered Lines:	5169
Line Coverage:	88.5%
Relevant Branches:	3403
Covered Branches:	2609
Branch Coverage:	76.67%
Branches in Coverage %:	Yes
Coverage Strength:	104153.57 hits per line

---

💛 - Coveralls

[ui5-webcomponents-react-bot]

🎉 This PR is included in version v2.22.0 🎉

The release is available on v2.22.0

Your semantic-release bot 📦🚀