Remove unreachable general-constraints UI (cbit.vcell.constraints.gui) (#1605 audit S6)

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#2,137 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
80/100
Loại issue
Tái cấu trúc
Độ 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
frontend

Hướng nghiên cứu

Bắt đầu bằng việc chạy rg -n "ConstraintPanel|ConstraintTableCellRenderer" --type java và mở rộng tìm kiếm sang các đăng ký menu/hành động để xác nhận không có gì (bao gồm phản xạ hoặc cấu hình menu XML) tham chiếu tới vcell-client/src/main/java/cbit/vcell/constraints/gui/. Nếu sạch, hãy xóa gói đó và kiểm tra BoundsNode cùng GeneralConstraintNode trong vcell-core/.../constraints/graph/ xem còn người dùng khác không trước khi gỡ bỏ chúng. Xong nghĩa là lệnh rg trả về không gì và mvn compile test-compile -pl vcell-client -am chạy qua.

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

Follow-up from the #1605 color audit, row S6 (docs/accessibility/color-audit.md).

Finding. The general-constraints UI in vcell-client/src/main/java/cbit/vcell/constraints/gui/ (ConstraintPanel, ConstraintTableCellRenderer, GeneralConstraintsTableModel, …) is not reachable. Nothing outside that package references ConstraintPanel. The renderer's red "inconsistent" text (4.00:1) is therefore not an accessibility problem in the shipped client, but it keeps showing up in color searches and audits.

Task. Confirm there is no reflective or menu entry point, then delete the package. Also delete the graph nodes it alone uses (vcell-core/.../constraints/graph/BoundsNode, GeneralConstraintNode) if they have no other users.

Acceptance.

  • rg -n "ConstraintPanel|ConstraintTableCellRenderer" --type java returns no matches.
  • mvn compile test-compile -pl vcell-client -am succeeds.

Part of #1603 (audit hygiene).

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.