[feat]: Update `<pf-v6-icon>` for PatternFly v6
@bennypowers đang làm issue này rồi.
Từ ngày 10/5/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Summary
Update `` to implement the PatternFly v6 Icon component.
Decorative or semantic icon display with size, color, and status variants. Supports inline and standalone usage.
Prior Art
| Source | Element | Status |
|---|---|---|
| PFE v4 | `` | Exists |
| cem | -- | -- |
| cockpit | -- | -- |
| chickadee | `` | Exists |
Requirements
- MUST implement PFv6 visual designs
- MUST provide end-user feature parity with `@patternfly/react-core` Icon
- MAY adjust element API to leverage web platform strengths
- SHOULD maintain CSS custom property theming compatibility
- SHOULD expose useful CSS shadow parts
Element-specific considerations
- Port early - used by many other elements (button, alert, label, progress-stepper, etc.)
- React `size`: `sm`, `md`, `lg`, `xl` plus status sizes
- React `color`: `default`, `blue`, `green`, `orange`, `red`, `purple`, `cyan`, `gold`
- React `status`: `custom`, `danger`, `info`, `success`, `warning` - drives color + aria
- React `isInline` - controls `display: inline` vs `inline-block`
- React `progress` prop wraps a Spinner inside the Icon - verify if still in v6
- Async icon loading (network fetch) must not block render; show slot/fallback during load
- `aria-hidden="true"` when decorative; surface `aria-label` for semantic usage
Checklist
Rename
- Delete `elements/pf-v5-icon/`
- Update `elements/package.json` exports
- Update `eslint.config.js` ignore globs if applicable
- Update docs `_snippets/` and `framework-integration/` references
Implementation
- API per `.claude/ADVICE.md`
- CSS uses v6 tokens, logical properties, nesting
- `@summary` and JSDoc on element class
Demos
- Demos match patternfly.org naming/structure
- CSS custom properties match computed styles from patternfly.org (Chrome MCP)
- Visual parity verified via Chrome MCP
Tests
- Public API covered (attributes, properties, events, slots)
- `a11ySnapshot` assertions
Reviews
- `/review-api` passes
- `/review-demos` passes
- `/review-a11y` passes
- `eslint` and `stylelint` pass
Ship
- Add changeset
- PR targets `staging/pfv6` (NOT `main`)
- Ngôn ngữ chính
- TypeScript
- Star
- 394
- Fork
- 107
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của patternfly/patternfly-elements
-
bug
patternfly/patternfly-elements#3158 · 1 người được giao ·
-
patternfly/patternfly-elements#3136 · 1 bình luận · 1 người được giao ·
-
docs
patternfly/patternfly-elements#3131 · 1 người được giao ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 55/100
patternfly/patternfly-elements#3122 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 38/100
patternfly/patternfly-elements#3118 ·
Tất cả issue của patternfly/patternfly-elements
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
vercel-labs/just-bash#464 ·
-
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
TanStack/tanstack.com#1293 ·