patternfly / patternfly/patternfly-react

Icons - add support for custom icons

Đang mở
#11,055 5 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Stale
Ngôn ngữ chính
TypeScript
Star
862
Fork
392
Merge trung bình
4 ngày 8 giờ
Pull request đã merge (30 ngày)
9

Mô tả

Is this a new component or an extension of an existing one?
What is the existing component, if any?

Existing, MenuToggle, ToolbarFilter

When using PF tokens to create a custom theme, MenuToggle and ToolbarFilter use default icons in PF that currently cannot be changed via a prop since the prop does not exist, while components like Label, Alert, and Button, offer this functionality.

An alternative option was discussed at today's working session (cc @mcoker), which would include adding support for custom icons directly to react-icons and then importing any custom overrides.

Related: https://github.com/patternfly/patternfly-react/issues/10634

Describe the feature
A clear and concise description of the new feature. What is the expected behavior?

Are there visuals for this feature? If applicable, please include examples for each state and for varying widths

See below. MUI uses a different toggle icon for their menu toggle.

Include screenshots or links to Marvel or other mockups.
MUI:
Screenshot 2024-09-24 at 4 27 39 PM
Screenshot 2024-09-25 at 3 21 01 PM

PF:
Screenshot 2024-09-24 at 4 28 05 PM
Screenshot 2024-09-25 at 3 26 46 PM

Any other information?
Would it be possible to prioritize this work for an upcoming sprint to align with RHOAI priorities for the MUI theming effort @nicolethoen ?

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.

Hướng nghiên cứu

Bắt đầu bằng việc xem xét các phần triển khai của component MenuToggle và ToolbarFilter, sau đó đọc issue #10634 và cuộc thảo luận về việc mở rộng react-icons. Công việc được xem là hoàn tất khi người dùng có thể cung cấp các icon tùy chỉnh cho cả hai component thông qua một API đã được thống nhất và lập tài liệu, với hành vi nhất quán với hỗ trợ icon hiện có trong Label, Alert và Button.

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

Đánh giá

Công nghệ
react, typescript
Lĩnh vực
design, frontend
Loại issue
Tính năng
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

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.