I have drawed cursors for the other tools. What do you think? If the patch its bad, I can send you the images. Gonzalo <div class="gmail_quote">On Wed, May 26, 2010 at 1:42 AM, James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>> wrote: <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">On Tue, May 25, 2010 at 06:40:44AM -0300, Gonzalo Odiard wrote: > Use standard cursors for pencil, brush, eraser and paint-bucket. The > other tools need a custom cursor. > Bugs #40, #296 and OLPC #4316, #8864 </div>Reviewed and tested ... 1. does fix <a href="http://dev.laptop.org" target="_blank">dev.laptop.org</a> #4316, 2. doesn't properly fix <a href="http://sugarlabs.org" target="_blank">sugarlabs.org</a> #296, some additional tuning of coordinates may be needed; drawing effect still appears at an offset, 3. doesn't fix <a href="http://sugarlabs.org" target="_blank">sugarlabs.org</a> #40, in that the other cursors continue to be invisible against similar backgrounds, 4. doesn't remove images/{pencil,brush,bucket,eraser}.png The coding style was a bit repetitive, here's an alternate method which uses a dictionary and wraps the code better: