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

Timeline filter by viewport #3851

Merged
merged 14 commits into from Jun 20, 2019

Conversation

offtherailz
Copy link
Member

Description

Add timeline sync with current map viewport.
image

This adds support list and snap times only in the current viewport

Other minor fixes:

  • Clear Openlayers cache when some layer's parameter changes, to avoid old lower-zoom previews
  • Fixed zoom to extent hook that didn't work for full world zoom to (typically when the bbox is calculated from the CRS and it is EPSG:4326)

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Feature

What is the current behavior? (You can also link to an open issue here)
Timeline show all times allover the worlds

What is the new behavior?
user can filter the times using the current viewport

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@offtherailz offtherailz changed the title Fix time viewport sync Timeline filter by viewport Jun 19, 2019
@tdipisa tdipisa added this to the 2019.02.00 milestone Jun 19, 2019
@geosolutions-it geosolutions-it deleted a comment from coveralls Jun 19, 2019
Copy link
Contributor

@allyoucanmap allyoucanmap left a comment

Choose a reason for hiding this comment

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

it seems good.
I had only an issue with the tooltip on the new sync map button.
The tooltip changes only when you toggle the enable range button. [ ]

@offtherailz offtherailz merged commit 1a317d3 into geosolutions-it:master Jun 20, 2019
@offtherailz offtherailz self-assigned this Jun 20, 2019
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