Incorrect handling of tabs in link resources compared to spaces, newlines
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 45/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
- tooling
Hướng nghiên cứu
Bắt đầu tại lib/inlines.js, khoảng dòng 651, và tái hiện các trường hợp tab và dấu cách bằng permalink dingus được liên kết. So sánh cách xử lý xung quanh đích đến và tiêu đề của liên kết, sau đó xác minh rằng tab và dấu cách hoạt động giống nhau trong từng trường hợp được liệt kê.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
- Spec currently defines whitespace can exist, and in some cases must exist around the destination and title, in the parens: https://spec.commonmark.org/0.29/#inline-link
- I clarified the wording recently in https://github.com/commonmark/commonmark-spec/pull/618
- But
whitespacewas used since the start: https://github.com/commonmark/commonmark-spec/blame/858a28941d0dd17c24b7240f21372652111bd38b/spec.txt#L7495
The bug probably stems from here: https://github.com/commonmark/commonmark.js/blob/8c698a295f7cea58597769ccfaab2219add45e44/lib/inlines.js#L651, which seems to include spaces and newlines but not tabs.
This bug can be reproduced with the following permalink to the dingus: https://spec.commonmark.org/dingus/?text=tab%3A%20%5Bx%5D(%09y)%0Aspace%3A%20%5Bx%5D(%20y)%0A%0Atab%3A%20%5Bx%5D(y%09)%0Aspace%3A%20%5Bx%5D(y%20)%0A%0Atab%3A%20%5Bx%5D(%09%3Cy%3E)%0Aspace%3A%20%5Bx%5D(%20%3Cy%3E)%0A%0Atab%3A%20%5Bx%5D(y%09%22z%22)%0Aspace%3A%20%5Bx%5D(y%20%22z%22)%0A.
The expected behavior is that tabs and spaces behave the same.
- Ngôn ngữ chính
- JavaScript
- Star
- 1.6k
- Fork
- 231
- Merge trung bình
- 2 ngày 17 giờ
- Pull request đã merge (30 ngày)
- 2
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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 commonmark/commonmark.js
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
commonmark/commonmark.js#303 · 2 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 38/100
commonmark/commonmark.js#298 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
commonmark/commonmark.js#287 · 1 reaction ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 20/100
commonmark/commonmark.js#275 ·
-
Delegate node walking to othersĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
commonmark/commonmark.js#267 · 3 bình luận ·
Tất cả issue của commonmark/commonmark.js
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
naver/egjs-flicking#971 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
RADAR-base/radar-self-enrolment-ui#118 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Renderer treats a sub-pixel width difference as a resize, which cancels the `motion()` entranceĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
Maintainer thường phản hồi trong vòng 1 ngày
-
triage-needed
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
MicrosoftDocs/edge-developer#3970 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
agent/scanner hive/hosted-available-lke648397-260827-5n31 quality testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày