TwitterEmbed mediatype incorrectly assumes there is only one element with className tl-media-twitter
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 48/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- frontend
Hướng nghiên cứu
Khởi động máy chủ kiểm thử cục bộ bằng npm start và tái hiện vấn đề bằng cấu hình "all media types". Đọc src/js/media/types/TwitterEmbed.js quanh dòng 80 và kiểm tra cách các phần tử DOM media được tạo. Hoàn tất khi nhiều phần tử có className tl-media-twitter mỗi phần tử đều kích hoạt callback cho Twitter embed tương ứng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
This must have been around for a long time, but I stumbled onto this bug which is visible using the "all media types" configuration using the local test server (npm start)
Probably the fix is to add a unique ID when the media DOM elements are created and use that in the callback instead of just assuming index there's only one.
But except for backwards compatibility is this something that affects many users? probably not.
- Ngôn ngữ chính
- JavaScript
- Star
- 3.2k
- Fork
- 645
- Merge trung bình
- 3 phút
- Pull request đã merge (30 ngày)
- 2
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của NUKnightLab/TimelineJS3
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
NUKnightLab/TimelineJS3#797 ·
-
multiple header comments Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 68/100
NUKnightLab/TimelineJS3#617 ·
-
Bug CSS
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 62/100
NUKnightLab/TimelineJS3#602 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
NUKnightLab/TimelineJS3#904 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
NUKnightLab/TimelineJS3#902 · 2 bình luận ·
Tất cả issue của NUKnightLab/TimelineJS3
Issue tương tự
-
Update HugeIcons library Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
antfu-collective/icones#398 ·
-
ECmail.com Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
wesbos/burner-email-providers#554 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
radiantearth/stac-browser#1023 ·
-
HMR stops working Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
components-web-app/docs#92 ·