<div dir="ltr"><div><div> - menu_controls : I did not really intend, we can reduce back to the ones shown as actions yes...<br><br></div>About the focus blocked, it mostly happens when I leave VLC by pressing home, then launch it back. In this case, with keys navigation I'm trapped in actions items.<br><br></div>And yes, you can work on master now, we pushed your patch to move forward even if it's not yet 100% okay ;)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 15, 2014 at 10:25 PM, Mik Amchislavsky <span dir="ltr"><<a href="mailto:hailmikhail@gmail.com" target="_blank">hailmikhail@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>Hi Geoffrey,<span class=""><br><br>>>> I pushed with some modifications:<br><br></span></div><div>OK. Edward wanted me to make some changes. Unless you tell me otherwise, I'll patch against bcce553c4a68d9c947ac144615c61039be1960a7.<br></div><span class=""><div><br>>>> - generateViewId() is an API 17 method, I backported it into Util.java to have it.<br><br></div></span><div>Great. Thank you.<br></div><div><br>I would have preferred to assign the ID through resource files but, don't see a way to do this because the framework creates home dynamically. Do I understand home creation correctly or am I missing something ?<br></div><div></div><div><br>>>> - menu_controls... get it more dynamic, and avoid hard-coded ids.<br><br></div>I see the change to setMenuFocusDown. The hard coded list only contained menu items with "vlc:showAsAction" attribute set to "ifRoom". Did you intend to update all the menu items, including those in the pop-up, menu made visible by the "Settings" key ? <br></div><div><br>>>> ... focus is not correctly set ... get blocked ... I hope you will find out why.<br><br></div><div>OK. This weekend, I'll play with bcce553c4a68d9c947ac144615c61039be1960a7.<br><br>I posted your 2 concerns to 11927. Please update this thread, as needed.<br><a href="https://trac.videolan.org/vlc/ticket/11927" target="_blank">https://trac.videolan.org/vlc/ticket/11927</a><br><br>What do you mean by "And generally, it still happens to get blocked when we navigate by keys." ?<br></div><div>To help me understand this issue, would you please identify the starting point, the sequence of input events, and the resulting state you consider to be faulty ?<br></div><div><br></div><div>Rather than allow free reign, I want to limit focus change flexibility to simplify implementation and the user experience. As an example, once the focus reaches the sliding menu I do not want the user to leave the menu without selecting a menu item. This restriction allows a resource file driven implementation and reduces likely outcomes to simplify testing and error handling with minimal impact to the user experience.<br><br></div></div></div>Regards,<br><br>Mik<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 15, 2014 at 7:29 AM, Geoffrey Métais <span dir="ltr"><<a href="mailto:geoffrey.metais@gmail.com" target="_blank">geoffrey.metais@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>I pushed with some modifications:<br><br>- generateViewId() is an API 17 method, I backported it into Util.java to have it.<br></div>- there was a crash when audiolist was empty.<br></div>- i change the way menu_controls ids table is filled, to get it more dynamic, and avoid hard-coded ids.<br><br></div>There is still a bug, when I re-launch VLC when I left it on audiobrowser fragment, focus is not correctly set.<br></div>And generally, it still happens to get blocked when we navigate by keys.<br><br></div>I hope you will find out why.<br><div><div><div><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 14, 2014 at 2:58 PM, Geoffrey Métais <span dir="ltr"><<a href="mailto:geoffrey.metais@gmail.com" target="_blank">geoffrey.metais@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>No sorry, I meant reformatting according to Edward's comments.<br></div>I understood you re-submit them according to his remarks.<br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 14, 2014 at 2:55 PM, Mik Amchislavsky <span dir="ltr"><<a href="mailto:hailmikhail@gmail.com" target="_blank">hailmikhail@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Geoffrey,<span><br><br>>>> I wait to have your patches reformatted to test and merge it :)<br><br></span></div>Are you still having problems with the format ?<br><br>I am able to apply the copy sent on Oct 12th of "[Android] [PATCH] AudioBrowserFragment: focus related functional improvement" to master commit 4810f7d4535fc1b82520417c76920f5ef07db4b0.<br><br></div><div>To apply I use the following command.<br><br></div><div>git apply ./patches/my_patch_file<br></div><div><br></div><div>What error does git give you when you try to apply this patch ? other patches ?<br></div><div><br>Regards,<br><br></div>Mik<br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>