[Webkit-unassigned] [Bug 52262] More keybindings for input[type=range]

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=52262


Kent Tamura <tkent at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #99809|review?                     |review-
               Flag|                            |




--- Comment #7 from Kent Tamura <tkent at chromium.org>  2011-07-06 08:07:06 PST ---
(From update of attachment 99809)
View in context: https://bugs.webkit.org/attachment.cgi?id=99809&action=review

r- because the key operations are not compatible with Windows.

> Source/WebCore/html/RangeInputType.cpp:202
> +    if (key == "Up" || key == "Right")
> +        newValue = current + step;
> +    else if (key == "Down" || key == "Left")
> +        newValue = current - step;
> +    else if (key == "PageDown")
> +        newValue = current + bigStep;
> +    else if (key == "PageUp")
> +        newValue = current - bigStep;
> +    else if (key == "Home")
> +        newValue = minimum();
> +    else if (key == "End")
> +        newValue = maximum();
> +    else
> +        return; // Did not match any key binding.

A horizontal slider in Control panel -> Keyboard in Windows 7 behaves:
 Up: Increase the value
 Down: Decrease the value
 PageUp: Increase the value by a large amount
 PageDown: Decrease the value by a large amount
 Home: Jump to the minimum value
 End: Jump to the maximum value

A vertical slider in Volume mixer in Windows 7 behaves:
 Right: decrease the value
 Left: Increase the value
 PageUp: Increase the value by a large amount
 PageDown: Decrease the value by a large amount
 Home: Jump to the maximum value
 End: Jump to the minimum value

You can detect slider orientation by renderer()->style()->appearance().

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.