[vlc-devel] [PATCH] Fix for Boss key not working properly on full screen mode in linux

Alexandre Janniaux ajanni at videolabs.io
Mon Apr 20 19:36:50 CEST 2020


Hi,

Can you be more precise about the «not working» part? It works
correctly here so I'm not sure of what you're fixing.

Is it linked to a ticket too? You should reference the ticket
in that case.

Finally, you should not use From: like that.

Put one author in From: (or in general, author of the commit)
and the other in Co-authored-by: .

Regards,
--
Alexandre Janniaux
Videolabs


On Mon, Apr 20, 2020 at 06:47:35PM +0200, nicolas.legoueix77 at gmail.com wrote:
> From: Nicolas Legoueix and Yann Queric <nicolas.legoueix77 at gmail.com>
>
> ---
>  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