<div dir="ltr"><div>Hi,</div><div>Will it be okay, if I add a toggle option in the look and feel section which comes in Preferences->Interface? If not, what can be done to make the feature more convenient?<br></div><div><div><div><br><div class="gmail_quote"><div dir="ltr">On Sun, Jul 15, 2018 at 7:32 PM Alexandre Janniaux <<a href="mailto:alexandre.janniaux@gmail.com">alexandre.janniaux@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi,</div><div><br></div><div>Thank you for taking the time to make this patch!
However, it doesn't seem to be what the ticket is saying. It will force 
this new behaviour for Qt whereas the ticket's suggestion is to add a new 
parameter to enable this feature.</div><div><br></div><div>There is also the behaviour consistency issue when using another interface as mentionned by Marvin.</div><div><br></div><div>I
 don't know if this feature is really wanted by many people, but maybe 
you could add this new parameter and update the other interfaces. I 
believe it would be difficult to find a common location for it as this 
feature might not really be wanted for libvlc and is only related to interfaces themselves.</div><br></div><div class="gmail_extra"><br><div class="gmail_quote">2018-07-15 15:48 GMT+02:00 Vaishnav Sivadas <span dir="ltr"><<a href="mailto:vaishnavsivadas@gmail.com" target="_blank">vaishnavsivadas@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi,</div><div><br></div><div>What change are you suggesting which could improve the patch?<br></div><div><div class="m_-303802876538975579h5"><br><div class="gmail_quote"><div dir="ltr">On Sun, Jul 15, 2018 at 7:13 PM Marvin Scholz <<a href="mailto:epirat07@gmail.com" target="_blank">epirat07@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi, thanks for your patch.<br>
<br>
I am not sure though if doing this in a specific interface only<br>
(in this case the qt interface) is a good solution.<br>
<br>
On 15 Jul 2018, at 15:40, Vaishnav Sivadas wrote:<br>
<br>
> Autoplays a video upon entering full screen which enhances UX<br>
><br>
> fixes #8533<br>
> ---<br>
>  modules/gui/qt/input_manager.cpp | 6 ++++++<br>
>  1 file changed, 6 insertions(+)<br>
><br>
> diff --git a/modules/gui/qt/input_manager.cpp <br>
> b/modules/gui/qt/input_manager.cpp<br>
> index 4c28933395..c94cffe164 100644<br>
> --- a/modules/gui/qt/input_manager.cpp<br>
> +++ b/modules/gui/qt/input_manager.cpp<br>
> @@ -1287,5 +1287,11 @@ int MainInputManager::PLItemRemoved( <br>
> vlc_object_t *obj, const char *,<br>
>  void MainInputManager::changeFullscreen( bool new_val )<br>
>  {<br>
>      if ( var_GetBool( THEPL, "fullscreen" ) != new_val)<br>
> +    {<br>
>          var_SetBool( THEPL, "fullscreen", new_val );<br>
> +    }<br>
> +    if (new_val)<br>
> +    {<br>
> +        playlist_Play( THEPL );<br>
> +    }<br>
>  }<br>
> -- <br>
> 2.17.1<br>
><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div><br clear="all"><br></div></div><span class="m_-303802876538975579HOEnZb"><font color="#888888">-- <br><div dir="ltr" class="m_-303802876538975579m_-8855671172158021463gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div>Vaishanav Sivadas<br></div><div>Amrita University.<br></div><div><a href="https://vaishnavsivadas.wordpress.com/" target="_blank">Wordpress</a> | <a href="https://github.com/vaishnavsivadas" target="_blank">Github</a> <br></div></div></div></div></font></span></div>
<br>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote></div><br></div>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div>Vaishanav Sivadas<br></div><div>Amrita University.<br></div><div><a href="https://vaishnavsivadas.wordpress.com/" target="_blank">Wordpress</a> | <a href="https://github.com/vaishnavsivadas" target="_blank">Github</a> <br></div></div></div></div></div></div></div></div>