Export to GitHub

grub4dos-chenall - issue #29

cursor key > 30 times


Posted on Jun 16, 2011 by Happy Kangaroo

What steps will reproduce the problem? 1. Create menu with 32 menu items 2. press KEY_DOWN key 30 times 3. 5 second countdown starts

What is the expected output? What do you see instead? countdown should not start

Also happens with v,^,left,right,up,down.

What version of the product are you using? On what operating system? latest/all builds

Please provide any additional information below.

Suggest code in stage2.c // if (!old_c_count_end && c == old_c && (old_c_count >= 30 || (old_c_count >= 8 && c != KEY_DOWN /&& c != KEY_RIGHT/ && c != KEY_UP /&& c != KEY_LEFT/))) if (!old_c_count_end && c == old_c && old_c_count >= 8 && c != KEY_LEFT && c != KEY_RIGHT && c != KEY_DOWN && c != KEY_UP && ((char)c) != '^' && ((char)c) != 'v') grub_timeout = 5;

in addition would be nice to have a debug setting to turn this off (or indeed turn all old dumb_terminal/vt52 features off using a grub4dos command - e.g. terminal disable )

Comment #1

Posted on Jun 16, 2011 by Happy Cat

Comment deleted

Comment #2

Posted on Jun 16, 2011 by Happy Cat

IIRC from what tinybit said, it is a "Feature" but not a "bug".

Comment #3

Posted on Jun 16, 2011 by Happy Kangaroo

what is the purpose of this undocumented feature? Why does it not work for v or key_right or ^ ???

Comment #4

Posted on Jun 17, 2011 by Massive Kangaroo

countdown should not start, Also happens with v,^,left,right,up,down. Suggest code in stage2.c ......

With your patch, you disabled the feature of the keys v,^,left,right,up,down for Single-Key-Selection. It may be an improvement. However it may also be a deterioration.

what is the purpose of this undocumented feature?

In case the user only has the UP and DOWN keys(working), this feature will give the user a possibility to select and boot a menu item.

And not all features have to be documented(in my opinion), though it might be better to document it.

Why does it not work for v or key_right or ^ ???

Do you mean these keys should not be used as Single-Key-Selection?

Are you sure the user never want to use these keys as a Single Key for selection and boot?

Any key with 8 times repeat will act as Single-Key-Selection, including the keys "v", key_right and "^".

a debug setting to turn this off

I personally think this feature does not bring too much trouble and need not turn off. This is mostly a choice of taste. Let's see chenall's decision.

Status: WontFix

Labels:
Type-Defect Priority-Medium