vhx / quartz-react

Quartz components using React
2 stars 1 forks source link

Tags: add tags and example usage #6

Closed sebastiansandqvist closed 7 years ago

sebastiansandqvist commented 7 years ago

This is the first component that has had to be stateful. The reason I think that is acceptable is because that state is just the hover state that's used internally in the component.

The alternative would be to have a higher-order component manage that state, but unlike checkboxes where a parent component might care about the checked value, you probably never will care about the hover state.

Some changes from the original Quartz Tag (http://quartz.vhx.tv/js/components) include:

sebastiansandqvist commented 7 years ago

Jira task VHX-894