[Bugs] #1335 HIGH: When in fullscreen mode in an activity, frame doesnt pop-up

[Sugar Labs Bugs]

#1335: When in fullscreen mode in an activity, frame doesnt pop-up
------------------------------+---------------------------------------------
    Reporter:  HoboPrimate    |          Owner:  alsroot          
        Type:  defect         |         Status:  assigned         
    Priority:  High           |      Milestone:  0.86             
   Component:  sugar-toolkit  |        Version:  Git as of bugdate
    Severity:  Unspecified    |       Keywords:  r+               
Distribution:  Unspecified    |   Status_field:  New              
------------------------------+---------------------------------------------
Changes (by tomeu):

  * keywords:  r? => r+
  * owner:  erikos => alsroot


Comment:

 Patch looks great to me, I would just not have those returns in the middle
 of the method, it makes the reader have to scan the whole method to know
 which exit points exist:

 {{{
         152             if not self.props.enable_fullscreen_mode:
         153                 return
 }}}

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