Skip to content

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.

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

Open
camdecoster wants to merge 18 commits into
base: master
Choose a base branch
from
Open

feat: Add min/max scale limits for geo plots #7371

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
57307ee
feat(geo): support min/max scale limits
mojoaxel Oct 5, 2020
d870252
refactor: rename min/max scale limits
mojoaxel Oct 26, 2020
2d9e5f2
chore: use dflt: -1 instad of null
mojoaxel Oct 26, 2020
dd635ee
feat: handle maxscale default in scaleExtent
mojoaxel Oct 26, 2020
8a2c0c5
chore: only rerender on zoom if scale changes
mojoaxel Oct 26, 2020
f882191
chore: fix typo in min/maxscale attributes
mojoaxel Oct 26, 2020
8ed949f
test: adopt test/plot-schema.json to new plot-schema
mojoaxel Jun 10, 2024
29b4316
Trigger initial zoom event to set minscale
camdecoster Feb 18, 2025
048f178
Switch to ES5 syntax
camdecoster Feb 21, 2025
c6007a8
Merge remote-tracking branch 'origin/master' into cam/5192/add-min-ma…
camdecoster Feb 21, 2025
84cd037
Switch to getBBox and fix zoom.event call
camdecoster Feb 22, 2025
d9fca8b
Add draft log
camdecoster Feb 22, 2025
5f0083f
Merge remote-tracking branch 'origin/master' into cam/5192/add-min-ma…
archmoj Jul 21, 2025
0a1f133
Merge remote-tracking branch 'origin/master' into cam/5192/add-min-ma…
camdecoster Jul 23, 2025
ee8597f
Add event check before triggering zoom transition
camdecoster Jul 24, 2025
9b04737
Don't delete default scales for fitbounds
camdecoster Jul 24, 2025
32feb89
add tests for various cases
archmoj Jul 21, 2025
65665fe
Update tests to check scale against scaleExtent
camdecoster Jul 25, 2025
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
refactor: rename min/max scale limits
  • Loading branch information
@mojoaxel @camdecoster
mojoaxel authored and camdecoster committed Feb 18, 2025
commit d870252e3e2d2838f39b6ca996f704fba106fefe
12 changes: 6 additions & 6 deletions src/components/modebar/buttons.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -543,16 +543,16 @@ function handleGeo(gd, ev) {

if(attr === 'zoom') {
var scale = geoLayout.projection.scale;
var minScale = geoLayout.projection.minScale;
var maxScale = geoLayout.projection.maxScale;
var minscale = geoLayout.projection.minscale;
var maxscale = geoLayout.projection.maxscale;

var newScale = (val === 'in') ? 2 * scale : 0.5 * scale;

// make sure the scale is within the min/max bounds
if(newScale > maxScale) {
newScale = maxScale;
} else if(newScale < minScale) {
newScale = minScale;
if(newScale > maxscale) {
newScale = maxscale;
} else if(newScale < minscale) {
newScale = minscale;
}

Registry.call('_guiRelayout', gd, id + '.projection.scale', newScale);
Expand Down
6 changes: 3 additions & 3 deletions src/plots/geo/layout_attributes.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ var attrs = module.exports = overrideAll({
'that fits the map\'s lon and lat ranges. '
].join(' ')
},
minScale: {
minscale: {
valType: 'number',
min: 0,
dflt: 0,
Expand All @@ -187,10 +187,10 @@ var attrs = module.exports = overrideAll({
'where the map has half the size of base zoom level.'
].join(' ')
},
maxScale: {
maxscale: {
valType: 'number',
min: 0,
dflt: Infinity,
dflt: null,
description: [
'Maximal zoom level of the map view.',
'A maxScale of *2* (200%) corresponds to a zoom level',
Expand Down
8 changes: 4 additions & 4 deletions src/plots/geo/layout_defaults.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -161,8 +161,8 @@ function handleGeoDefaults(geoLayoutIn, geoLayoutOut, coerce, opts) {
}

coerce('projection.scale');
coerce('projection.minScale');
coerce('projection.maxScale');
coerce('projection.minscale');
coerce('projection.maxscale');

show = coerce('showland', !visible ? false : undefined);
if(show) coerce('landcolor');
Expand Down Expand Up @@ -207,8 +207,8 @@ function handleGeoDefaults(geoLayoutIn, geoLayoutOut, coerce, opts) {
// clear attributes that will get auto-filled later
if(fitBounds) {
delete geoLayoutOut.projection.scale;
delete geoLayoutOut.projection.minScale;
delete geoLayoutOut.projection.maxScale;
delete geoLayoutOut.projection.minscale;
delete geoLayoutOut.projection.maxscale;

if(isScoped) {
delete geoLayoutOut.center.lon;
Expand Down