Skip to content

Remove icon option disableSvgIcons #1510

@spliffone

Description

@spliffone

Prerequisites

  • I have read the Contributing Guidelines.
  • I have not leaked any internal/restricted information like screenshots, videos, code snippets, links etc.

Affected component

Drop the disableSvgIcons option from IconConfig

Summary

We should drop the disableSvgIcons option from IconConfig since it cause issues for applications which provide icons as SVG. Vendors which plan to use custom icons can create their own provider which call the addIcons method to override the existing icons. We should only ensure that addIcons allows to register the same icon name only once. To ensure the default component icons don't override the vendor icons.

Further, here are some icons examples where the automatic camel to kebap-case transformation would fail:

  • element-co2-outdoor
  • element-co2-indoor
  • element-room-3d-filled
  • element-room-3d

Version

No response

Steps to reproduce

Code reproduction example

What is the current bug behavior

What is the expected correct behavior

Metadata

Metadata

Assignees

Labels

breaking-changesMarks issues and PRs that are breaking the API

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions