mapbox / mapbox/mapbox-maps-flutter
Various flutter style issues
Nobody has claimed this yet.
- Dominant language
- Dart
- Stars
- 380
- Forks
- 204
- PR merge metrics
- No merged PRs in 30d
Description
Minor nit, but I noticed in various places things don't quite align with how Flutter standard libraries would do them. For example, MapAnimationOptions accepts a int duration in milliseconds, where it should accept a Duration type, or CameraOptions's center being a json map, instead of a Point.
I suspect these come from the underlying API, but if you'd accept the changes I'd be happy to make various cleanups to bring more inline with a standard Flutter library.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the public APIs for MapAnimationOptions and CameraOptions, then compare their duration and center types with standard Flutter library conventions and the underlying API. Clarify which cleanups are in scope and confirm that the resulting interfaces consistently use Flutter-idiomatic types.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- dart, flutter
- Domain
- api, mobile-dev
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100