Skip to content
This repository has been archived by the owner on Apr 2, 2019. It is now read-only.

v0.3.0 adds auto-position: service can calculate optimal {center,zoom} #3

Merged
merged 4 commits into from Feb 8, 2017

Conversation

juliengirault
Copy link
Contributor

Following this PR: #2

Working examples

(FWIW, the data for this example is taken from https://www.mediawiki.org/wiki/Help:Extension:Kartographer#.3Cmaplink.3E_and_auto-counters)

  • Fixed position:
/img/osm-intl,13,37.801510600701,-122.39902496338,300x300.png?domain=www.mediawiki.org&title=Help%3AExtension%3AKartographer&groups=_063499af64a79f361aab32753fbe1ca8f5d3782d
  • Auto zooming
/img/osm-intl,a,37.801510600701,-122.39902496338,300x300.png?domain=www.mediawiki.org&title=Help%3AExtension%3AKartographer&groups=_063499af64a79f361aab32753fbe1ca8f5d3782d
  • Auto centering
/img/osm-intl,13,a,a,300x300.png?domain=www.mediawiki.org&title=Help%3AExtension%3AKartographer&groups=_063499af64a79f361aab32753fbe1ca8f5d3782d
  • Auto positioning (centering + zooming)
/img/osm-intl,a,a,a,300x300.png?domain=www.mediawiki.org&title=Help%3AExtension%3AKartographer&groups=_063499af64a79f361aab32753fbe1ca8f5d3782d

Throwing exceptions

  • Auto positioning with no overlays
/img/osm-intl,a,a,a,300x300.png
  • Auto centering with a fixed coordinate
/img/osm-intl,12,37.801510600701,a,300x300.png?domain=www.mediawiki.org&title=Help%3AExtension%3AKartographer&groups=_063499af64a79f361aab32753fbe1ca8f5d3782d

Phabricator ticket

T149147

lib/snapshot.js Outdated
@@ -104,7 +104,13 @@ function requestHandler(req, res, next) {
}

let noOverlay = !qparams.domain && !qparams.title,
useAutoPositioning = params.lat === 'a' || params.lon === 'a' || params.zoom === 'a';
useAutoCentering = params.lat === 'a' || params.lon === 'a',
useAutoZooming = params.zoomo === 'a',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

zoomo :)

@nyurik nyurik merged commit c94c426 into kartotherian:master Feb 8, 2017
@juliengirault juliengirault deleted the autopos2 branch February 8, 2017 19:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants