-
Notifications
You must be signed in to change notification settings - Fork 14
Conversation
… with a flag in the API.
packages/clay-table/package.json
Outdated
"metal" | ||
], | ||
"dependencies": { | ||
"clay-button": "1.0.0-alpha.8", |
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.
hey @carloslancha, Upgrade to the latest versions of Clay to not have some problems in compiling and jest.
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.
Thx @matuzalemsteles I didn't realize versions changed while developing the component ;)
Just started reviewing :) |
packages/clay-table/src/ClayTable.js
Outdated
class ClayTable extends Component { | ||
/** | ||
* @inheritDoc | ||
* Handles document click in order to hide menu. * Handles document click in order to hide menu. */ |
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.
Just organize. 🙂
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.
D'oh!
Awesome working @carloslancha. Merged, thanks! |
Developed with clay#develop
Missing features: