selectBox: fix character navigation skipping last matching option

[xingsy97]

Problem

SelectBoxList.onCharacter() mimics native <select> keyboard navigation: pressing a letter key should cycle through options whose text starts with that letter. The loop iterates this.options.length - 1 times instead of this.options.length, so one candidate position is never checked. When the only remaining match is at the skipped position—typically the currently selected option after a full cycle—pressing the character key does nothing.

Fix

Change the loop bound from this.options.length - 1 to this.options.length so all positions are checked.

[Copilot]

Pull request overview

Fixes custom select box “type-to-select” behavior so character navigation checks all options (including the currently selected one), matching native <select> cycling semantics and avoiding no-op keypresses when the only match is at the previously skipped position.

Changes:

• Adjusted SelectBoxList.onCharacter() loop bound to iterate this.options.length times (instead of this.options.length - 1) so every candidate index is evaluated.