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

KML Example inconsistent behaviour #481

Closed
markpet49 opened this issue Feb 23, 2018 · 2 comments
Closed

KML Example inconsistent behaviour #481

markpet49 opened this issue Feb 23, 2018 · 2 comments
Assignees
Labels

Comments

@markpet49
Copy link
Member

markpet49 commented Feb 23, 2018

Child of #565

Description

The KML Example demonstrates the following inconsistent behaviours:

  1. Initially, without the Sentinel-2 Imagery checkbox checked, a Sentinel image shows up in the vicinity of Germany. This image disappears and reappears farther south as you zoom in (See attached screen shots)
  2. Unchecking KML Samples initially turns off the polygons in the vicinity of Poland, toggling the Polygon checkbox off and then on causes them to persist when unchecking KML Samples.

Steps to Reproduce

  1. Run KMLExample
  2. Rotate the globe so that europe is visible.

Operating System and Version

Ubuntu 16.04/Chrome 64

ss1
ss2

@balhar-jakub
Copy link
Contributor

This Behavior is a complex one and some pieces of it aren't bugs but instead intentional usage of the KML Features.

  1. The image near the borders of the Germany shows and hides because it is connected to the Region element which says it should be vissible only in certain distance.
    Image showing to the south is actually different image with different settings of the Region element.
  2. It is intended that the lower level checkbox change persists even when the top level changes but only when user interaction changed it. I am not even sure what would be the perfect model as there are actually 6 different properties coming in the play when you decide whether the element should be displayed.

This is more of an explanation of what is happening behind the scenes. The Number 2 is probably something I don't want to change or at least I am a bit unsure about how it should happen.

As for Number 1) I propose that I will change the imagery that is shown in the two cases. First being the satellite imagery and second some other type of image so that is less confusing. I would like to hear thoughts on this.

@pdavidc pdavidc removed the story label May 7, 2018
@AkeluX
Copy link
Contributor

AkeluX commented Aug 20, 2018

@markpet49 any thoughts on the feedback from @balhar-jakub?

@AkeluX AkeluX closed this as completed Apr 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

4 participants