plotly / plotly/plotly.js

Use newer version of Maki icons for scattermapbox plots

Ouverte
#7,447 1 commentaire 2 réactions 1 personne assignée Voir sur GitHub

@camdecoster y travaille déjà.

Depuis le 24/6/2025.

feature P3
Langage dominant
JavaScript
Étoiles
18.3k
Forks
2k
Merge moyen
2 j 12 h
PR mergées (30 j)
28

Description

Description

The library currently uses (via unpkg) v2.1.0 of Maki icons for the scattermapbox plot. The current version is 8.2.0. Our documentation points to the Mapbox page listing the Maki icons and it doesn't match what icons are currently available. We should update the library to utilize the newer version of Maki.

Acceptance criteria

  • Library points to newer version of Maki
  • Documentation is updated to describe what icon options are available

Notes

  • After some initial testing, there are some CORS issues with pointing to the newer version of Maki. This appears to be an unpkq issue.
  • The newer icons don't have the size suffix (-11, -15, etc.) so the library will need to updated to account for that
  • This issue has been reported in the plotly.py repo here and here
  • An alternative icon source is the jsdelivr CDN (which doesn't have CORS issues from first testing)

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.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

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