UI: double line ExtraKeysView.

app/src/main/java/io/neoterm/view/ExtraKeysView.java

@@ -9,7 +9,6 @@ import android.view.ViewGroup;
 import android.widget.Button;
 import android.widget.HorizontalScrollView;
 import android.widget.LinearLayout;
-import android.widget.ScrollView;
 import android.widget.ToggleButton;
 
 import java.io.File;
@@ -32,7 +31,7 @@ import static io.neoterm.view.eks.ExtraButton.KEY_TAB;
  * A view showing extra keys (such as Escape, Ctrl, Alt) not normally available on an Android soft
  * keyboard.
  */
-public final class ExtraKeysView extends HorizontalScrollView {
+public final class ExtraKeysView extends LinearLayout {
 
     public static final ControlButton ESC = new ControlButton(KEY_ESC);
     public static final ControlButton TAB = new ControlButton(KEY_TAB);
@@ -55,24 +54,35 @@ public final class ExtraKeysView extends HorizontalScrollView {
     private List<ExtraButton> builtinExtraKeys;
     private List<ExtraButton> userDefinedExtraKeys;
 
-    private LinearLayout contentView;
+    private LinearLayout lineOne;
+    private LinearLayout lineTwo;
+
 
     public ExtraKeysView(Context context, AttributeSet attrs) {
         super(context, attrs);
-        setFillViewport(true);
+        setGravity(Gravity.TOP);
-        contentView = new LinearLayout(context);
+        setOrientation(VERTICAL);
-        contentView.setGravity(Gravity.CENTER);
+        HorizontalScrollView scrollOne = new HorizontalScrollView(context);
-        contentView.setLayoutParams(new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT));
+        HorizontalScrollView scrollTwo = new HorizontalScrollView(context);
-        contentView.setOrientation(LinearLayout.HORIZONTAL);
-        contentView.setGravity(Gravity.LEFT);
-        addView(contentView);
-        builtinExtraKeys = new ArrayList<>(7);
-        userDefinedExtraKeys = new ArrayList<>(7);
         loadDefaultBuiltinExtraKeys();
         loadDefaultUserDefinedExtraKeys();
+        lineOne = initLine(scrollOne);
+        lineTwo = initLine(scrollTwo);
+        addView(scrollOne);
+        addView(scrollTwo);
         updateButtons();
     }
 
+    private LinearLayout initLine(HorizontalScrollView scroll) {
+        scroll.setLayoutParams(new LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, 0, 1f));
+        scroll.setFillViewport(true);
+        LinearLayout line = new LinearLayout(getContext());
+        line.setGravity(Gravity.START);
+        line.setOrientation(LinearLayout.HORIZONTAL);
+        scroll.addView(line);
+        return line;
+    }
+
 
     public boolean readControlButton() {
         return CTRL.readState();
@@ -91,6 +101,7 @@ public final class ExtraKeysView extends HorizontalScrollView {
     }
 
     public void loadDefaultUserDefinedExtraKeys() {
+        userDefinedExtraKeys = new ArrayList<>(7);
         File defaultFile = new File(NeoTermPath.EKS_DEFAULT_FILE);
         if (!defaultFile.exists()) {
             generateDefaultFile(defaultFile);
@@ -102,7 +113,8 @@ public final class ExtraKeysView extends HorizontalScrollView {
             parser.setInput(defaultFile);
             ShortcutConfig config = parser.parse();
             userDefinedExtraKeys.addAll(config.getShortcutKeys());
-        } catch (Exception ignore) {
+        } catch (Exception e) {
+            e.printStackTrace();
         }
     }
 
@@ -111,6 +123,7 @@ public final class ExtraKeysView extends HorizontalScrollView {
     }
 
     void loadDefaultBuiltinExtraKeys() {
+        builtinExtraKeys = new ArrayList<>(7);
         builtinExtraKeys.clear();
         builtinExtraKeys.add(ESC);
         builtinExtraKeys.add(CTRL);
@@ -122,16 +135,16 @@ public final class ExtraKeysView extends HorizontalScrollView {
     }
 
     public void updateButtons() {
-        for (final ExtraButton extraButton : builtinExtraKeys) {
-            addExtraButton(extraButton);
-        }
         for (final ExtraButton extraButton : userDefinedExtraKeys) {
-            addExtraButton(extraButton);
+            addExtraButton(lineOne, extraButton);
+        }
+        for (final ExtraButton extraButton : builtinExtraKeys) {
+            addExtraButton(lineTwo, extraButton);
         }
 
     }
 
-    private void addExtraButton(final ExtraButton extraButton) {
+    private void addExtraButton(LinearLayout contentView, final ExtraButton extraButton) {
         final Button button;
         if (extraButton instanceof StatedControlButton) {
             StatedControlButton btn = ((StatedControlButton) extraButton);

app/src/main/res/layout/term.xml

@@ -17,7 +17,7 @@
         android:id="@+id/extra_keys"
         style="?android:buttonBarStyle"
         android:layout_width="match_parent"
-        android:layout_height="@dimen/eks_height_one_line"
+        android:layout_height="@dimen/eks_height_two_line"
         android:layout_alignParentBottom="true"
         android:background="@color/terminal_background"
         android:orientation="horizontal" />