fix: menu dividers aligned with slots 6|7, labels centred; debug page Info Off by default (#174, #176)

[compot-dev]

• Menu (Menu: two items per row — [ Settings | WebUI ] and [ Exit + version | Reset ] #173 follow-up): the left item of each two-item row ([ Settings | WebUI ], [ Exit | Reset ]) ends at the gap between slots 6 and 7, measured after layout, so the dividers line up with that gap. When the slot row is wider than the menu and scrolls, the rows keep equal halves (Qodo).
• Menu labels are centred in their halves (Menu: Settings, WebUI, Exit and Reset centred in their halves #176). For Exit the word itself is centred: an invisible copy of the version text in front balances the visible one after it.
• Debug page: Info is Off until switched on. The snapshot interval default stays 5 s. Both are still remembered in the browser. README (EN/RU/ZH) says so.

Version 2.06.

Closes #174
Closes #176

[compot-dev]

/review

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

Code Review by Qodo

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

1. Dividers drift from scrolled slots 📎 Requirement gap ≡ Correctness

Description

alignPairs() sets the pair-row divider positions from slot coordinates inside camRow without
accounting for the slot row’s horizontal scroll offset. When focus scrolls the slot row on a narrow
screen, the gap between slots 6 and 7 moves across the popup while both dividers remain fixed in the
separate pair rows.

Code

android/app/src/main/java/com/openipc/decoder/MenuManager.java[R439-442]

+    private void alignPairs(LinearLayout camRow) {
+        if (camButtons == null || camButtons.length < 7 || camButtons[6] == null) return;
+        int gap = (camButtons[5].getRight() + camButtons[6].getLeft()) / 2;
+        int width = gap - dp(1) / 2;

Evidence

The slot row is horizontally scrollable and follows focus, while alignPairs() reads child
coordinates within that row and applies the resulting divider widths to separate pair rows outside
the scroll view. Its layout-change listener does not track scrolling, so the dividers do not follow
the visible gap between slots 6 and 7 when the scroll offset changes.

Align menu dividers with the gap between slots 6 and 7
android/app/src/main/java/com/openipc/decoder/MenuManager.java[188-199]
android/app/src/main/java/com/openipc/decoder/MenuManager.java[439-449]
android/app/src/main/java/com/openipc/decoder/MenuManager.java[188-207]
android/app/src/main/java/com/openipc/decoder/MenuManager.java[417-450]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The pair-row dividers use positions calculated in slot-row coordinates, but the slot row scrolls independently of the pair rows. After scrolling, the dividers no longer align with the gap between slots 6 and 7.
## Fix Focus Areas
- android/app/src/main/java/com/openipc/decoder/MenuManager.java[188-200]
- android/app/src/main/java/com/openipc/decoder/MenuManager.java[439-450]
## Recommended Fix
Calculate the visible gap position using the horizontal scroll offset and update both divider positions whenever that offset changes, or keep the slot row and pair rows in a shared coordinate system. Keep both pair rows usable when the gap moves outside the viewport.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

2. Right menu buttons are clipped on narrow screens ✓ Resolved

Description

alignPairs() replaces each left button's weighted width with a fixed width based on six slot
buttons, even though the pair rows are limited to the popup's visible width. On a narrow screen or
with enlarged text, the scrollable slot row can be wider than that popup, leaving too little space
for the single-line WebUI and Reset buttons.

Code

android/app/src/main/java/com/openipc/decoder/MenuManager.java[R447-449]

+            lp.width = width;
+            lp.weight = 0;
+            left.setLayoutParams(lp);

Evidence

The slot row sits inside a horizontal scroll view, while both pair rows are MATCH_PARENT within a
WRAP_CONTENT popup. The new fixed left width is derived from the wider slot row; the right buttons
remain weighted but are single-line, so they cannot wrap when the remaining space shrinks.

android/app/src/main/java/com/openipc/decoder/MenuManager.java[176-207]
android/app/src/main/java/com/openipc/decoder/MenuManager.java[209-213]
android/app/src/main/java/com/openipc/decoder/MenuManager.java[317-319]
android/app/src/main/java/com/openipc/decoder/MenuManager.java[417-450]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The slot-gap width can consume the visible pair row and clip its right-hand button on narrow screens or with large fonts.
## Fix Focus Areas
- android/app/src/main/java/com/openipc/decoder/MenuManager.java[439-450]
- android/app/src/main/java/com/openipc/decoder/MenuManager.java[417-430]
## Recommended Fix
Limit the aligned left width to the measured pair-row width minus the divider and enough space for the right button; retain usable right-hand controls when exact alignment cannot fit.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

Tip of the day

💡 Did you know, you can switch off images and animations for a plain-text comment

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

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

1. Dividers drift from scrolled slots 📎 Requirement gap ≡ Correctness

alignPairs() sets the pair-row divider positions from slot coordinates inside camRow without
accounting for the slot row’s horizontal scroll offset. When focus scrolls the slot row on a narrow
screen, the gap between slots 6 and 7 moves across the popup while both dividers remain fixed in the
separate pair rows.

Agent Prompt

## Issue description
The pair-row dividers use positions calculated in slot-row coordinates, but the slot row scrolls independently of the pair rows. After scrolling, the dividers no longer align with the gap between slots 6 and 7.

## Fix Focus Areas
- android/app/src/main/java/com/openipc/decoder/MenuManager.java[188-200]
- android/app/src/main/java/com/openipc/decoder/MenuManager.java[439-450]

## Recommended Fix
Calculate the visible gap position using the horizontal scroll offset and update both divider positions whenever that offset changes, or keep the slot row and pair rows in a shared coordinate system. Keep both pair rows usable when the gap moves outside the viewport.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

[compot-dev]

Qodo: both findings were about a slot row too wide for the menu, which then scrolls. Fixed: the dividers follow the gap between slots 6 and 7 only while the slot row fits the menu. A scrolling row keeps equal halves, so both items stay whole and nothing drifts.

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

Qodo: both findings were about a slot row too wide for the menu, which then scrolls. Fixed: the dividers follow the gap between slots 6 and 7 only while the slot row fits the menu. A scrolling row keeps equal halves, so both items stay whole and nothing drifts.

That addresses both findings:

• finding 1: when the slot row scrolls, the pair rows retain equal halves, so their dividers no longer claim to track an off-screen slot gap and cannot drift relative to the visible menu.
• finding 2: fixed slot-gap widths are used only when the slot row fits; otherwise both controls retain equal, usable widths and the right buttons are not clipped.

The remaining trade-off is that exact slot 6/7 alignment is intentionally unavailable while the slot row is scrollable.

[compot-dev]

Superseded by 178, which includes these commits and replaces the two-item rows with the 8 / 4 / 4 menu grid (#177).