Skip to content

Split up mdi icons#4379

Merged
bramkragten merged 13 commits intodevfrom
icons
May 5, 2020
Merged

Split up mdi icons#4379
bramkragten merged 13 commits intodevfrom
icons

Conversation

@bramkragten
Copy link
Copy Markdown
Member

@bramkragten bramkragten commented Dec 24, 2019

To do:

  • Better way to split up the icons so the size of the different parts is more equal
  • Make everything that uses icons use ha-icon (like paper-icon-button)
  • Replace all hass: icons with imports from @mdi/js so they are code splitted (and look in to caching them when used in ha-icon so when a user uses it they wont need to load a part?)
  • Create a fallback for none mdi icons
  • Remove svg iconsets
  • Cleanup and (http) error handling

Comment thread build-scripts/gulp/gen-icons-json.js Outdated
Comment thread src/components/ha-icon.ts Outdated
Comment thread src/components/ha-icon.ts Outdated
Comment thread src/components/ha-icon.ts Outdated
Comment thread src/components/ha-icon.ts Outdated
Comment thread src/components/ha-icon.ts Outdated
Comment thread src/components/ha-icon.ts Outdated
@bramkragten bramkragten changed the title WIP: Split up mdi icons Split up mdi icons May 5, 2020
@bramkragten bramkragten merged commit 0a92c28 into dev May 5, 2020
@bramkragten bramkragten deleted the icons branch May 5, 2020 14:40
@bramkragten bramkragten mentioned this pull request May 5, 2020
return curString.substring(0, i + 1);
}
}
console.error("Cannot find differentiator", curString, prevString);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that it could happen, but we should probably raise here to make sure that it breaks.

@github-actions github-actions Bot locked and limited conversation to collaborators Jul 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants