plotly / plotly/plotly.js

Use newer version of Maki icons for scattermapbox plots

Đang mở
#7,447 1 bình luận 2 reaction 1 người được giao Xem trên GitHub

@camdecoster đang làm issue này rồi.

Từ ngày 24/6/2025.

feature P3
Ngôn ngữ chính
JavaScript
Star
18.3k
Fork
2k
Merge trung bình
2 ngày 12 giờ
Pull request đã merge (30 ngày)
28

Mô tả

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)

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.