Make wraparound work if you're at the ends of the hue slider
Add JSDocs to color editor code, plus a few other code cleanups:
- Clean up two args that were being fed jQuery objects but treating them as some other type
- Remove unneeded bind() calls
- Change swatches prop to an arg of addSwatches(), since it's only used that one time
- Rename a few members for clarity (lastColor -> originalColor; addFieldListeners -> addListeners; colorRegEx -> COLOR_REGEX; _getNewOffset()'s args)
- Replace defaultColor "const" with a literal in getColor() (only place it was used)
- Simplify setColorAsHsv() by using $.extend() & removing unneeded intermediate vars
- Remove redundant synchronize() call in bindColorFormatToRadioButton()
- Remove redundant textfield content reset in commitColor()
- Remove seemingly unneeded `!colorLabel` check in InlineColorEditor.setColor()
- Split selectedColors var in InlineColorEditor.load() so its type doesn't vary
Color picker code cleanups:
- Move creation of ColorEditor DOM into ColorEditor instead of making each
  client do it on their own
- Change undo/redo key handling to a single bubble listener at the top of
  the colorpicker DOM, simplifying other key listeners (see #2200)
- Fix bug #2202 (unit tests leave stray global mouse listeners) and remove
  the fallback code in getColor() that was masking it
- Change usedColors() to a simpler impl using a map to uniquify
- Fix crashes in handleHslKeydown() due to wrong 'this' (see #2199)
- Rename setColor()'s arg for clarity
- Add bug #s to TODOs
Add underscores to private methods/fields in ColorEditor & InlineColorEditor.
Leaves a few methods public: ColorEditor.focus(), getColor(), setColorAsHsv(),
undo(), redo(); and InlineColorEditor.getCurrentRange() and its various
override methods.

Also rename a few other methods/args for clarity.
Add expand/collapse triangle icons to Find in Files search results headings
(which were already collapsible, just with no visual cue making it apparent).

Also clean up jsTreeSprites-related CSS styles: use constants that mean
what they say; remove unused (& broken) "disclosure-arrow-*" styles; move
"ins.jstree-icon" style into jsTreeTheme.less.

And also reduce code duplication in jsTreeSprites.svg.
Fix issue 2138 by updating this.hsv when commitColor call fails to update it.
Fix Shift-Tab key issue by setting focus to parent li tag instead of the .value label.
Add A-Z to the regular expression that detects 6-digit hex colors.
Fix hue slider issue again as Peter's changes require the second parameter of setColorAsHsv() to be explicitly set to false. By setting false we always keep the hue value changed by the user's up/down key input.
Add unit tests for cases in #2138
Code review fixes:
- rename InlineEditorTemplate -> ColorEditorTemplate
- rename $lastColor and .last_color to match originalColor
- rename _topUsedColors() -> _collateColors()
- remove setColorAsHsv()'s arg commitHsv: assume always false (changes the
  behavior only of opacity slider keyboard interaction at this point)
- add official public API for _commitColor() (without the unsafe option of
  leaving _hsv untouched)
- add official public API for getting $element (aka the root)
- move $opacitySlider init to a better line
Merge pull request #2205 from adobe/pflynn/colorpicker-docs

JSDocs, method name underscores, and code cleanup for Color Editor
For #2193 and #2196, make the main color editor focusable so it can take keyboard events, and block arrow keys from bubbling upward
Code review: break up long line for readability
Merge pull request #2206 from adobe/pflynn/find-triangles

Add expand/collapse triangle icons to Find in Files search results headings
Merge with master
Merge pull request #2217 from adobe/nj/issue-2193

Fix for #2193 and #2196: allow root to take focus, and block arrow keys that bubble up to it.
Merge pull request #2194 from adobe/nj/issue-2139

Make wraparound via keyboard work if you're at the exact end of the hue slider