Request for "mdc-icon-button"

[lynnmercier]

Can we pull the "navigation icon" and "action item" out of mdc-top-app-bar and into it's own "mdc-icon-button" component?

Having a button which triggers an unbounded ripple is useful on it's own.

[williamernest]

Should this be it's own package? It sounds like it should just be added as a variant to the current mdc-button

[kfranqueiro]

It wouldn't really have any styles in common with button, though, I suppose? Though I guess it'd basically be like flat button but with unbounded ripple instead...

I am worried about naming this clearly to avoid confusion with an actual icon-only MDC Button if that also becomes a thing, though.

I am wondering whether this warrants being its own component or if it's overkill, if all it's ostensibly doing is applying ink color + states and adding a set padding.

[williamernest]

The entire package is ~40 lines of scss. It seems like it's just a variant of button with an unbounded ripple.

https://gist.github.com/williamernest/d852281e711f4f10bd2ef255cc518885

[bonniezhou]

Looks like it'll stand as variant of button, i.e. mdc-button--icon.
(We've also decided that mdc-button--icon with nested mdc-button__icon is not awkward at all.)

Added to tracker.

[williamernest]

We're condensing this and icon-toggle into their own mdc-icon-button package. The icon button had ripple conflicts with button.