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

Feature/improved selection management #13

Merged

Conversation

aarranz
Copy link
Member

@aarranz aarranz commented Feb 5, 2019

Update current algorithm for updating the selected PoIs:

  • If current zoom level is lower than the configured one, then zoom in the view and center the view on the PoI.
  • Zoom out current view if the PoI representation didn't fit current view
  • Move current view if the zoom level is adequate, but it isn't visible on the current viewport.

Probably this is not the perfect behaviour, but I think this is better than the current one.

@coveralls
Copy link

coveralls commented Feb 5, 2019

Coverage Status

Coverage increased (+0.1%) to 98.121% when pulling a9629d2 on aarranz:feature/improved-selection-management into 4853723 on Wirecloud:develop.

@aarranz aarranz merged commit b68bcc1 into Wirecloud:develop Feb 8, 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.

2 participants