Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide a way to replace part of (or all) icons used by components with icons from other icon sets #13987

Closed
ChronosSF opened this issue Mar 13, 2024 · 0 comments · Fixed by #14035
Assignees
Labels
🖼️ icon 🏃 priority: high styling 🧰 feature-request version: 18.1.x ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@ChronosSF
Copy link
Member

Is your feature request related to a problem? Please describe.

There is no simple way to have a sweeping replace of icons if apps' styling guidelines require the usage of an icon library that is not Material. Templating can be cumbersome solution and not all icons are exposed as part of templates anyway.

Describe the solution you'd like

The implementation could be something like an option with a mapping between icon names we use and what should be displayed (say a list of css classes) for igx-icons for these names. TBD

Describe alternatives you've considered

Alternatively every single icon should be templatable though that makes a full replace more difficult.

@simeonoff simeonoff self-assigned this Mar 25, 2024
@ChronosSF ChronosSF added 🛠️ status: in-development Issues and PRs with active development on them version: 18.0.x triage: blocking labels May 17, 2024
@simeonoff simeonoff added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. and removed 🛠️ status: in-development Issues and PRs with active development on them labels May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖼️ icon 🏃 priority: high styling 🧰 feature-request version: 18.1.x ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants