Skip to content
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

V2 #4

Merged
merged 63 commits into from
Apr 25, 2017
Merged

V2 #4

merged 63 commits into from
Apr 25, 2017

Conversation

jaames
Copy link
Owner

@jaames jaames commented Apr 25, 2017

A complete rewrite of the library:

  • The library is separated into modules, which makes it much easier to work on
  • Added webpack build script, with dev server and other neat things
  • Full API docs added
  • Color wheel rendering has been greatly improved
  • Stylesheet API got some under-the-hood improvements, and some extra methods thrown in
  • IE9 support was dropped in favor of file size -- V1 will be kept around for this reason
  • Docsite was dropped in favor of just using github. The fancy Vue & vue-router site sure looked cool, but it was terrible for SEO. I may use hexo in the future, though
  • Landing page has been completely redesigned so be more colorful
  • Added quick start guide to readme

This has been a lot of work. I'm so glad that it's finally ready (:

 - rgb
 - hsl
 - rgb string
 - hsl string
 - hex rgb string
 - stylesheets were getting updated with the previous color selection,
not the current one

 - minor watch method cleanup for color.js and colorPicker.js

 - hslString.to fixed

 - space-saving cleanup for colorPicker.js and stylesheet.js
I've decided to remove tests for the time being, for the following
reasons:

 - jsdom has a number of holes, primarily in its support for the
stylesheet API, which makes it difficult to test `iro.stylesheet`.
 - Canvas rendering needs to be tested but it's difficult to do that
programmatically.
 - Browser-based testing might be better
jaames and others added 28 commits April 6, 2017 14:36
converting the color to hsl to draw the wheel makes very little
difference visually
 - color models also have a new `name` property which is used internally
 - build script has been updated to compress these property and method
names, plus others that are only intended to be used internally
slider `type` props are now `sliderType`
(with the intention of making stylesheet.js *just* handle stylesheets
not any features that are unique to colorPicker.js)
@jaames jaames merged commit a4e8623 into master Apr 25, 2017
@jaames jaames deleted the v2 branch April 25, 2017 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant