Re: Using VLC 4.0.0 on WSL2 Ubuntu 25.04 with wayland
Fatih Uzunoğlu <[email protected]> Tue, 24 Jun 2025 00:53:12 +0300
| Newsgroups | gmane.comp.video.videolan.vlc.devel |
|---|---|
| Message-ID | <AM6PR0202MB33684946F453B6FCC8818EE8AF79A@AM6PR0202MB3368.eurprd02.prod.outlook.com> |
> Of course not. The display is expected to pick a scale according to VLC video settings. The *default* scale, whether fetched from the settings, or from the interface window in embedding case, is supposed to come from the (same) environment (depending on high DPI display, for example). And this default value would naturally match for different windows in the same screen, the interface window or the video window. Of course, it can deviate if overridden by the user, but by default they should match so as embedded window fits perfectly. We should not present the user a situation where the video window exceeds the boundaries of the interface window, by default. > That's how it's supposed to work. That's how it works for all other displays, all other providers, Win32 and X11, and how the core makes its calculations. > If you don't respect that convention, this is 100% a bug in the Qt module. I shouldn't have to explain something like that. If we do that, we lose information. The video window can no longer know the scale, so technically we can not support high dpi. How does the OSD behave in high dpi scenario, for example, without knowing the scale? > You can't add restrictions on how the display module is using the surface for rendering. I wanted to mean not for rendering, but for something that legitimately concerns both the display module and the window provider. I just mean that if they are not compatible, it does not mean that one might go. As I said, "If in some situations video embedding is not possible, they should not be enabled or offered when the interface is used.". _______________________________________________ vlc-devel mailing list To unsubscribe or modify your subscription options: https://mailman.videolan.org/listinfo/vlc-devel