-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add Chips page #103
feat: Add Chips page #103
Conversation
src/ChipsCatalog.js
Outdated
this.chipSet = new MDCChipSet(chipSetEl); | ||
} | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: you can pull this all out of the constructor and just do this
initChipSet = (chipSetEl) => chipSetEl && this.chipSet = new MDCChipSet(chipSetEl);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, except I think React doesn't like assignments next to a logical operator.
src/ChipsCatalog.js
Outdated
<div> | ||
<h3>Choice Chips</h3> | ||
<div className='mdc-chip-set mdc-chip-set--choice' ref={this.initChipSet}> | ||
{this.renderChip('Extra Small')} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
something weird is happening with the ripple on choice chips...not sure if that is catalog's fault or the mdc-chip's fault
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to be the 1-pixel gap issue that's also happening in button. Fixed in Chrome v67.
Display filter, choice, and action chips. Fixes #25.
Input chips are still WIP and the API may change significantly.