<div dir="ltr"><div>Hi Edward,<br><br>Thanks for writing; your request encouraged me to review the patch further and find room for improvement. Previously, I incorrectly assumed android.R.id.list views lacked a resource definitions to house nextFocus attributes. Now, I see these views defined as layouts.<br><br>>>>  _one_ commit per change<br><br>At a minimum, I'll move nextFocus attribute initalization into a separate patch that affects resource files instead of source.<br></div><div><br>I could have made the subject clearer. Aside from nextFocus attribute initialization, the code dynamically updates nextFocus attributes to match fragment selection.<br><br></div><div>Should I split up this patch into multiple chunks with one per fragment ?<br><br>Do you prefer resource and source changes together ?<br><br></div><div>Once I hear back from you, I'll resubmit this patch. Would you let me know if you require the same done for any of the other patches, I sent this weekend ?<br><br>>>> use more descriptive function names. (e.g. what does "focusHelper"<br>
do?)<br><br><div>In this case, focusHelper serves as a wrapper for
 setMenuFocusDown and setSearchAsFocusDown. For me, this wrapper makes 
the code easier to read. Which of the following would you consider an 
improvement or would you recommend another name ?<br><br></div>setNextFocuses<br>setNextFocusForPersistentViews<br><br></div><div>Regards,<br><br>Mik<br></div></div>