Material theme: add icon support

dojo / themes

Dojo Framework Themes

11 stars 19 forks source link

Material theme: add icon support #62

Closed msssk closed 4 years ago

msssk commented 5 years ago

Type: feature

The following has been addressed in the PR:

[x] There is a related issue
[ ] All code has been formatted with prettier as per the readme code style guidelines
[ ] Unit or Functional tests are included in the PR

Description:

add material-design-icons to dev dependencies
update src/material/icon.m.css to load Material Icons font and apply appropriate fonts to icon component CSS classes

Resolves #61

msssk commented 5 years ago

If we update the structure of the icon component to insert the icon name as the content of the <i> element then we don't need all of the CSS classes in this theme.

tomdye commented 4 years ago

@msssk please re-raise this against widgets. Themes has been ported there.