[Bugs] #650 UNSP: Screenshots to use name of current activity as part of its default title

[SugarLabs Bugs]

#650: Screenshots to use name of current activity as part of its default title
------------------------------------------+---------------------------------
    Reporter:  garycmartin                |          Owner:  jzGreen    
        Type:  enhancement                |         Status:  accepted   
    Priority:  Unspecified by Maintainer  |      Milestone:  0.86       
   Component:  sugar                      |        Version:  0.84.x     
    Severity:  Minor                      |     Resolution:             
    Keywords:  sugar-love, r?             |   Distribution:  Unspecified
Status_field:  New                        |  
------------------------------------------+---------------------------------
Changes (by erikos):

 * cc: erikos (added)


Comment:

 Nitpicks:

 - I like the if-else way Bernie suggested
 - the handling of the title should be out of the try-finally clause, as it
 is logically not part of that
 - please pay attention to trailing whitespace errors, tabs etc
 - pylint helps to locate unneeded imports and few other common errors

 I have attached a reworked patch with the changes from above

 Thanks very much for the patch, nice work!

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