github / github/include-fragment-element
<include-fragment> and Screen Reader Accessibility
- Ngôn ngữ chính
- JavaScript
- Star
- 568
- Fork
- 43
- Merge trung bình
- 1 ngày 6 giờ
- Pull request đã merge (30 ngày)
- 5
Mô tả
I have the following issue.
I'm a totally blind software engineer and use a screen reader. Obviously, in our company we use Github for managing code. I'm used to review pull requests without any hassles, but recently _that_ has started.
In our company larger pull requests are common (that's debatable whether it's good or bad, but it is as it is). I had to review a pull request containing 54 changed files. With my screen reader I could see only 18 first diffs rendered, then there was nothing at all, nada. The remaining diffs were however visible on screen for sighted people.
When inspecting the source code of the page in my browser, I saw that the first 18 diff tables are indeed rendered correctly, then there is an `` element with a link to all of the remaining diffs. I tried to open that link as a separate web page, but of course I had no luck in seeing comments properly and so on.
My question is: Is it incorrect usage pattern of this library on github or is this library inaccessible in its nature?
this is a huge show stopper for my work. Github has always been an extremely accessible and friendly network to work with, but now it's really disappointing.
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện hành vi được báo cáo với một pull request lớn và kiểm tra cách phần tử tải các diff bị trì hoãn. Xác định xem nội dung đã tải, bao gồm cả các bình luận, có được cung cấp chính xác cho trình đọc màn hình hay không; để được xem là hoàn tất, cần có bản sửa lỗi về khả năng tiếp cận được ghi lại hoặc lời giải thích rõ ràng về giới hạn của thư viện.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- javascript
- Lĩnh vực
- accessibility, frontend
- Loại issue
- Lỗi
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 25/100