Fix hover with period alignment points and improve positioning of spikes and unified hover label#5846

Merged

archmoj merged 28 commits intomasterfrom

fix5822-hover-with-period

Jul 23, 2021

Commits on Jul 22, 2021

refactor: enter long lines
archmoj
committed
position hover labels in respect to max and min of hoverset not the winning point
archmoj
committed
for end alignment of unified hover label use minimum to avoid overlap with data points
archmoj
committed
similar positioning of hover box for y unified
archmoj
committed
refactor
archmoj
committed
improve unified hover box positioning
archmoj
committed
edit PR log
archmoj
committed
simplify logic and handle the case of not fitting inside the subplot
archmoj
committed
when the scatter point wins, OK to occlude bar & other points
archmoj
committed
fix syntax
archmoj
committed
drop out of range pixel from the end
archmoj
committed
for x|y hovermodes display spikeline on the winning point
archmoj
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Fix hover with period alignment points and improve positioning of spikes and unified hover label#5846

Fix hover with period alignment points and improve positioning of spikes and unified hover label#5846
archmoj merged 28 commits intomasterfrom
fix5822-hover-with-period

Commits on Jul 21, 2021

fix scattergl test

fix hover on scatter and scattergl with periods

fix hover on bars with period

improve and adjust jasmine tests

draft log for PR 5846

drop unused line

fix typo and adjust test

scatters with period ranges

adjust scatter and bar test titles

fit -> it

Commits on Jul 22, 2021

refactor: enter long lines

position hover labels in respect to max and min of hoverset not the winning point

for end alignment of unified hover label use minimum to avoid overlap with data points

similar positioning of hover box for y unified

refactor

improve unified hover box positioning

edit PR log

simplify logic and handle the case of not fitting inside the subplot

when the scatter point wins, OK to occlude bar & other points

fix syntax

drop out of range pixel from the end

for x|y hovermodes display spikeline on the winning point

Commits on Jul 23, 2021

adjust vertical and horizontal alignment of unified hover label

handle end edge case

adjustment also for winning types e.g. heatmap use scatter logic

choose the side of the paper which is closest if unified hover did not fit in subplot

update PR log

move unified hover by one pixel better help adjustment tested with period_positioning9 mock & horizontal variant

Uh oh!

Fix hover with period alignment points and improve positioning of spikes and unified hover label#5846archmoj merged 28 commits intomasterplotly/plotly.js:masterfrom fix5822-hover-with-periodplotly/plotly.js:fix5822-hover-with-periodCopy head branch name to clipboard

Commits

Commits on Jul 21, 2021

Commits on Jul 22, 2021

Commits on Jul 23, 2021

Fix hover with period alignment points and improve positioning of spikes and unified hover label#5846
archmoj merged 28 commits intomasterfrom
fix5822-hover-with-period