mapbox / mapbox/node-tilejson

support for HiDPI

Offen
#53 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
33
Forks
23
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

The module doesn't seem to works with HiDPI screens like Retina.

Example:
save following under config.json, then run /usr/bin/nodejs /usr/bin/tessera -c . -p 28089

{
 "/toner": {
   "source": "tilejson+http://staging.tile.stamen.com/toner-background/index.json"
  },
 "/toner-http": {
   "source": "http://toner-elb-3vaew2i23b6d-1666871272.us-east-1.elb.amazonaws.com/toner-background/{z}/{x}/{y}.png"
  }
}

http://127.0.0.1:28089/toner-http/6/11/26@2x.png returns a 512x512 image. Fine.

http://toner-elb-3vaew2i23b6d-1666871272.us-east-1.elb.amazonaws.com/toner-background/6/11/26@2x.png returns a 512x512 image (it's the link described in the tilejson file). Fine.

http://127.0.0.1:28089/toner/6/11/26@2x.png returns a 256x256 image. Wrong :-(.

The solution is probably as simple as integrating the modifications proposed in https://github.com/mapbox/node-tilejson/pull/26/files.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, das Verhalten mit der bereitgestellten config.json und den /toner- und /toner-http-@2x-URLs zu reproduzieren. Vergleiche das Modul mit den im verknüpften node-tilejson Pull Request vorgeschlagenen Änderungen. Fertig ist die Aufgabe, wenn der /toner/6/11/26@2x.png-Endpunkt das erwartete 512x512-Bild statt eines 256x256-Bildes zurückgibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.