Merge "Add language switch key to symbols and symbols shifted layout"

java/res/xml-sw600dp/key_space.xml

@@ -21,6 +21,9 @@
 <merge
     xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
 >
+    <Spacer
+        latin:keyWidth="0dp"
+        latin:keyXPos="30.750%p" />
     <switch>
         <case
             latin:languageCode="fa"

java/res/xml-sw600dp/row_symbols4.xml

@@ -19,7 +19,7 @@
 -->
 
 <merge
-        xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
+    xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
 >
     <Row
         latin:keyWidth="8.9%p"
@@ -31,10 +31,8 @@
             latin:keyStyle="tabKeyStyle" />
         <Key
             latin:keyLabel="\@" />
-        <Key
+        <include
-            latin:keyStyle="spaceKeyStyle"
+            latin:keyboardLayout="@xml/key_space" />
-            latin:keyXPos="30.750%p"
-            latin:keyWidth="39.750%p" />
         <Key
             latin:keyLabel="&quot;"
             latin:moreKeys="!text/more_keys_for_tablet_double_quote" />

java/res/xml-sw600dp/row_symbols_shift4.xml

@@ -19,7 +19,7 @@
 -->
 
 <merge
-        xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
+    xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
 >
     <Row
         latin:keyWidth="8.9%p"
@@ -29,10 +29,8 @@
             latin:keyWidth="13.0%p" />
         <Key
             latin:keyStyle="tabKeyStyle" />
-        <Key
+        <include
-            latin:keyStyle="spaceKeyStyle"
+            latin:keyboardLayout="@xml/key_space" />
-            latin:keyXPos="30.750%p"
-            latin:keyWidth="39.750%p" />
         <Spacer
             latin:keyXPos="-10.00%p"
             latin:keyWidth="0%p" />

java/res/xml-sw768dp/key_space.xml

@@ -21,13 +21,15 @@
 <merge
     xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
 >
+    <Spacer
+        latin:keyWidth="0dp"
+        latin:keyXPos="31.250%p" />
     <switch>
         <case
             latin:languageCode="fa"
         >
             <Key
                 latin:keyStyle="spaceKeyStyle"
-                latin:keyXPos="31.250%p"
                 latin:keyWidth="29.453%p" />
             <!-- U+200C: "" ZERO WIDTH NON-JOINER
                  U+200D: "" ZERO WIDTH JOINER -->
@@ -37,7 +39,6 @@
         <default>
             <Key
                 latin:keyStyle="spaceKeyStyle"
-                latin:keyXPos="31.250%p"
                 latin:keyWidth="37.500%p" />
         </default>
     </switch>

java/res/xml-sw768dp/row_symbols4.xml

@@ -19,7 +19,7 @@
 -->
 
 <merge
-        xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
+    xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
 >
     <Row
         latin:keyWidth="8.047%p"
@@ -31,10 +31,8 @@
             latin:keyXPos="15.157%p" />
         <Key
             latin:keyLabel="\@" />
-        <Key
+        <include
-            latin:keyStyle="spaceKeyStyle"
+            latin:keyboardLayout="@xml/key_space" />
-            latin:keyXPos="31.250%p"
-            latin:keyWidth="37.500%p" />
         <Key
             latin:keyLabel="&quot;"
             latin:moreKeys="!text/more_keys_for_tablet_double_quote" />

java/res/xml-sw768dp/row_symbols_shift4.xml

@@ -19,17 +19,15 @@
 -->
 
 <merge
-        xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
+    xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
 >
     <Row
         latin:keyWidth="8.047%p"
     >
         <include
             latin:keyboardLayout="@xml/key_settings" />
-        <Key
+        <include
-            latin:keyStyle="spaceKeyStyle"
+            latin:keyboardLayout="@xml/key_space" />
-            latin:keyXPos="31.250%p"
-            latin:keyWidth="37.500%p" />
         <switch>
             <case
                 latin:shortcutKeyEnabled="true"

java/res/xml/key_space.xml

@@ -21,6 +21,9 @@
 <merge
     xmlns:latin="http://schemas.android.com/apk/res/com.android.inputmethod.latin"
 >
+    <Spacer
+        latin:keyWidth="0dp"
+        latin:keyXPos="25%p" />
     <switch>
         <case
             latin:languageCode="fa"

java/res/xml/row_symbols4.xml

@@ -43,9 +43,8 @@
                     latin:keyStyle="f1MoreKeysStyle" />
             </default>
         </switch>
-        <Key
+        <include
-            latin:keyStyle="spaceKeyStyle"
+            latin:keyboardLayout="@xml/key_space" />
-            latin:keyWidth="50%p" />
         <Key
             latin:keyStyle="punctuationKeyStyle" />
         <Key

java/res/xml/row_symbols_shift4.xml

@@ -35,9 +35,8 @@
         <Key
             latin:keyLabel="&#x201E;"
             latin:backgroundType="functional" />
-        <Key
+        <include
-            latin:keyStyle="spaceKeyStyle"
+            latin:keyboardLayout="@xml/key_space" />
-            latin:keyWidth="50%p" />
         <!-- U+2026: "…" HORIZONTAL ELLIPSIS -->
         <Key
             latin:keyLabel="&#x2026;"

java/src/com/android/inputmethod/keyboard/Keyboard.java

@@ -313,8 +313,11 @@ public class Keyboard {
 
         public void onAddKey(Key newKey) {
             final Key key = (mKeysCache != null) ? mKeysCache.get(newKey) : newKey;
-            mKeys.add(key);
+            final boolean zeroWidthSpacer = key.isSpacer() && key.mWidth == 0;
-            updateHistogram(key);
+            if (!zeroWidthSpacer) {
+                mKeys.add(key);
+                updateHistogram(key);
+            }
             if (key.mCode == Keyboard.CODE_SHIFT) {
                 mShiftKeys.add(key);
             }