Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Recurring review comments: 3 proposed changes

Đang mở
  • Có gì không ổn?
#6,319 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

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
25/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
javascript, markdown
Lĩnh vực
accessibility, developer-experience, documentation, testing-qa

Hướng nghiên cứu

Read AGENTS.md and inspect the project’s lint configuration before deciding whether to add each proposed guideline or vuejs-accessibility rule. Use the linked files as examples, especially the QTI editor and TipTap editor tests/components. Done means each of the three proposals is implemented or explicitly declined in a comment, but this issue is marked not open for contribution.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

❌ This issue is not open for contribution. Visit Contributing guidelines to learn about the contributing process and how to find suitable issues.

Reviewers keep making these points on PRs to this repo, each on 3+ PRs by 2+ authors in the last 90 days. Each proposes an AGENTS.md line or a lint rule that would have prevented them.

Acceptance criteria

Each change is made, or declined in a comment here.

  • Tests assert outcomes through translator references and roles, not literals, internals or presence
  • Prefer lodash and existing utilities over hand-rolled helpers
  • Accessible names, live regions and focus follow ARIA/WCAG patterns

Proposed changes

Tests assert outcomes through translator references and roles, not literals, internals or presence (5 PRs, 3 authors)

AGENTS.md Testing: "Assert the resulting state, such as document HTML, the role-scoped text (getByRole('status')), or saved DB rows equal to the input, not call chains, fetch strategy or toBeInTheDocument() presence. Reference strings through their translator (tr.$tr(...)/str$()), never literals. Never query KDS/keen-ui internal classes; use within(getByTestId(...))."

Prefer lodash and existing utilities over hand-rolled helpers (3 PRs, 2 authors)

AGENTS.md Code: "Before writing a helper, use lodash (lodash/shuffle, lodash/flatMap), existing composables (shared/composables/useFetch), or a maintained library (@internationalized/number for locale numbers)."

Accessible names, live regions and focus follow ARIA/WCAG patterns (6 PRs, 3 authors)

AGENTS.md a11y: "Live regions (role="status"/aria-live) stay mounted empty and only their text changes, with one writer per region. Key sibling v-ifs. Never unmount the focused control. Use KDS widgets (KDropdownMenu, KTabsPanel, KCard :to) for their ARIA wiring. Accessible names start with the visible text. Focus stays visible on every state." Lint: vuejs-accessibility plugin.

Ngôn ngữ chính
Python
Star
192
Fork
310
Merge trung bình
2 ngày 19 giờ
Pull request đã merge (30 ngày)
70

Chuẩn bị môi trường

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.

Issue khác của learningequality/studio

Tất cả issue của learningequality/studio

Issue tương tự

Thêm issue về Python

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.