enlohhy opened a new pull request, #40046:
URL: https://github.com/apache/superset/pull/40046
### SUMMARY
Takes over the GeoJSON point radius controls work from #33247 into a
fork-based branch so it can continue moving forward.
This carries over the same three user-facing pieces from the existing PR:
- adds a new `Point Radius` control
- adds a new `Point Radius Units` control
- keeps `Point Radius Scale`, but makes it work with more intuitive defaults
It also preserves the backward-compatibility follow-up from the earlier
review:
- existing charts without the new fields fall back to deck.gl defaults
- new charts still get the improved control-panel defaults
- tests cover both legacy and new-chart behavior
### TESTING INSTRUCTIONS
1. Create a GeoJSON chart with point features.
2. In the chart controls, verify you can set `Point Radius`, `Point Radius
Units`, and `Point Radius Scale`.
3. Check that:
- small values like `1`, `5`, or `10` work with `Pixels`
- `Meters` scales with zoom
- older charts without the new saved fields keep their prior behavior
### ADDITIONAL INFORMATION
- Follow-up/takeover of #33247
- Related bounty issue: #34398
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
