* Percentage of completion: 95% (need to get tool tips and accelerators working)
* Percentage of completion: 95% (need to get tool tips and accelerators working)
+
+
=== Summary of the discussion with the Design Team ===
+
+
We have gotten a lot of [http://lists.sugarlabs.org/archive/sugar-devel/2009-December/021260.html feedback] on this feature from the Design Team. There is consensus regarding its merits, but a few outstanding issues remain to be resolved:
+
+
# Should the order of the icons be such that the next/prev buttons are on the outside (Eben) or between (Walter) the small and large XO icons (the latter is the order in the current implementation)? It is trivial to change. Gary proposed putting them under the icons due to considerations of centering.
+
+
# Should the behavior be such that the next/prev buttons get you the next/previous random color or the next/previous color in the region of the color space you are currently in (the latter being the behavior of the current implementation)? The motivation for the latter implementation is that it lets a user explore a region in the color space once they have randomly stumbled across a combination to their liking. The motivation for the former is that next/prev would give you a more standard behavior. A compromise could be to eliminate the random selection all together (except upon initialization); this would be trivial to implement--simply eliminate the callback from the center XO icon.
+
+
# Should there be be keyboard shortcuts? Yes, but I don't know how to implement them using Hippo Canvas. (There is a failed attempt in the current patch to do this by adding a hidden undo button. There was another purpose to the undo button, but it should be removed from the patch.)