Skip to content

Horizontal box hover labels out of order #3389

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
nicolaskruchten opened this issue Jan 3, 2019 · 9 comments
Closed

Horizontal box hover labels out of order #3389

nicolaskruchten opened this issue Jan 3, 2019 · 9 comments
Labels
bug something broken

Comments

@nicolaskruchten
Copy link
Contributor

This is pretty rough... median to the left of min and max to the left of upper:

image

Can we enforce an order here?

@nicolaskruchten
Copy link
Contributor Author

only happens in closest hovermode... Why is closest so different for boxes?

@nicolaskruchten
Copy link
Contributor Author

Seems related to but distinct from #3050 ... the hovers are pointing to the correct place, they're just 'snagged' on each other :)

@nicolaskruchten
Copy link
Contributor Author

Theory: they're getting laid out per the order here:

var usedVals = {};
var attrs = ['med', 'min', 'q1', 'q3', 'max'];
if(trace.boxmean || (trace.meanline || {}).visible) {
attrs.push('mean');
}
if(trace.boxpoints || trace.points) {
attrs.push('lf', 'uf');
}

@nicolaskruchten nicolaskruchten added the bug something broken label Jan 3, 2019
@etpinard
Copy link
Contributor

etpinard commented Jan 3, 2019

Again here, the graph's "data" / "layout" would be nice.

@nicolaskruchten
Copy link
Contributor Author

as it happens, violins do this also: https://plot.ly/~nicolaskruchten/252

@nicolaskruchten
Copy link
Contributor Author

Here's the box version: https://plot.ly/~nicolaskruchten/254

@etpinard
Copy link
Contributor

etpinard commented Jan 4, 2019

Yeah, this one is a duplicate of #3050

@nicolaskruchten
Copy link
Contributor Author

Well, I tried my reordering fix and it addresses my issue. Not sure how easily I can test #3050

@nicolaskruchten
Copy link
Contributor Author

OK, pretty easily, and it's a fix :)

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

Successfully merging a pull request may close this issue.

2 participants