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

feat: Integrate wagtailcharts #388

Open
wants to merge 5 commits into
base: feat/modern-footer
Choose a base branch
from

Conversation

mgax
Copy link
Contributor

@mgax mgax commented Mar 25, 2024

Fixes #236

⚠️ Migrations will conflict with #403. Whichever gets merged first, the migrations in the other branch should just be deleted and re-created, they are just a side-effect of the Streamfield schema being changed.

This PR integrates the wagtailcharts library with all its chart types: lines, vertical bars, horizontal bars, area, combo (lines + bars + area), pie, doughnut, waterfall, radar, and polar.

Charts include an extra Accessible label field, which is used as an aria-label attribute on the chart, for screen readers.


The branch is deployed at https://ws-wagtailcharts.dev.ietf.org/ for testing.


Here's a quick demo, with IETF attendance data, reproducing the chart from this article.

Screenshot 2024-03-25 at 16-45-39 Editing Standard page Charts demo - Wagtail

@mgax mgax marked this pull request as ready for review March 26, 2024 14:20
@mgax
Copy link
Contributor Author

mgax commented Mar 26, 2024

@kesara it would be useful to have a way to try out a branch before merging. For example, https://www.squash.io can spin up an instance of the project, based on a pull request. We've had good experiences with them, and they offer free accounts for Open Source projects. I could reach out to them if you like.

@kesara kesara changed the base branch from main to feat/modern-footer April 23, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add chart library to page editor
2 participants