-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Rangeslider allow zoom on oppaxis #2364
Merged
alexcjohnson
merged 44 commits into
plotly:master
from
TomDemulierChevret:rangeslider-allow-zoom-on-oppaxis
Mar 7, 2018
Merged
Changes from 13 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
519e1a7
better visual cue of zoom on oppaxis with rangeslider
TomDemulierChevret 044e633
add a new attributes to rangeslider to switch back to the old behaviour
TomDemulierChevret f335258
fix check when using new data
TomDemulierChevret 707f432
fix default value rangeslider test
TomDemulierChevret 6a7f652
renammed new attributes to "fixedyrange"
TomDemulierChevret fc4f082
Introduce new attributes to axes to allow to specifiy fixed range to …
TomDemulierChevret 92c6550
Fix broken test from attribute removal
TomDemulierChevret b30167c
Add missing gl attribute
TomDemulierChevret 91caff1
Merge branch 'master' into rangeslider-allow-zoom-on-oppaxis
TomDemulierChevret 3dd91b5
Renammed new attributes for more clarity and portability
TomDemulierChevret 00fd1d7
Coerce of new attributes is only made for y axis
TomDemulierChevret 5e0486a
Merge remote-tracking branch 'upstream/master' into rangeslider-allow…
TomDemulierChevret 32488f1
Changed rangeslidermode values for more clarity
TomDemulierChevret bc2a40a
Switch to new rangeslider range opts (default missing)
TomDemulierChevret d3b6071
Added default attributes
TomDemulierChevret b496641
Remove old attributes
TomDemulierChevret 149ef70
Add autorange computation
TomDemulierChevret 54d8b58
Merge remote-tracking branch 'upstream/master' into rangeslider-allow…
TomDemulierChevret dd48914
Add missing check
TomDemulierChevret a8a3ee5
Add missing catch to test to retrieve the tracebacks
TomDemulierChevret 0ae2831
Change yAxis rangeslider rangeOpts computation + fix yAxis rangeslide…
TomDemulierChevret 8764e4d
Add missing check
TomDemulierChevret a17d3a8
Fix rangeslider test
TomDemulierChevret e85e3d0
Renamed parameter to keep name consistency across the file
TomDemulierChevret 0dd8562
Merge remote-tracking branch 'upstream/master' into rangeslider-allow…
TomDemulierChevret 3929be6
Changes requested by @alexcjohnson
TomDemulierChevret e2ad33d
Changes requested by @alexcjohnson
TomDemulierChevret df85f35
Changes requested by @etpinard
TomDemulierChevret fe83e07
Default range for 'fixed' rangemode value is now the range of the axe
TomDemulierChevret 3b6b81a
Changes requested by @etpinard
TomDemulierChevret 9de4e3e
Changes requested by @etpinard
TomDemulierChevret f2819c8
Changes requested by @etpinard
TomDemulierChevret df96a13
Merge branch 'master' into rangeslider-allow-zoom-on-oppaxis
TomDemulierChevret 9c66f81
Add test for new attributes default and coerce
TomDemulierChevret f092c44
Add interaction test for new attributes
TomDemulierChevret 041ce3d
Fixed autorange test precision
TomDemulierChevret 23c8401
Fixed autorange test precision
TomDemulierChevret 73560f6
Merge branch 'master' into rangeslider-allow-zoom-on-oppaxis
alexcjohnson c556f8c
rename range_attributes file -> oppaxis_attributes
etpinard 43f15a7
add rangeslider.yaxis? to schema flagging it with _isSubplotObj
etpinard ccb6091
fix rangeslider.y.rangemode=auto and add a test image
alexcjohnson 6a88b24
Merge branch 'rangeslider-allow-zoom-on-oppaxis' of github.com:TomDem…
alexcjohnson 95b8c10
test turning on autorange via rangeslider attrs
alexcjohnson 4f54f62
drop rangeslider style attributes to editType: plot and test
alexcjohnson File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noticed this. Why pick a different opacity value here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this effect - you see that the y axis is constrained, but it's not shaded as darkly as the x axis constraint, which aligns with the fact that you can only affect the x axis constraint with the range slider handles.
But a different thought occurred to me on looking at this: this is semitransparent black, so does that mean range sliders don't work on a black background? That's a totally separate issue, doesn't need to be addressed in this PR, but we should look into it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, that's what I wanted to show with the 2 different values of opacity.
And yeah, the black backgrond will be an issue.