My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 15971: drop down list moves 2 positions when using the keyboard
7 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  hb...@chromium.org
Closed:  Jul 2009
Cc:  est...@chromium.org, hb...@chromium.org, evan@chromium.org, jshin@chromium.org

Restricted
  • Only users with Commit permission may comment.


Sign in to add a comment
 
Project Member Reported by f...@sofaraway.org, Jul 5, 2009
Chrome Version       : 3.0.192.0~svn20090703r19907
OS + version : Ubuntu / Karmic
window manager : Metacity
URLs (if applicable) : https://code.google.com/p/chromium/issues/list

What steps will reproduce the problem?
1. Load a page with a drop down list, like the one above
2. use the down arrow key on the keyboard

What is the expected result?

the selection should move 1 position down

What happens instead?

it moves 2 positions down

(when writing this, i also lost the cursor in all text fields.. it's still 
possible to type though)
Jul 6, 2009
#1 est...@chromium.org
are you talking about select dropdowns or autocomplete dropdowns? In either case, it 
WFM. Does it happen for you on this reduced test case:

<select name="mydropdown">
<option value="Milk">Fresh Milk</option>
<option value="Cheese">Old Cheese</option>
<option value="Bread">Hot Bread</option>
</select>


Cc: est...@chromium.org
Jul 6, 2009
#2 est...@chromium.org
 Issue 15994  has been merged into this issue.
Jul 6, 2009
#3 est...@chromium.org
ah I am seeing this now. Probably IME related.
Status: Available
Cc: hb...@chromium.org e...@chromium.org
Labels: -Area-Misc Area-WebKit Pri-2 Regression Mstone-LinuxBeta
Jul 8, 2009
#4 hb...@chromium.org
Thank you for your bug report and sorry for your inconvenience,
Yes, this is a side-effect of r19238, which integrates IME support into Chromium.
To avoid  Issue 15024 , this change sends Char events even when they don't include valid 
characters and it caused this regression.
I'm writing a fix for this issue.

Regards,
Status: Started
Owner: hb...@chromium.org
Jul 9, 2009
#5 est...@chromium.org
 Issue 16281  has been merged into this issue.
Cc: js...@chromium.org
Jul 10, 2009
#6 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=20370 

------------------------------------------------------------------------
r20370 | hbono@chromium.org | 2009-07-10 01:32:52 -0700 (Fri, 10 Jul 2009) | 9 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_gtk.cc?r1=20370&r2=20369

A quick fix for  Issue 15971 .

This is another my stupid mistake that my r19238 sends a Char event even when its text[] member does not contains any characters.
I mis-understoold I needed to send a Char event every time when gtk_keyval_to_unicode() returns 0 to avoid  Issue 15024 . But this is wrong.
(Page Down, Page Up, Arrow Keys work without Char events. Only return keys need Char events.)

BUG=15971 "drop down list moves 2 positions when using the keyboard".
TEST=Open the attached test-case
Review URL: http://codereview.chromium.org/155207
------------------------------------------------------------------------

Jul 10, 2009
#7 juhpeter...@gmail.com
What about Return?  It still misbehaves for me: eg when I do google a/c login
and use username history, pressing Return in the dropdown history menu POSTs
the selection giving a loging failure.
Is it related? Should I file another bug for that?
Jul 12, 2009
#8 mikeandm...@gmail.com
still get this bug when XMODIFIERS environment variable is "@im=SCIM".

keys like backspace, return, arrow key are still double-triggered.

Jul 12, 2009
#9 hb...@chromium.org
mikeandmore,

It seems your issue is  Issue 16281  <http://crbug.com/16281>, which is caused by a 
completely different reason (technically).

Regards,
Jul 12, 2009
#10 mikeandm...@gmail.com
umm, i see. seems this bug is IME unrelated?

Jul 12, 2009
#11 hb...@chromium.org
mikeandmore,

Technically, it is an issue of Chromium and its IME code.
We are working for fixing this issue.
Sorry for your inconvenience.

Regards,
Jul 12, 2009
#12 hb...@chromium.org
juhpetersen,

Thank you for your comments.
Technically, your issue is caused by our autofill code, which is a different area 
from WebKit. So, it is better to file another issue.
In general, it is better for you to file a new issue if you cannot find a completely 
same issue, i.e. an issue whose reproduction steps are the same as yours. We will go 
through its reproduction steps, investigate them, and merge them if it is a 
duplicate.

As for your return issue, it is easy to fix this issue (just changing one line.)

Regards,
Jul 13, 2009
#13 hb...@chromium.org
juhpetersen,

Filed a new  Issue 16597  <http://crbug.com/16597> for your return-key issue.

Regards,
Jul 29, 2009
#14 evan@chromium.org
Looks like the relevant sub-bugs have been split out.
Status: Fixed
Mar 18, 2011
#15 lafo...@chromium.org
Chrome Version       : 3.0.192.0~svn20090703r19907
OS + version : Ubuntu / Karmic
window manager : Metacity
URLs (if applicable) : https://code.google.com/p/chromium/issues/list

<b>What steps will reproduce the problem?</b>
1. Load a page with a drop down list, like the one above
2. use the down arrow key on the keyboard

<b>What is the expected result?</b>

the selection should move 1 position down

<b>What happens instead?</b>

it moves 2 positions down

(when writing this, i also lost the cursor in all text fields.. it's still 
possible to type though)
Labels: -Regression bulkmove Type-Regression
Oct 12, 2012
#16 bugdro...@chromium.org
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Labels: Restrict-AddIssueComment-Commit
Mar 9, 2013
#17 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-WebKit -Type-Regression Cr-Content Type-Bug-Regression
Apr 5, 2013
#18 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Cr-Content Cr-Blink
Sign in to add a comment

Powered by Google Project Hosting