Rule-based glyph error outline is hue-only; red status text below 4.5:1 (#1605 audit D4, S8)

Đang mở
  • Có gì không ổn?
#2,140 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
Nửa ngày
Mức phù hợp với người mới
85/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
java
Lĩnh vực
accessibility

Hướng nghiên cứu

Start with the accessible error-text color added to GuiConstants by the #1605 remediation, then replace Color.red in StructureMappingTableRenderer, MathOverridesPanel, MultiPurposeTextPanel, DefaultScrollTableCellRenderer:126, DefineROI_SummaryPanel and NumericsTestCellRenderer. For D4, edit the error-state outline branch in vcell-core/.../cbit/vcell/graph/MolecularTypeLargeShape.java (lines ~666-690) so error differs by stroke width or a glyph, not only Color.red.darker(). Done when the rg command in Acceptance returns no matches and the outline check passes; optionally read docs/accessibility/color-audit.md (branch docs/1605-color-audit) for rows D4/S8.

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

Mô tả

Accessibility Requirement Java-based User Interface

Follow-up from the #1605 color audit, rows D4 and S8 (docs/accessibility/color-audit.md). These are smaller findings the audit surfaced outside the model and results viewer remediation.

D4: error outline on rule-based molecule glyphs is hue-only. In vcell-core/.../cbit/vcell/graph/MolecularTypeLargeShape.java (around lines 666–690), a molecule with errors is outlined in Color.red.darker() instead of dark gray: 1.43:1 between the two outlines. The same error is listed with an icon in the issue tables, but on the glyph itself only hue changes. Fix: a thicker or dashed outline, or an error glyph, for error state.

S8: red status text below 4.5:1 (WCAG 1.4.3). Each of these already states its meaning in words or bold, so WCAG 1.4.1 is met, but Color.red is 4.00:1 on white and 3.45:1 on #eeeeee:

  • StructureMappingTableRenderer ("Unmapped")
  • MathOverridesPanel ("remove unused parameter overrides" button text)
  • MultiPurposeTextPanel (error line, bold)
  • DefaultScrollTableCellRenderer:126 (changed network-constraint value, bold)
  • DefineROI_SummaryPanel
  • NumericsTestCellRenderer (internal test UI)

Fix: switch them to the accessible error-text color added to GuiConstants by the #1605 remediation.

Acceptance.

  • rg -n "setForeground\((java\.awt\.)?Color\.(red|RED)\)" --type java returns no matches in these files.
  • The D4 error state differs from normal by stroke or glyph, not only hue.

Part of #1603.

Ngôn ngữ chính
Java
Star
140
Fork
35
Merge trung bình
22 giờ 6 phút
Pull request đã merge (30 ngày)
14

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 virtualcell/vcell

Tất cả issue của virtualcell/vcell

Issue tương tự

Thêm issue về Java

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.