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

Clicking on the chart panel should bring it in front of the feature info panel #1691

Closed
RacingTadpole opened this issue Jun 27, 2016 · 7 comments

Comments

@RacingTadpole
Copy link
Contributor

RacingTadpole commented Jun 27, 2016

This used to work, but somewhere along the way it stopped. Extracted from #1427.

@kring
Copy link
Member

kring commented Jun 30, 2016

Are you sure we want this? The user can just close or collapse the feature info panel if it's in the way of the chart. Having a dynamic z-order for our windows makes things a lot more complicated.

@AlexGilleran
Copy link
Contributor

That ship has sailed, we already have dynamic z order 😬 . I think chloe did it, it's a pretty simple implementation in viewstate.

@kring
Copy link
Member

kring commented Jun 30, 2016

...except for bugs like this and #1714.

@AlexGilleran
Copy link
Contributor

Well, yeah, the chart hasn't implemented it (yet ?)

@AlexGilleran
Copy link
Contributor

Although I would argue that the chart should always be below everything given that it's tiled, and floating stuff like feature info panels should always be over the top of it.

@kring
Copy link
Member

kring commented Jun 30, 2016

Yep, exactly. I just don't see the need for dynamically putting the chart on top.

@RacingTadpole
Copy link
Contributor Author

That's a good argument. OK, I'll close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants