mrn icon indicating copy to clipboard operation
mrn copied to clipboard

Add ability to hide Toolbar navIcon

Open mrcasals opened this issue 9 years ago • 1 comments

PR #14 added the ability to overwrite the default Toolbar navIconName. I want to use a Toolbar without any navIcon, so I was thinking that the default behaviour could be no icon, and if the user wanted an icon then they would need to explicitly name it:

Toolbar without an icon:

        <Toolbar
          actions={this.props.actions}
          primary="paperLightBlue"
          title={this.props.title}
        />

Toolbar with an icon:

        <Toolbar
          navIconName="menu"
          actions={this.props.actions}
          primary="paperLightBlue"
          title={this.props.title}
        />

What do you think?

mrcasals avatar Dec 04 '15 10:12 mrcasals

Yes, You are right. It was my original intention. I forget it.

binggg avatar Dec 04 '15 10:12 binggg