[vlc-devel] [PATCH] hotkeys: fullscreen state of the vout should have precedence over fullscreen state of the playlist

Felix Abecassis felix.abecassis at gmail.com
Mon Jan 27 16:03:42 CET 2014


The state of the playlist is now set using the state of the vout (if
it exists) instead of doing the opposite, we need this because the
DoubleClick event only changes the state of the vout since commit
http://git.videolan.org/?p=vlc/vlc-2.1.git;a=commit;h=7b8083d228cf76798e33a161772bb53e02928c39.
This is what's causing bug https://trac.videolan.org/vlc/ticket/10433

Are you saying we should toggle the state of the playlist in both cases?

2014-01-27 Rémi Denis-Courmont <remi at remlab.net>:
> On Mon, 27 Jan 2014 14:16:12 +0100, Felix Abecassis
> <felix.abecassis at gmail.com> wrote:
>> Close #10433. Double clicking to toggle to fullscreen mode only changed
>> the state of the vout.
>
> Err, the diff removes 'bool fs = var_ToggleBool( p_playlist, "fullscreen"
> );' which changed the state of playlist... So this cannot be right.
>
> --
> Rémi Denis-Courmont
> Sent from my collocated server
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel



-- 
Félix Abecassis
http://felix.abecassis.me



More information about the vlc-devel mailing list