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: add support for changing the receiver’s viewport to fit given bounds #133

Merged
merged 9 commits into from
Aug 3, 2022
Merged

feat: add support for changing the receiver’s viewport to fit given bounds #133

merged 9 commits into from
Aug 3, 2022

Conversation

TimAlber
Copy link
Contributor

Add a feature to fit the viewport to given coordinate bounds.

@TimAlber TimAlber changed the title feat: add support for changing the receiver’s viewport to fit the bounds feat: add support for changing the receiver’s viewport to fit given bounds Jul 27, 2022
@m0nac0
Copy link
Collaborator

m0nac0 commented Jul 27, 2022

@TimAlber It would be great if you could also add a simple example to the example app. That makes it easier to test this and it also showcases the feature.

@m0nac0
Copy link
Collaborator

m0nac0 commented Jul 29, 2022

Thank you for the example. Do you also want to add a web implementation? If not, I'll create an issue and someone can add it later on.

@TimAlber
Copy link
Contributor Author

I think it makes most sense if you create an issue and someone can add it later on.

padding: 25,
);
},
child: const Text('Set Bounds to Germanny'),
Copy link
Collaborator

@m0nac0 m0nac0 Jul 29, 2022

Choose a reason for hiding this comment

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

Suggested change
child: const Text('Set Bounds to Germanny'),
child: const Text('Set bounds to Germany'),

Copy link
Collaborator

Choose a reason for hiding this comment

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

Just one nit ;)

Copy link
Collaborator

@m0nac0 m0nac0 Jul 31, 2022

Choose a reason for hiding this comment

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

Ah, I also left this tiny review in "pending" mode 🤦‍♂️

padding: 25,
);
},
child: const Text('Set Bounds to Afrika'),
Copy link
Collaborator

@m0nac0 m0nac0 Jul 31, 2022

Choose a reason for hiding this comment

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

Suggested change
child: const Text('Set Bounds to Afrika'),
child: const Text('Set bounds to Africa'),

@m0nac0 m0nac0 enabled auto-merge (squash) July 31, 2022 09:17
@m0nac0 m0nac0 disabled auto-merge July 31, 2022 09:17
Copy link
Collaborator

@m0nac0 m0nac0 left a comment

Choose a reason for hiding this comment

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

Ready to merge this (apart from the very small suggestions), once this PR is up to date with the main branch ;)


class GivenBoundsPage extends ExamplePage {
GivenBoundsPage()
: super(const Icon(Icons.map_sharp), 'Changing given Bounds');
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
: super(const Icon(Icons.map_sharp), 'Changing given Bounds');
: super(const Icon(Icons.map_sharp), 'Changing given bounds');

@TimAlber
Copy link
Contributor Author

TimAlber commented Aug 1, 2022

I fixed those typos.

@TimAlber TimAlber requested a review from m0nac0 August 1, 2022 08:32
@TimAlber
Copy link
Contributor Author

TimAlber commented Aug 1, 2022

I don't know why iOS doesn't build. Maybe try to restart it?

Copy link
Collaborator

@m0nac0 m0nac0 left a comment

Choose a reason for hiding this comment

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

Thank you again for contributing this feature!

@m0nac0 m0nac0 merged commit 3c368b8 into maplibre:main Aug 3, 2022
@m0nac0
Copy link
Collaborator

m0nac0 commented Aug 3, 2022

@TimAlber Your contributions would also be interesting for the flutter-mapbox-gl repo where this plugin was forked from. Would you like to "upstream" your changes? If not, someone else could also do it.

@TimAlber
Copy link
Contributor Author

TimAlber commented Aug 3, 2022

@m0nac0 I think it would be best if someone else could do that.

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.

None yet

3 participants