Check if window size changed and resize it

[jonata]

Fixes #1036.

Can I add PySide6/PyQt/PyGtk script sample together with play.py?

[ddennedy]

Yes please do add your player sample to help me verify this before merge.

[ddennedy]

I am using Pop!_OS 22.04. play_gtk.py crashes on me for some reason deep inside GTK3 libs. OK, that might just be my system. I installed pyqt6 and pyside6 using pip3. The Qt 6 examples get into a cycle where the resizeEvent updates sdl2, which calls SDL_SetWindowSize, but that causes another smaller resizeEvent until the size diminishes to not visible. I have to do something like this prevent it:

                self.blockResize = False
            
            def resizeEvent(self, event):
                if not self.blockResize:
                    self.blockResize = True
                    self.window().consumer.set('window_width', self.width())
                    self.window().consumer.set('window_height', self.height())
                    event.accept()
                else:
                    self.blockResize = False

This results in the player not filling the window:

Not a big deal, but I wonder if you have the same event cycle problem and have a better way to prevent it?

[jonata]

It would not work on Wayland. Are you using X11?

In any case, the problem on my side is what I put on the bug report, where the sdl render rect would not update when the window is resized. SDL_SetWindowSize with window_width and window_height is a workaround for this.

[ddennedy]

I am on X11.

the sdl render rect would not update when the window is resized

Well, that is confirmed working. I just want to head off any future bug report that these pyqt6 examples do not show video. For me, they only do for a fraction of a second because the size decreases very quickly. I can add my workaround to the examples.

[jonata]

Sure, yes, I agree. Thanks!