Talon/RemSound
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Connectivity tab order: Add-by-IP after the lists (per Ed's revised order)

Browse Source 
Order is now connected, details, rename, discovered, remembered, add-by-IP, lock, status — in both
SetTabOrder (authoritative) and the visual layout. Held for next release.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Ednunp
2026-07-08 11:57:03 +01:00
co-authored by Claude Opus 4.8
parent 5e974f7064
commit 9d54d4c9e9
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/RemSound.App/MainForm.cs
+9 -9
View File
@@ -3038,12 +3038,12 @@ public sealed class MainForm : Form
panel.Controls.Add(renamePeerButton, 1, 3);
connectedPeersList.SelectedIndexChanged += (_, _) => UpdatePeerDetails();
// Add a peer by address — grouped with the connected-peer actions, before the discovery lists.
panel.Controls.Add(new Label { Text = "Manual peer", AutoSize = true, Anchor = AnchorStyles.Left }, 0, 4);
panel.Controls.Add(manualAddButton, 1, 4);
FormLayoutRows.AddCheckedListRow(panel, 4, "Discovered peers (Alt+&D)", discoveredPeersList, discoveredPeersStatus, FocusListControl);
FormLayoutRows.AddCheckedListRow(panel, 5, "Remembered peers (Alt+&R)", rememberedPeersList, rememberedPeersStatus, FocusListControl);
FormLayoutRows.AddCheckedListRow(panel, 5, "Discovered peers (Alt+&D)", discoveredPeersList, discoveredPeersStatus, FocusListControl);
FormLayoutRows.AddCheckedListRow(panel, 6, "Remembered peers (Alt+&R)", rememberedPeersList, rememberedPeersStatus, FocusListControl);
// Add a peer by address, then the lock toggle — the manual / advanced options after the lists.
panel.Controls.Add(new Label { Text = "Manual peer", AutoSize = true, Anchor = AnchorStyles.Left }, 0, 6);
panel.Controls.Add(manualAddButton, 1, 6);
lockPeerAddressesBox.Checked = settings.LoadLockPeerAddresses();
lockPeerAddressesBox.CheckedChanged += (_, _) =>
@@ -4684,14 +4684,14 @@ public sealed class MainForm : Form
//
// Connectivity tab. This is the authoritative tab order (it overrides layout add-order), so it
// must list EVERY focusable control on the tab or the unlisted ones collapse to TabIndex 0 and
// land in the wrong place. Order: connected, details, rename, add-by-IP, discovered, remembered,
// land in the wrong place. Order: connected, details, rename, discovered, remembered, add-by-IP,
// lock, status.
connectedPeersList.TabIndex = 0;
peerDetailsBox.TabIndex = 1;
renamePeerButton.TabIndex = 2;
manualAddButton.TabIndex = 3;
discoveredPeersList.TabIndex = 4;
rememberedPeersList.TabIndex = 5;
discoveredPeersList.TabIndex = 3;
rememberedPeersList.TabIndex = 4;
manualAddButton.TabIndex = 5;
lockPeerAddressesBox.TabIndex = 6;
statusReadout.TabIndex = 7;
// Audio I/O tab. The driver picker is row 0 when present (a real driver chosen here