[compot-dev]

Closes #263.

Four actions existed only as touch gestures, so a box with a D-pad remote could not reach them. With the menu closed the remote now does what the gestures do.

Keys (menu closed)

Key	Single view	Multi-view
short OK	open the menu (now on release)	open the menu (now on release)
held OK, or Menu	save a screenshot (the long press)	the selected cell's sound on / off (the long press on a cell)
held Up / Down	zoom in / out, 1×–5×; down to 1× is the reset	—
short arrow	pan the zoomed picture	move the yellow selection frame
ZOOM, CH, PAGE, + / −	zoom in / out by 1.25×	—

• The menu opens when OK is released, so that OK can be held; onKeyLongPress needs the press tracked. An OK that closed the menu, or was pressed in it, opens nothing on release (the Save report: stats.json and the log as one file, from the app and from the debug page #207 case).
• Menu is the fallback for remotes whose OK has no long press, as on Debug in the menu (Save report: stats.json and the log as one file, from the app and from the debug page #207).
• A key zoom keeps the point at the centre of the screen there (a remote has no point to zoom into).
• Multi-view: the first arrow shows the frame on the cell whose sound plays (or the first), the next ones move it; it stops at the edges and goes away after 4 s. The U cell can be selected, and a held OK on it says that it has video only.
• RemoteKeys holds the key mapping and the zoom, pan and grid arithmetic (RemoteKeysTest).
• docs/usage.md ("Gestures and keys", "Remote control", multi-view audio, screenshots), the three READMEs and the CHANGELOG describe it. Version 2.20.0 (a feature).

Checks

• Local build in Docker (compot/android), as in CI: testDebugUnitTest, lintDebug (no new findings), assembleRelease. APK +0.8 KB.
• Android TV 16 emulator (API 36, 1920×1080), D-pad input only (input keyevent), a lab camera in the single view and a 2×2 grid:
  • short OK opens the menu, Back closes it; the menu is still driven by the D-pad (slot 2 chosen, Grid turned on with it);
  • CH+ ×3 zooms in, Left/Up pan to the top left, CH− back to 1× restores the full picture;
  • a held Up zooms (one key repeat = one step); a held OK saves Pictures/OpenIPC/Screenshot_….png and opens no menu;
  • grid: the first Right shows the frame on cell 1, the next moves it to cell 2; a held OK turns its sound on (AUDIO badge, toast), Menu turns it off; Left + Menu moves the sound to cell 1.
• Android 5.0 emulator (API 21): the key handling (a held OK opens no menu, a short one opens it). Zoom on video could not be seen there: the emulator's software decoder failed on these streams ("Codec exception"); the decode path is not touched here.
• Continuous key repeats of a physical remote held for seconds are generated by Android itself; they are not reproducible with input keyevent and remain for a check on a box with a real remote.

[qodo-free-for-open-source-projects]

PR Summary by Qodo

Add D-pad access to zoom, screenshots, and multi-view audio

✨ Enhancement 🧪 Tests 📝 Documentation ⚙️ Configuration changes 🕐 20-40 Minutes

AI Description

• Enable remote zoom and pan in single view and cell selection in multi-view.
• Map held OK or Menu to existing long-press actions without reopening the menu.
• Add key-arithmetic tests, usage guidance, and the 2.20.0 release version.

Diagram

graph TD
  Input["Remote keys"] --> Gate{"Menu open?"} -->|No| Router["Decoder key routing"] --> Math["RemoteKeys helpers"] --> Video["Single view"]
  Router --> Grid["Grid selection"]
  Router --> Actions["Long actions"]
  Router -->|Short OK| Popup["Menu popup"]
  Gate -->|Yes| Popup

Loading

High-Level Assessment

The following are alternative approaches to this PR:

1. Shared input-action layer

• ➕ Could centralize gesture and remote commands as more input methods are added.
• ➖ Requires a broader refactor of established touch handling for this focused feature.

Recommendation: Keep the scoped key router and pure RemoteKeys calculations: they reuse existing screenshot, audio, and video-transform behavior without disturbing touch input. Consider a shared action layer only if further input methods make the routing diverge.

Files changed (9) +452 / -39

Enhancement (2) +290 / -3

Decoder.javaRoute remote keys to view actions and grid selection +190/-3

Route remote keys to view actions and grid selection

• Opens the menu on short OK release while tracking held OK for screenshots or cell-audio toggling; Menu provides the same long action. Adds bounded single-view zoom and pan plus a temporary selection frame for multi-view cells.

android/app/src/main/java/com/openipc/decoder/Decoder.java

RemoteKeys.javaAdd testable remote key and navigation calculations +100/-0

Add testable remote key and navigation calculations

• Defines zoom-key mappings, zoom and pan steps, centre-preserving pan calculations, and grid movement constrained to cell boundaries.

android/app/src/main/java/com/openipc/decoder/RemoteKeys.java

Tests (1) +111 / -0

RemoteKeysTest.javaTest remote mappings, zoom, pan, and grid movement +111/-0

Test remote mappings, zoom, pan, and grid movement

• Covers key directions, zoom limits and hold steps, pan arithmetic, reachable cells, and grid-edge behavior.

android/app/src/test/java/com/openipc/decoder/RemoteKeysTest.java

Documentation (5) +49 / -34

CHANGELOG.mdRecord the 2.20.0 remote-control feature +9/-0

Record the 2.20.0 remote-control feature

• Adds release notes for D-pad zoom, pan, grid selection, long-press actions, and OK-on-release menu behavior.

CHANGELOG.md

README.mdSummarize remote gestures in the English README +8/-8

Summarize remote gestures in the English README

• Expands the input table to mention held Up/Down, arrow panning, held OK, and remote cell selection.

README.md

README.ru.mdDescribe remote actions in the Russian README +8/-8

Describe remote actions in the Russian README

• Updates the translated input table with remote zoom, pan, held OK, and multi-view selection.

README.ru.md

README.zh-CN.mdDescribe remote actions in the Chinese README +8/-8

Describe remote actions in the Chinese README

• Updates the translated input table with remote zoom, pan, held OK, and multi-view selection.

README.zh-CN.md

usage.mdDocument closed-menu remote controls +16/-10

Document closed-menu remote controls

• Explains zoom, pan, grid selection, held OK and Menu actions, and OK-on-release behavior. Updates the multi-view audio and screenshot instructions.

docs/usage.md

Other (1) +2 / -2

build.gradleBump the app version to 2.20.0 +2/-2

Bump the app version to 2.20.0

• Increases versionCode from 49 to 50 and versionName from 2.19.1 to 2.20.0.

android/app/build.gradle

[qodo-free-for-open-source-projects]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0) 🎨 UX issues (0) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

1. The first arrow highlights the wrong cell ✓ Resolved

Description

onArrowDown() calls ensureQuadSelected() when the frame is hidden, but that method retains any
valid earlier selection instead of using the current audio cell. If the frame times out and a touch
long-press changes the audio to another cell, the next arrow highlights the old cell and subsequent
arrows move from it.

---

2. The remote reports sound for empty cells ✓ Resolved

Description

remoteLongAction() sends every selected non-U grid index to toggleQuadAudio(), even when that
slot has no camera cell. In a grid with an unconfigured slot, holding OK or pressing Menu on it
saves that index and shows an audio-enabled toast, but no audio route is attached.

---

3. Some grid selections remain invisible ✓ Resolved

Description

buildQuadGrid() adds the selection frame before the startup overlay, and showQuadSelection()
changes its visibility without bringing it above that overlay. Empty camera slots never receive a
first frame, and the U cell may wait indefinitely for a stream, so remote users cannot see which of
those cells is selected.

---

View review recommended (1)

4. Canceled arrow presses still pan video ✓ Resolved

Description

onKeyUp() treats an Up or Down release as a short press whenever arrowRepeated is false, without
checking event.isCanceled(). If a zoomed single view receives a canceled release before any
repeat, that release changes the picture position even though the press was canceled.

---

Tip of the day

💡 Did you know, you can commit Qodo's fix in one click with committable suggestions (GitHub & GitLab)

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[compot-dev]

Both findings fixed in 2d57ead:

1. Selection frame under a cell's hint — showQuadSelection() now brings the selected cell's frame to the front of its cell. The frame has no fill, so the hint, the AUDIO badge and the stats stay readable. Checked on the Android TV 16 emulator: in a 2×2 grid with two slots configured, Down twice from cell 1 shows the frame on the empty cell 3 (before the fix it was hidden by the cell's hint).
2. Cancelled Up / Down release panned — the release pans only for a press that went down in the single view (arrowDown), was not held (arrowRepeated) and is not cancelled (event.isCanceled()).

Local build in Docker as in CI: testDebugUnitTest, lintDebug (no new findings), assembleRelease.

[compot-dev]

/review

[qodo-free-for-open-source-projects]

Code review by qodo was updated up to the latest commit 2d57ead

[compot-dev]

Both new findings fixed in c6719a0:

1. The first arrow highlights the wrong cell — a long press on a cell (touch) now makes it the remote's selection too, so the next arrow starts from the cell touched last rather than an older one. A cell chosen with the arrows stays selected after its frame goes away, so a later held OK still acts on it; docs/usage.md says so.
2. Sound for empty cells — a held OK / Menu on a slot without a camera cell now only says Camera N is not configured; it stores no audio index. A stored index on such a slot (from before) can still be turned off.

Local build in Docker as in CI: testDebugUnitTest, lintDebug (no new findings), assembleRelease. Not rerun on the emulator: both are small changes to the two branches above.

[compot-dev]

/review

[qodo-free-for-open-source-projects]

Code review by qodo was updated up to the latest commit c6719a0