TBv3: results-table header, opened row and toolbar selection are near-identical lavenders

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

Chưa có ai nhận issue này.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
35/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
javascript, react
Lĩnh vực
design

Hướng nghiên cứu

Start with src/components/search/TableResults.jsx and src/components/search/SearchResults.jsx to compare the header, opened-row, and checked-row colors, then review the design-system tokens. The issue leaves the design choice open and allows closing as won't-do; discuss which direction to take before changing code. If a color change is chosen, record it on the design system's TableResults page (ocl-design-system#17) and include before/after screenshots showing all three states and hover.

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

Mô tả

component/web decision/human-review signal/low-risk signal/needs-human signal/small-scope stage/triaged type/feature

Follow-up to ocl_issues#2948 (oclweb3#71), which gave results-table header rows a tinted background. Three nearby colours are now near-identical lavenders:

  • Header row: surface.s90 (#e3e0f9), in src/components/search/TableResults.jsx.
  • Opened row (the item shown in the side panel): primary.90 (#e2dfff), in the same file (ResultRow).
  • Results toolbar with rows checked: primary.main at the theme's activatedOpacity, in src/components/search/SearchResults.jsx.

When the first row is the opened one and some rows are checked, the toolbar, header and opened row read as one lavender block. oclweb3#71 added 1px surface.nv80 rules under the toolbar and header so the bands stay apart, but the colours themselves don't separate them. A before/after zoom is in oclweb3#71.

To consider: give the opened row (or the toolbar's selection state) a tone that's clearly different from the header, using design-system tokens, and record the choice in the design system's TableResults page (ocl-design-system#17). If we decide the rules are enough, close this as won't-do.

If we change it

  • The header, the opened row and the toolbar's selection state are each clearly distinguishable at a glance, light theme.
  • Hover and checked-row states stay distinct from all three.
  • The ocl-design-system TableResults page shows the new colours.
  • Before/after screenshots in the PR.
Ngôn ngữ chính
Không có dữ liệu ngôn ngữ
Star
4
Fork
2
Merge trung bình
2 giờ 12 phút
Pull request đã merge (30 ngày)
1

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 OpenConceptLab/ocl_issues

Tất cả issue của OpenConceptLab/ocl_issues

Issue tương tự

Thêm issue về Design

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.