[Sugar-devel] [PATCH] Fix for Ticket #1927

[James Cameron]

On Thu, May 06, 2010 at 10:39:43AM +0530, Anish Mangal wrote:
> James wrote:
> > A fix is to reorder the widget signals so that flash_cb() is called
> > before clearbutton_cb(). ??See patch below.
> 
> Incorporated as suggested. I see no change at my end in
> sugar-emulator, so please verify it.

I've verified it.  Your latest patch dcca838 properly highlights the
clear button using the coloured icon.

There is a short delay before the highlight begins, but I judge that
this is because of .save().

The accelerators make it easier to demonstrate the delay.  Typing Alt/S
Alt/R Alt/C in rapid succession results in Stop and Run highlighted,
then a pause, then they return to normal at the same time Clear is
highlighted, then 400ms later Clear returns to normal.

Typing Alt/C Alt/R Alt/S in rapid succession results in a pause, then
Clear, Stop and Run are highlighted together, then 400ms later they
return to normal.

You could probably simulate this in sugar-emulator by adding a
time.sleep(0.5) call at the call to .save().

-- 
James Cameron
http://quozl.linux.org.au/