mapbox / mapbox/node-tilejson

support for HiDPI

Ouverte
#53 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
33
Forks
23
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par reproduire le comportement avec le config.json fourni et les URL /toner et /toner-http @2x. Comparez le module avec les modifications proposées dans la pull request node-tilejson liée. C’est terminé lorsque le endpoint /toner/6/11/26@2x.png renvoie l’image attendue de 512x512 au lieu d’une image de 256x256.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript
Domaine
backend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.