[2.0] Give repeated inline logotypes unique SVG IDs

Đang mở
  • Có gì không ổn?
#479 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
56/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
html
Lĩnh vực
frontend

Hướng nghiên cứu

Start with site/components/CSSLogotype.svelte around lines 26-32, where the raw SVG artwork is inserted, and check how the file-level SVGO prefix is applied to the light and dark assets. Then run site/tests/docs-quality.e2e.mjs against a local build of site/out to see the no new or changed duplicate document IDs failure. Done means each rendered logotype instance has unique IDs and local references, and that assertion passes in both themes at 390, 768 and 1280 px without a baseline.

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

Mô tả

Parent: #473

Reproduction

Build the candidate site, serve site/out, and run MASTER_SITE_URL=<local-preview> node site/tests/docs-quality.e2e.mjs with no duplicate-ID baseline. Current documentation pages fail no new or changed duplicate document IDs in light and dark themes at 390, 768 and 1280 px.

A static scan finds duplicate IDs in 151 of 155 built pages. Typical document routes repeat eight master-css-horizontal-on-{light,dark}_svg__{a,b,c,d} IDs twice. The homepage additionally repeats some integration artwork IDs.

Source evidence

site/components/CSSLogotype.svelte:26-32 inserts raw SVG artwork without an instance-specific prefix. The file-level SVGO prefix separates light and dark assets, but does not separate multiple instances of the same asset. Shared header and footer both render the product logotype.

Acceptance

  • Make each inline SVG instance's IDs and local references unique, or use an external-image representation where appropriate.
  • Inspect other repeated homepage artwork.
  • Run the documentation browser audit without a duplicate-ID baseline and verify both color modes.

This is a document-integrity and validation defect. Visible gradient corruption has not been established. Temporary audit-only exact baselines may allow later visual assertions to run, but do not resolve this finding.

Evidence: local candidate 9567e1158f995272c45d341e0eeb3a02a49a8dd0 with pre-existing working-tree changes, 2026-11-04.

Completed full browser run

All 148 documentation routes were tested at 390/768/1280 px in both themes: 888/888 profiles fail at the duplicate-ID assertion. Assertions preceding it (response, geometry, broken article images, browser errors) did not fail. With an exact per-route baseline retained separately, six representative routes pass all 36 profiles including later illustration checks. The unmodified gate remains failing.

Ngôn ngữ chính
TypeScript
Star
2k
Fork
44
Không có pull request nào được merge trong 30 ngày

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 master-co/css

Tất cả issue của master-co/css

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.