[Bugs] #1725 UNSP: Sugar main window fails to resize on randr resolution change

[Sugar Labs Bugs]

#1725: Sugar main window fails to resize on randr resolution change
------------------------------------------+---------------------------------
    Reporter:  bernie                     |          Owner:  aa                         
        Type:  defect                     |         Status:  accepted                   
    Priority:  Unspecified by Maintainer  |      Milestone:  Unspecified by Release Team
   Component:  sugar                      |        Version:  Unspecified                
    Severity:  Unspecified                |       Keywords:  r?                         
Distribution:  Unspecified                |   Status_field:  Unconfirmed                
------------------------------------------+---------------------------------
Changes (by sascha_silbe):

 * cc: sascha_silbe (added)


Comment:

 aa, thanks for working on this issue. I'm not sure your approach is the
 best one, though:

 >    def __screen_size_change_cb(self, screen):
 >        # Give the system some time to change the screen resolution.
 >        # Otherwise, when going to a higher resolution the resize may
 >        # not be performed correctly.
 >        sleep(1)
 >        self.resize(screen.get_width(), screen.get_height())

 Why do we get called before the resize is complete? This smells like an
 upstream bug.
 I also don't like using a fixed timeout, especially with time.sleep(). It
 artificially slows down fast systems and is likely to break on slow ones
 under high load.

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