dojo / widgets

:rocket: Dojo - UI widgets.
https://widgets.dojo.io
Other
88 stars 66 forks source link

1733 - Assume 24px icon size for icon only button in material theme #1740

Closed KaneFreeman closed 3 years ago

KaneFreeman commented 3 years ago

Type: feature

The following has been addressed in the PR:

Description: Assume 24px icon size for icon only button in material theme (per material specs). Change icon button example to reflect this.

Resolves #1733

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

widget-test-docs – ./

πŸ” Inspect: https://vercel.com/dojo/widget-test-docs/85xtAiefrnu79bqvwH8YJKfS7eJC
βœ… Preview: https://widget-test-do-git-fork-kanefreeman-bugfix-1733-icon-onl-12588b.vercel.app

dojo.widgets – ./

πŸ” Inspect: https://vercel.com/dojo/dojo.widgets/7MqFYMRxVmSHJtZAELXcDedtE8Hm
βœ… Preview: https://dojowidgets-git-fork-kanefreeman-bugfix-1733-icon-onl-c378f4.vercel.app

codecov[bot] commented 3 years ago

Codecov Report

Merging #1740 (67abdb2) into master (7fbe4e9) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1740   +/-   ##
=======================================
  Coverage   90.24%   90.24%           
=======================================
  Files          94       94           
  Lines        5116     5116           
  Branches     1392     1392           
=======================================
  Hits         4617     4617           
  Misses        244      244           
  Partials      255      255           

Continue to review full report at Codecov.

Legend - Click here to learn more Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data Powered by Codecov. Last update 7fbe4e9...67abdb2. Read the comment docs.