[vlc-devel] [PATCH] Fix for Boss key not minimizing the player on full screen mode in linux

Alexandre Janniaux ajanni at videolabs.io
Tue May 5 20:23:41 CEST 2020


Hi,

I suppose this is a fix for VLC 3.0?

You might want to write it against VLC master too so as to
avoid seeing this bug happen again in future version.

Note that I didn't check the current behaviour and patch yet.

Regards,
--
Alexandre Janniaux
Videolabs

On Sun, Apr 26, 2020 at 05:56:56PM +0200, nicolas.legoueix77 at gmail.com wrote:
> From: Nicolas Legoueix <nicolas.legoueix77 at gmail.com>
>
> Fix for the bug issue #21126 : https://trac.videolan.org/vlc/ticket/21126
> In fullscreen mode on linux, using the boss key would only pause the player instead of minimizing completely, forcing the user to press it twice to get the desired effect.
>
> ---
>  modules/gui/qt/main_interface.cpp | 10 +++-------
>  1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/modules/gui/qt/main_interface.cpp b/modules/gui/qt/main_interface.cpp
> index 259dcf7109..2968210bd4 100644
> --- a/modules/gui/qt/main_interface.cpp
> +++ b/modules/gui/qt/main_interface.cpp
> @@ -1707,14 +1707,10 @@ void MainInterface::emitBoss()
>  void MainInterface::setBoss()
>  {
>      THEMIM->pause();
> -    if( sysTray )
> -    {
> +
> +    showMinimized();
> +    if(sysTray)
>          hide();
> -    }
> -    else
> -    {
> -        showMinimized();
> -    }
>  }
>
>  void MainInterface::emitRaise()
> --
> 2.20.1
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list