diff options
author | Jeremy Klein <jlklein@google.com> | 2015-08-10 15:49:14 -0700 |
---|---|---|
committer | Jeremy Klein <jlklein@google.com> | 2015-08-10 15:50:23 -0700 |
commit | 84e2759b1ec87fac59581c0c33fe0cf6e293be74 (patch) | |
tree | 6d7b68c1bce0d145f1d7fce7180c8eb878270be8 /app/src/main/java/org/connectbot/ConsoleActivity.java | |
parent | ef3b005fdb3db2f42632bc304bef6b424b748d6d (diff) | |
download | connectbot-84e2759b1ec87fac59581c0c33fe0cf6e293be74.tar.gz connectbot-84e2759b1ec87fac59581c0c33fe0cf6e293be74.tar.bz2 connectbot-84e2759b1ec87fac59581c0c33fe0cf6e293be74.zip |
Ensure that console title overlays are shown when switching views.
This bug was due to an error is findCurrentView() since it was
looking in the TerminalView rather than its wrapper.
Diffstat (limited to 'app/src/main/java/org/connectbot/ConsoleActivity.java')
-rw-r--r-- | app/src/main/java/org/connectbot/ConsoleActivity.java | 28 |
1 files changed, 12 insertions, 16 deletions
diff --git a/app/src/main/java/org/connectbot/ConsoleActivity.java b/app/src/main/java/org/connectbot/ConsoleActivity.java index 351ec27..ff42f89 100644 --- a/app/src/main/java/org/connectbot/ConsoleActivity.java +++ b/app/src/main/java/org/connectbot/ConsoleActivity.java @@ -222,7 +222,7 @@ public class ConsoleActivity extends Activity { } protected View findCurrentView(int id) { - TerminalView view = adapter.getCurrentTerminalView(); + View view = pager.findViewWithTag(adapter.getBridgeAtPosition(pager.getCurrentItem())); if (view == null) { return null; } @@ -425,9 +425,8 @@ public class ConsoleActivity extends Activity { final Button upButton = (Button) findViewById(R.id.button_up); upButton.setOnClickListener(new OnClickListener() { public void onClick(View view) { - View flip = findCurrentView(R.id.console_flip); - if (flip == null) return; - TerminalView terminal = (TerminalView) flip; + TerminalView terminal = adapter.getCurrentTerminalView(); + if (terminal == null) return; TerminalKeyListener handler = terminal.bridge.getKeyHandler(); @@ -437,9 +436,8 @@ public class ConsoleActivity extends Activity { final Button dnButton = (Button) findViewById(R.id.button_down); dnButton.setOnClickListener(new OnClickListener() { public void onClick(View view) { - View flip = findCurrentView(R.id.console_flip); - if (flip == null) return; - TerminalView terminal = (TerminalView) flip; + TerminalView terminal = adapter.getCurrentTerminalView(); + if (terminal == null) return; TerminalKeyListener handler = terminal.bridge.getKeyHandler(); handler.sendPressedKey(vt320.KEY_DOWN); @@ -448,9 +446,8 @@ public class ConsoleActivity extends Activity { final Button leftButton = (Button) findViewById(R.id.button_left); leftButton.setOnClickListener(new OnClickListener() { public void onClick(View view) { - View flip = findCurrentView(R.id.console_flip); - if (flip == null) return; - TerminalView terminal = (TerminalView) flip; + TerminalView terminal = adapter.getCurrentTerminalView(); + if (terminal == null) return; TerminalKeyListener handler = terminal.bridge.getKeyHandler(); handler.sendPressedKey(vt320.KEY_LEFT); @@ -459,9 +456,8 @@ public class ConsoleActivity extends Activity { final Button rightButton = (Button) findViewById(R.id.button_right); rightButton.setOnClickListener(new OnClickListener() { public void onClick(View view) { - View flip = findCurrentView(R.id.console_flip); - if (flip == null) return; - TerminalView terminal = (TerminalView) flip; + TerminalView terminal = adapter.getCurrentTerminalView(); + if (terminal == null) return; TerminalKeyListener handler = terminal.bridge.getKeyHandler(); handler.sendPressedKey(vt320.KEY_RIGHT); @@ -1228,7 +1224,7 @@ public class ConsoleActivity extends Activity { } } - public TerminalBridge getItemAtPosition(int position) { + public TerminalBridge getBridgeAtPosition(int position) { ArrayList<TerminalBridge> bridges = bound.getBridges(); if (position < 0 || position >= bridges.size()) { return null; @@ -1243,7 +1239,7 @@ public class ConsoleActivity extends Activity { @Override public CharSequence getPageTitle(int position) { - TerminalBridge bridge = getItemAtPosition(position); + TerminalBridge bridge = getBridgeAtPosition(position); if (bridge == null) { return "???"; } @@ -1251,7 +1247,7 @@ public class ConsoleActivity extends Activity { } public TerminalView getCurrentTerminalView() { - View currentView = pager.findViewWithTag(adapter.getItemAtPosition(pager.getCurrentItem())); + View currentView = pager.findViewWithTag(getBridgeAtPosition(pager.getCurrentItem())); if (currentView == null) return null; return (TerminalView) currentView.findViewById(R.id.console_flip); } |