Skip to content

Legend toggles visibility of all groupby groups #2013

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

Closed
rreusser opened this issue Sep 18, 2017 · 5 comments
Closed

Legend toggles visibility of all groupby groups #2013

rreusser opened this issue Sep 18, 2017 · 5 comments
Labels
bug something broken

Comments

@rreusser
Copy link
Contributor

Observed: clicking group toggles visibility of all groups.

Expected: clicking group in legend toggles only that group.

28334288-b0330298-6bc8-11e7-9d1b-49fcfe7d4f83

@rreusser
Copy link
Contributor Author

A simple reproducible example:

https://codepen.io/rsreusser/pen/PJZWQq

What is the desired behavior here? @cldougl says there have been some complaints about this, so seems worth addressing. Should groupby simply not set the legendgroup? Or should there be an option not to set the legendgroup? Does the legendgroup do anything apart from linking visibility? Final question: is there any existing way around this?

@alexcjohnson
Copy link
Collaborator

@etpinard may have more to say about this, but I don't think we have a way around this right now, and I suspect we only set legendgroup originally because we didn't have a way to address the groups independently. But with the generic styles attribute it should be doable with explicit groupby handling, similar to #1919.

@etpinard
Copy link
Contributor

etpinard commented Sep 19, 2017

Should groupby simply not set the legendgroup?

Hmm. Where is this set?

I'm only seeing this block as potentially being wrong.

@rreusser
Copy link
Contributor Author

rreusser commented Sep 19, 2017

My mistake regarding legendgroup. Upon thinking about it, that's a bit of an oversimplification. I set it aside for a moment when I got hung up on testing, but almost have a PR ready that applies same the keyedContainer trick to set this in group-specific styles. The only downside then is that actually using legendgroup is somewhat trickier. For a first cut, I think I'll skip that part.

@rreusser
Copy link
Contributor Author

Fixed in #2022. /cc @cldougl

@etpinard etpinard added the bug something broken label Sep 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken
Projects
None yet
Development

No branches or pull requests

3 participants