[Bugs] #428 UNSP: connect('map') to activity widow not to current canvas

[SugarLabs Bugs]

#428: connect('map') to activity widow not to current canvas
------------------------------------------+---------------------------------
    Reporter:  alsroot                    |          Owner:  marcopg    
        Type:  defect                     |         Status:  new        
    Priority:  Unspecified by Maintainer  |      Milestone:  0.84       
   Component:  sugar-toolkit              |        Version:  0.83.x     
    Severity:  Unspecified                |     Resolution:             
    Keywords:  r+                         |   Distribution:  Unspecified
Status_field:  Needinfo                   |  
------------------------------------------+---------------------------------
Changes (by tomeu):

  * keywords:  r? => r+


Comment:

 Replying to [comment:10 alsroot]:
 > Replying to [comment:9 tomeu]:
 > > That's nice, but I'm a bit worried about other activities. Checking
 all of them can be quite a bit of work.
 > from soas-1's only 3 activity uses map event for set_cavas's widget and
 they are ok (in fact they uses that in #267 way)
 >
 > > What if in set_canvas we disconnect the previous handler if there's
 one and then connect to the new one?
 > problem still exists - if user wants set_canvas() to set current
 tab(after changing tab in ActivitToolbox) it will catch one read_file()
 per each tab changing

 Ok, if the only activity known as needing this was Write and this isn't
 needed there any more, then I think we should apply your patch.

-- 
Ticket URL: <http://dev.sugarlabs.org/ticket/428#comment:13>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system