Skip to content
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

SVG Charts for PDFs (OX-9272) #2054

Merged
merged 24 commits into from
Dec 12, 2024

Conversation

jakobvogel
Copy link
Member

@jakobvogel jakobvogel commented Dec 5, 2024

Description

Provides a framework for charts that can be rendered (via SVG) into PDFs. Currently, only pie and spider charts are supported.

Screenies

Screenshot 2024-12-05 at 02 54 49 Screenshot 2024-12-05 at 02 54 36

Additional Notes

  • This PR fixes or works on following ticket(s): OX-9272

Checklist

  • Code change has been tested and works locally
  • Code was formatted via IntelliJ and follows SonarLint & best practices

@jakobvogel jakobvogel added 🔥 Hotfix PRs that fix a major bug and can be hotfixed 🧬 Enhancement Contains new features labels Dec 5, 2024
@jakobvogel jakobvogel changed the title SVG Charts (OX-9272) SVG Charts for PDFs (OX-9272) Dec 5, 2024
Copy link
Member

@sabieber sabieber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking very nice indeed! 👌🏻
My suggestions are just nitpicking ;)

src/main/java/sirius/biz/charts/BaseChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/BaseChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/BaseChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/BaseChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/Charts.java Show resolved Hide resolved
src/main/java/sirius/biz/charts/PieChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/PieChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/SpiderChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/SpiderChart.java Outdated Show resolved Hide resolved
src/main/java/sirius/biz/charts/SpiderChart.java Outdated Show resolved Hide resolved
jakobvogel and others added 13 commits December 6, 2024 11:28
OX-9272

Co-authored-by: Sascha Bieberstein <sabieber@users.noreply.github.com>
…as requested by @sabieber during review.

OX-9272
…as requested by @sabieber during review.

OX-9272
…as requested by @sabieber during review.

OX-9272

Co-authored-by: Sascha Bieberstein <sbi@scireum.de>
…as inspired by a review comment by @sabieber.

OX-9272
…as requested by @sabieber during review,

OX-9272
…as suggested by @sabieber during review.

OX-9272
…as requested by @sabieber during review.

OX-9272
In order to make the chart more easily accessible, a new class is introduced. However, the functionality all resides in the pie chart, as a doughnut chart can degenerate to a pie chart.

The renaming was suggested by @sabieber during review. Writing adapted to BE, as is done in Wikipedia.

OX-9272
…as requested by @sabieber during review.

OX-9272
…as suggested by @sabieber during review.

OX-9272
@jakobvogel jakobvogel marked this pull request as ready for review December 11, 2024 15:42
@jakobvogel jakobvogel requested a review from sabieber December 11, 2024 15:43
Copy link
Member

@sabieber sabieber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for all the actions on my nitpicking 🙏🏻💚

…as I misunderstood @sabieber's review feedback.

OX-9272
…as implied by a review comment by @sabieber.

OX-9272
@jakobvogel jakobvogel requested a review from sabieber December 11, 2024 17:37
@jakobvogel jakobvogel merged commit 2883f89 into hotfix/dev-107.3.0 Dec 12, 2024
3 checks passed
@jakobvogel jakobvogel deleted the feature/jvo/OX-9272-SVG-Charts branch December 12, 2024 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧬 Enhancement Contains new features 🔥 Hotfix PRs that fix a major bug and can be hotfixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants