[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                
------------------------------------------+---------------------------------

Comment(by aa):

 Replying to [comment:3 sascha_silbe]:
 > 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.

 Sigh. Sorry Sascha, you're absolutely right, of course. This was a bug I
 saw when running sugar under Kwin, and not the latest version (there's
 also a typo, so it doesnt even run).

 I updated the patch, and tested it under sugar-jhbuild and a VM running
 SoaS Blueberry.

 Would be nice if bernie could try it with an external projector or screen.

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