commit 9203788d3f55e84d55db4c5a87bbbf1fa1e36b43
author Brian Attwell <brianattwell@google.com> Thu Oct 16 18:24:44 2014 -0700
committer Brian Attwell <brianattwell@google.com> Thu Oct 16 18:24:44 2014 -0700
tree 0e9fbaeff958627374e5f9cfe0a2742eefa95b3f
parent 711c3440e01df8fc0a94fbc411195184d9d7cdfc

Map MenuItems to ViewPager tabs correctly in RTL

Also improved the handling of the mContactsUnavailableFragment in RTL.
The tabs still don't work perfectly in RTL for mContactsUnavailableFragment,
but this isn't a regression (and proved very tricky to solve without
changing the design). I'll file a bug for the remaining work after
checking this in.

Bug: 18011426
Change-Id: I695018464625f3f8329a33882d2d728f61c3b110

src/com/android/contacts/activities/PeopleActivity.java

diff --git a/src/com/android/contacts/activities/PeopleActivity.java b/src/com/android/contacts/activities/PeopleActivity.java
index 9f25e4e..ba77438 100644
--- a/src/com/android/contacts/activities/PeopleActivity.java
+++ b/src/com/android/contacts/activities/PeopleActivity.java
@@ -572,7 +572,7 @@
 
     private void showEmptyStateForTab(int tab) {
         if (mContactsUnavailableFragment != null) {
-            switch (tab) {
+            switch (getTabPositionForTextDirection(tab)) {
                 case TabState.FAVORITES:
                     mContactsUnavailableFragment.setMessageText(
                             R.string.listTotalAllContactsZeroStarred, -1);
@@ -1022,7 +1022,7 @@
             clearFrequentsMenu.setVisible(false);
             helpMenu.setVisible(false);
         } else {
-            switch (mActionBarAdapter.getCurrentTab()) {
+            switch (getTabPositionForTextDirection(mActionBarAdapter.getCurrentTab())) {
                 case TabState.FAVORITES:
                     contactsFilterMenu.setVisible(false);
                     clearFrequentsMenu.setVisible(hasFrequents());