#5445 closed bug (fixed)
[patch] (ReadOnlyBootPrompt) Keymap list doesn't scroll to selection
| Reported by: | idefix | Owned by: | nobody |
|---|---|---|---|
| Priority: | normal | Milestone: | R1 |
| Component: | Applications | Version: | R1/Development |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
When ReadOnlyBootPrompt is started, the selection in the Keymap list isn't visible because the list isn't scrolled to it. The list should scroll to the selection, but it doesn't do that. I think because the list isn't added to a view at that point.
That could be a bug in the interface kit (as this TODO points out), but I attached a patch that will (temporarily) fix this issue in ReadOnlyBootPrompt.
Attachments (1)
Change History (3)
by , 14 years ago
| Attachment: | readonlybootprompt.patch added |
|---|
comment:1 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
The patch as is is wrong, since it removes the scrolling when the keymap is adjusted later on (the lists are rebuilt when you update the lanuage). But I've added a comment and applied the rest of the patch in hrev35570. Thanks!
comment:2 by , 14 years ago
Ah yes, I didn't think of the rebuilding. Thanks for the quick response!
On a related note:
When there will be a lot of languages above English, the Language list also needs a ScrollToSelection() there. I didn't add it in this patch, because the list would look a little bit weird in this case (too few languages above English)...
(patch) Scroll to selection after list is added to a view.