Skip to content

Conversation

@ndonkoHenri
Copy link
Contributor

@ndonkoHenri ndonkoHenri commented Nov 9, 2025

Fix #5776

Summary by Sourcery

Fix polyline rendering issue in Flet map and standardize enum documentation

Bug Fixes:

  • Render polylines by querying "polylines" and filtering "PolylineMarker" instead of polygons in PolylineLayerControl

Enhancements:

  • Improve and standardize enum docstrings for ImageRepeat and VisualDensity in Python Flet package

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We've reviewed this pull request using the Sourcery rules engine

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 9, 2025

Deploying flet-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0fd9108
Status: ✅  Deploy successful!
Preview URL: https://a8c25517.flet-docs.pages.dev
Branch Preview URL: https://fix-polylinelayer.flet-docs.pages.dev

View logs

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR includes documentation improvements for enum values and a bug fix in the polyline layer control. The changes enhance code clarity and correct incorrect references.

  • Documentation added/improved for ImageRepeat and VisualDensity enum values
  • Fixed incorrect references in polyline_layer.dart from "polygons"/"PolygonMarker" to "polylines"/"PolylineMarker"
  • Minor formatting cleanup in type alias documentation

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
sdk/python/packages/flet/src/flet/controls/types.py Added documentation strings for ImageRepeat enum values and improved formatting of VisualDensity enum documentation
sdk/python/packages/flet-map/src/flutter/flet_map/lib/src/polyline_layer.dart Fixed incorrect children selector and type check from "polygons"/"PolygonMarker" to "polylines"/"PolylineMarker"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@FeodorFitsner
Copy link
Contributor

Please check copilot comments here: https://github.com/flet-dev/flet/pull/5794/files#diff-dd984269d51a00ebf6d86e01e0dc5cd02cfd6f00a9451245045712142894270f

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.

PolylineLayer is not showing in map

3 participants