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.

Sign up for GitHub

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

Jump to bottom

Add Bounds property to Map #30

Open
AlexKven opened this issue May 26, 2020 · 0 comments
Open

Add Bounds property to Map #30

AlexKven opened this issue May 26, 2020 · 0 comments
Assignees
@AlexKven
Labels
enhancement Concerns changes that improve or upgrade the existing code base.

Comments

@AlexKven
Copy link

I find myself needing to access the bounds of the map in one of my projects. I have added this feature to Map in my own fork using a property like this:

/// <summary>
/// Map bounds
/// </summary>
public Bounds Bounds { get; private set; }

Assign this issue to me if you want me to make a pull request for this feature.
@Mehigh17 Mehigh17 assigned Mehigh17 and AlexKven and unassigned Mehigh17 May 26, 2020
@Mehigh17 Mehigh17 added the enhancement Concerns changes that improve or upgrade the existing code base. label May 26, 2020
AlexKven pushed a commit to AlexKven/BlazorLeaflet that referenced this issue Jun 3, 2020
Mehigh17#29, Mehigh17#30, 32 Updates to map properties, fixed memory …
d0baf0b 
…leak.

Mehigh17#29 Enables the Center and Zoom properties to set these properties on the map in the background.
Mehigh17#30 Adds a Bounds property to the map that is automatically kept up to date with the map bounds.
Mehigh17#32 Fixes a memory leak related to creating layers.
@AlexKven AlexKven mentioned this issue Jun 3, 2020
Closed
@Mehigh17 Mehigh17 linked a pull request Jun 6, 2020 that will close this issue
#29, #30, 32 Updates to map properties, fixed memory leak. #33
Closed
AlexKven pushed a commit to AlexKven/BlazorLeaflet that referenced this issue Jun 10, 2020
Mehigh17#30 Add bounds property to the map.
a78552d
AlexKven pushed a commit to AlexKven/BlazorLeaflet that referenced this issue Jun 11, 2020
Mehigh17#30 Add bounds property to the map.
bd76e2d
maxreb added a commit to ACOAhlmann/ACO.Blazor.Leaflet that referenced this issue Sep 12, 2020
@maxreb
Merge pull request #1 from AlexKven/feature/map-bounds
09fd8b5 
Mehigh17#30 Add bounds property to the map.
charleypeng added a commit to charleypeng/BlazorLeaflet that referenced this issue May 6, 2022
@charleypeng
Merge pull request #1 from AlexKven/feature/map-bounds
1fa53ad 
Mehigh17#30 Add bounds property to the map.
@charleypeng charleypeng mentioned this issue May 9, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlexKven AlexKven

Labels
enhancement Concerns changes that improve or upgrade the existing code base.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#29, #30, 32 Updates to map properties, fixed memory leak. AlexKven/BlazorLeaflet
2 participants
@Mehigh17 @AlexKven