Field Paper Atlas Integration #145
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…gacy (Deprecated)
…, but behind the osm overlays
…page message too.
…y gets cleared so that none of the boxes are green
… envelope derived from a single coordinate. Not sure what's with QuadTree?
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Here we are!
When you check out a deployment
and that deployment has a
fp.geojsonfile, you can see the atlas pagesThe name of the page and atlas is on top. The green page is the one referred to on top.
When you tap outside of the atlas area, that top view goes away.
And when you totally leave the area, you also don't see that top text view.
Of course, if you have an MBTiles on, you can't do that, since you are constrained to the bounds of an MBTiles.
I also closed #119. Also, if the vectors are not getting rendered, we don't have the tag list view pop up (pretty confusing for end user).
Note, I still have this bug lingering: #144
Nonetheless, it's about time for a new release!
cc/ @dalekunce @mojodna @DrishT @PaulUithol