-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat: Add min/max scale limits for geo plots #7371
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
Open
camdecoster
wants to merge
18
commits into
master
Choose a base branch
from
cam/5192/add-min-max-scale-limits
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
57307ee
feat(geo): support min/max scale limits
mojoaxel d870252
refactor: rename min/max scale limits
mojoaxel 2d9e5f2
chore: use dflt: -1 instad of null
mojoaxel dd635ee
feat: handle maxscale default in scaleExtent
mojoaxel 8a2c0c5
chore: only rerender on zoom if scale changes
mojoaxel f882191
chore: fix typo in min/maxscale attributes
mojoaxel 8ed949f
test: adopt test/plot-schema.json to new plot-schema
mojoaxel 29b4316
Trigger initial zoom event to set minscale
camdecoster 048f178
Switch to ES5 syntax
camdecoster c6007a8
Merge remote-tracking branch 'origin/master' into cam/5192/add-min-ma…
camdecoster 84cd037
Switch to getBBox and fix zoom.event call
camdecoster d9fca8b
Add draft log
camdecoster 5f0083f
Merge remote-tracking branch 'origin/master' into cam/5192/add-min-ma…
archmoj 0a1f133
Merge remote-tracking branch 'origin/master' into cam/5192/add-min-ma…
camdecoster ee8597f
Add event check before triggering zoom transition
camdecoster 9b04737
Don't delete default scales for fitbounds
camdecoster 32feb89
add tests for various cases
archmoj 65665fe
Update tests to check scale against scaleExtent
camdecoster 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 hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
- Add `minscale`, `maxscale` geo plot attributes [[#7371](https://github.com/plotly/plotly.js/pull/7371)] |
This file contains hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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.
Let's try setting
dflt: 0
in the API and handle the case of zero instead of -1.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.
We represent
Infinity
with -1 sinceInfinity
isn't valid in JSON. If we made this change, how would we handle when a user wants to useInfinity
instead of 0 (which is also a valid value).