Make the default `size` be equal to 1, not null

Please make the size prop default to 1 if no value is provided. If you leave size off, the icon simply will not render (0 viewbox).

In most cases,

<Icon path={mdiMenu} />

is preferable to

<Icon path={mdiMenu} size={1} />

[mririgoyen]

It seems extremely silly to have to specify widths for every use of every icon. I want to use an icon, and I expect it to appear when I put it in my code. If I want to further style that icon, I can then choose to add the optional size prop or use CSS.

Currently, a page with 15+ icons on it either has to look like:

<Icon path={mdiMenu} size={1} />
<Icon path={mdiAccount} size={1} />
<Icon path={mdiAttachment} size={1} />
...

or

<Icon className='my-class' path={mdiMenu} />
<Icon className='my-class' path={mdiAccount} />
<Icon className='my-class' path={mdiAttachment} />
...

The latter having additional CSS just to get over the hurdle of making the icon appear. Both have more boilerplate than I think should be needed for an out-of-the-box and quick way to get an icon to appear in my app.

---

EDIT: Perhaps a default class goes on all icons of “mdi”, so that at least all MDI icons can be targeted with CSS instead of needing to specify classes on each use.

So, with this proposal…

<Icon path={mdiMenu} />

Would render:

<svg viewbox="0 0 24 24" class="mdi">...</svg>

and

<Icon className='my-class' path={mdiMenu} />

Would render:

<svg viewbox="0 0 24 24" class="mdi my-class">...</svg>

I also like this approach, even though it does still require me to put something in my stylesheet just to get an icon to appear, which I find weird.

[Templarian]

Prior to the v1.0 release of this it was defaulting to 1 for size, but was causing a lot of issues for people using it alongside their frameworks since it added the style attribute.