[Bugs] #1948 UNSP: Race condition with name widget in the activity toolbar

[Sugar Labs Bugs]

#1948: Race condition with name widget in the activity toolbar
------------------------------------------+---------------------------------
    Reporter:  bernie                     |          Owner:  erikos     
        Type:  defect                     |         Status:  reopened   
    Priority:  Unspecified by Maintainer  |      Milestone:  0.88.x     
   Component:  sugar-toolkit              |        Version:  0.88.x     
    Severity:  Critical                   |     Resolution:             
    Keywords:  olpc-0.84                  |   Distribution:  Unspecified
Status_field:  New                        |  
------------------------------------------+---------------------------------
Changes (by bernie):

  * status:  closed => reopened
  * resolution:  fixed =>


Comment:

 Replying to [comment:13 erikos]:
 > Bernie, there is an issue with your patch. Using the focus out event is
 critical when the kid does the following: open activity, change title,
 close activity. The focus out event will not be triggered.

 Ouch, this is true.

 On the other hand, I think that using a 1-second timeout to save also
 opens a race-condition: change the name, then quickly press ctrl-q before
 the timeout expires.

 The timeout approach also results in spurious saves which make the
 activity hang for a while if the user isn't fast at typing.

 Isn't there another event which is guaranteed to fire when the user
 finishes editing a textfield? If not, couldn't we rework my patch add an
 extra check on activity close time?


 > Why not use the patch from Aleksey in 0.84, too? Did t land already?

 I not planning any new releases of Sugar 0.84 for the XO-1. Perhaps OLPC
 might look into it for the XO-1.5.

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/1948#comment:16>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system