8000 Fixup hover on period positioned points by archmoj · Pull Request #5618 · plotly/plotly.js · GitHub
[go: up one dir, main page]
Skip to content

Fixup hover on period positioned points #5618

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
May 13, 2021
Merged

Fixup hover on period positioned points #5618

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
689adfa
place period positioned points at the end to avoid winning hover
archmoj Apr 28, 2021
c96bf99
adjust min and max positions of bars in respect to period alignment
archmoj Apr 28, 2021
c6260e3
order hover data to possibly get a non-period point as the winning point
archmoj May 1, 2021
f4a465e
clarify winning point - avoid extra hd variable
archmoj May 1, 2021
c89784d
refactor - make getVal function
archmoj May 1, 2021
d584520
rename function to customVal
archmoj May 1, 2021
e15da90
optimize customVal function
archmoj May 1, 2021
b4bda4e
also look for other alignment periods when a period point wins
archmoj May 1, 2021
e96ad53
fixup violin bug introduced in e15da902d043400c476f0f3fd22aafa876728eb3
archmoj May 3, 2021
428e248
fixup ordering period points
archmoj May 3, 2021
ef65b9f
add jasmine tests
archmoj May 3, 2021
543698a
filter seen traces in period hovers
archmoj May 6, 2021
170dd5d
add jasmine test
archmoj May 6, 2021
4f80d95
merge two tests
archmoj May 6, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
optimize customVal function
  • Loading branch information
@archmoj
archmoj committed May 1, 2021
commit e15da902d043400c476f0f3fd22aafa876728eb3
23 changes: 14 additions & 9 deletions src/components/fx/hover.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -650,11 +650,10 @@ function _hover(gd, evt, subplot, noHoverEvent) {
hoverData[0].trace.type !== 'splom' // TODO: add support for splom
) {
var winningPoint = hoverData[0];
var isGrouped = (fullLayout.boxmode === 'group' || fullLayout.violinmode === 'group');

findHoverPoints(
customVal('x', winningPoint, isGrouped),
customVal('y', winningPoint, isGrouped)
customVal('x', winningPoint, fullLayout),
customVal('y', winningPoint, fullLayout)
);

// Remove duplicated hoverData points
Expand Down Expand Up @@ -1898,15 +1897,21 @@ function orderPeriod(hoverData, hovermode) {
hoverData = first.concat(last);
}

function customVal(axLetter, winningPoint, isGrouped) {
var cd0 = winningPoint.cd[winningPoint.index];
function customVal(axLetter, winningPoint, fullLayout) {
var ax = winningPoint[axLetter + 'a'];
var val = winningPoint[axLetter + 'Val'];

if(ax.type === 'category') val = ax._categoriesMap[val];
if(ax.type === 'date') val = ax.d2c(val);
if(cd0 && cd0.t && cd0.t.posLetter === ax._id && isGrouped) {
val += cd0.t.dPos;
if(ax.type === 'category') return ax._categoriesMap[val];
if(ax.type === 'date') return ax.d2c(val);

var cd0 = winningPoint.cd[winningPoint.index];
if(cd0 && cd0.t && cd0.t.posLetter === ax._id) {
if(
fullLayout.boxmode === 'group' ||
fullLayout.violinmode === 'group'
) {
val += cd0.t.dPos;
}
}

return val;
Expand Down
0