Uh oh!
There was an error while loading. Please reload this page.
better default order for multiple hovers - #3396
Conversation
nicolaskruchten
commented
Jan 4, 2019
Note that this makes #2970 a bit worse, because it'll show only 'min' and not 'median' any longer, because that's now the first hover. |
nicolaskruchten
commented
Jan 4, 2019
LMK if you like the fix and I'll fix the tests |
etpinard
commented
Jan 4, 2019
Thanks for the PR! Going from: to: is amazing 🎉 I'm a little confused about why your patch works though.
|
nicolaskruchten
commented
Jan 4, 2019
Hehehe like I said, I just looked at it and thought "seems like they're laying out in order and jamming each other up" ... I didn't dig into the root cause at all, but if there's a deeper/more elegant fix I'm all for it. |
etpinard
commented
Jan 4, 2019
My attempt in -> #3401 |


Closes#3389 AND closes#3050 :)