Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new RadarChart control to the flet-charts package, expanding the available chart types with radar/spider chart functionality. The implementation includes both Python API components and Flutter visualization logic, along with comprehensive documentation and examples.
- Adds RadarChart with customizable data sets, titles, and styling options
- Implements supporting classes for radar data management and events
- Updates documentation structure and examples to include the new chart type
Reviewed Changes
Copilot reviewed 28 out of 29 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/python/pyproject.toml | Expands known-first-party packages list for proper import sorting |
| sdk/python/packages/flet/src/flet/controls/border.py | Improves documentation formatting and error handling consistency |
| sdk/python/packages/flet/mkdocs.yml | Reorganizes navigation structure and adds RadarChart documentation entries |
| sdk/python/packages/flet-charts/src/flutter/flet_charts/lib/src/utils/radar_chart.dart | Implements Flutter utilities for radar chart parsing and event handling |
| sdk/python/packages/flet-charts/src/flutter/flet_charts/lib/src/radar_chart.dart | Core Flutter implementation of the RadarChart control |
| sdk/python/packages/flet-charts/src/flet_charts/radar_chart.py | Python API for RadarChart with validation and configuration options |
| sdk/python/packages/flet-charts/src/flet_charts/radar_data_set.py | Data structures for radar chart entries and datasets |
| sdk/python/examples/controls/charts/radar_chart/example_1.py | Comprehensive example demonstrating RadarChart usage |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Deploying flet-docs with
|
| Latest commit: |
6b5df1e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e9f2412d.flet-docs.pages.dev |
| Branch Preview URL: | https://radar-chart.flet-docs.pages.dev |
Added 3D, animated, and event-handling chart examples to the Matplotlib chart examples directory. Renamed example files for clarity, updated associated media images, and removed unused images. Enhanced MatplotlibChart class with docstrings and refactored resize handler. Updated documentation to include new examples and images, and adjusted mkdocs.yml to improve API documentation filtering.
Refactored MatplotlibChart, MatplotlibChartWithToolbar, and PlotlyChart to gracefully handle missing optional dependencies by deferring import errors and providing clearer error messages. Added runtime checks to ensure required packages are installed before usage, improving robustness and user experience.
The _require_plotly() call was moved from before_update() to the init() method to ensure plotly is available during initialization rather than before updates.
Moved the call to _require_matplotlib() from build() to a new init() method in MatplotlibChart. This separates initialization from build logic for better code organization.
FeodorFitsner
approved these changes
Oct 16, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by Sourcery
Add a full-fledged RadarChart feature to the flet-charts package, including data set and title controls, Flutter rendering, event support, documentation, and examples, while improving border validation and candlestick chart properties.
New Features:
Enhancements: