<div dir="ltr"><div>I tip overlay will be done for TV, because the current one does not apply.<br><br></div>And I changed my Eclipse settings for tabs this morning :p<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 6, 2014 at 3:54 PM, Edward Wang <span dir="ltr"><<a href="mailto:edward.c.wang@compdigitec.com" target="_blank">edward.c.wang@compdigitec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Geoffrey,<br>
<span class=""><br>
On Mon, Oct 6, 2014 at 2:54 AM, Geoffrey Métais<br>
<<a href="mailto:geoffrey.metais@gmail.com">geoffrey.metais@gmail.com</a>> wrote:<br>
> Here is the mapping<br>
><br>
> A : play/pause<br>
> B : Audio Track<br>
> X : toggle mute<br>
> Y : Select subtitle<br>
> R1 : seek forward<br>
> L1 : seek backward<br>
><br>
> Left stick:<br>
> Horizontal axis: seek<br>
> Vertical axis : brightness<br>
><br>
> Right stick:<br>
> Vertical axis: volume<br>
><br>
> Please tell me if you have a better idea of configuration<br>
> ---<br>
>  .../vlc/gui/video/VideoPlayerActivity.java         | 68 +++++++++++++++++++---<br>
>  .../src/org/videolan/vlc/util/AndroidDevices.java  | 22 +++++++<br>
>  2 files changed, 82 insertions(+), 8 deletions(-)<br>
<br>
</span>It looks mostly good. I had just a few comments in mind:<br>
<br>
1) The vertical axis for brightness should respect the setting in the<br>
preferences. Some people may not wish to have their brightness changed<br>
by VLC during playback or may prefer to manage it themselves, and it<br>
should respect the preference for that.<br>
<br>
2) This mapping should be documented somewhere. If I, as a user,<br>
didn't know this, it would be a lot of guesswork to figure out which<br>
button is doing what.<br>
<br>
3) Some tab/spacing problems seem to have snuck into the patch ;)<br>
<br>
Regards,<br>
Edward<br>
_______________________________________________<br>
Android mailing list<br>
<a href="mailto:Android@videolan.org">Android@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/android" target="_blank">https://mailman.videolan.org/listinfo/android</a><br>
</blockquote></div><br></div>