choonkeat/jquery-miniColors — comment on commit 50b9704eb4e95b030e748b48804e96f86b84c887
File:  line 0
By claviska:

While the goal of MiniColors is a minimalistic approach to a color picker, I'm not opposed to additional features that make it more usable. This seems like a good concept, but I have a few issues that prevent me from accepting this pull request:

1. This should probably be a setting (disabled by default)
2. It's possible to get an improper value when using the textbox. For example, try setting it to #ZZZ and then hitting done; read the value and you'll get '#' (acceptable values are a valid hex or empty). In fact, this could probably be refactored to use the same logic that controls non-hidden input validation.
3. It's not aesthetically pleasing. I'd like to see a better 'Close' icon instead of a button. We can work on this if it gets to that point though.