8000 Persistent point selection with transforms compatibility by etpinard · Pull Request #2163 · plotly/plotly.js · GitHub
[go: up one dir, main page]
Skip to content
8000

Persistent point selection with transforms compatibility #2163

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 11 commits into from
Nov 16, 2017
Merged

Persistent point selection with transforms compatibility #2163

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
54cc67b
make hover AND select use same event data pipeline
etpinard Nov 15, 2017
638d03d
adapt Histogram.eventData so that it works with selection pts
etpinard Nov 15, 2017
b30fab4
add Scattercarpet.eventData method
etpinard Nov 15, 2017
949b311
add ScatterTernary.eventData method
etpinard Nov 15, 2017
315e632
add Lib.tagSelected made to make selectedpoints work w/ transforms
etpinard Nov 15, 2017
9b79b3f
add support for selection of aggregations
etpinard Nov 15, 2017
9210278
update point-selection baseline
etpinard Nov 15, 2017
261387d
update event-data jasmine tests (mostly adding pointIndex)
etpinard Nov 15, 2017
451778b
speed up is-pt-index-valid check
etpinard Nov 16, 2017
c8d715b
emit 'normalized' a/b/c coords in scatterternary event data
etpinard Nov 16, 2017
7543dc6
move ptNumber2cdIndex computation to main binning loop
etpinard Nov 16, 2017
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
speed up is-pt-index-valid check
  • Loading branch information
@etpinard
etpinard committed Nov 16, 2017
commit 451778b2adf7e4584bb5d55406712da9ef85eb7d
2 changes: 1 addition & 1 deletion src/lib/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -495,7 +495,7 @@ lib.tagSelected = function(calcTrace, trace, ptNumber2cdIndex) {
}

function isPtIndexValid(v) {
return lib.validate(v, {valType: 'integer', min: 0});
return isNumeric(v) && v >= 0 && v % 1 === 0;
}

function isCdIndexValid(v) {
Expand Down
0