Spacing scale: where do your spacing tokens live? (audit from the Rhythmguard benchmark)

Đang mở
#31,430 0 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ó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
45/100
Loại issue
Tài liệu
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Sôi nổi
Công nghệ
css
Lĩnh vực
design, frontend

Hướng nghiên cứu

Bắt đầu bằng cách kiểm tra core/src/components ở commit 5874331 và tìm kiếm --space-, --spacing-, các biến hoặc map spacing của Sass, cùng với một spacing base của Tailwind. Hoàn thành có nghĩa là xác định nguồn token cho tác giả issue, hoặc xác nhận rằng repository này cố ý không có spacing scale và phản hồi bằng kết quả đó.

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

Mô tả

triage

Spacing scale: where do your spacing tokens live? (audit from the Rhythmguard benchmark)

Hi. I maintain stylelint-plugin-rhythmguard, a Stylelint rule that checks spacing values against a project's own scale. To keep it quiet on code I do not control, I run it against public design systems on pinned commits and publish the numbers. This repository is one of them, and I would rather you saw the audit here than in a report first.

What was run. npx rhythmguard audit . --scale auto at 5874331 over core/src/components. Hairlines of one pixel or less, percentages, and generated or test paths are excluded. Anyone can reproduce it in a checkout of that commit.

What it found. The audit could not find a spacing token set here (it looks for --space-* / --spacing-* custom properties, Sass $spacer / $spacing-* variables and maps, or a Tailwind --spacing base), so it measured against a default 4px scale instead. Against that default it reported 8 literal values, a number that says more about my token discovery than about your CSS, so I am not treating it as a finding.

The ask. If you can point me at where the spacing scale is defined (a token file, a Sass map, a package), I will teach the tool to read it, re-run the audit on the real scale, and post the result here. If spacing is intentionally not on a scale, saying so is just as useful and I will mark the row that way.

The row for this repository will appear in a periodic "State of Spacing" table in the Rhythmguard repository, with this issue linked. If you would rather not be listed, say so here and I will remove it.

Feel free to close this if it is not useful. Thanks for the CSS.

Ngôn ngữ chính
TypeScript
Star
52.7k
Fork
13.3k
Merge trung bình
2 ngày 20 giờ
Pull request đã merge (30 ngày)
35

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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 ionic-team/ionic-framework

Tất cả issue của ionic-team/ionic-framework

Issue tương tự

Thêm issue về TypeScript

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.