Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 288348

Summary: The focus stays on the browser location bar after clicking some of the widgets.
Product: [RT] RAP Reporter: Jordan Yakovchev <dany>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 1.2   
Target Milestone: 1.3 M2   
Hardware: All   
OS: All   
Whiteboard:

Description Jordan Yakovchev CLA 2009-09-02 08:22:32 EDT
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2
Build Identifier: 20090619-0625

It appears just in Firefox.

Reproducible: Always

Steps to Reproduce:
1. Open the controls demo in the list tab
2. Click on the browsers location bar to focus it
3. Click on some of the list items.
4. The location bar remains focused and receives keyboard events.



This is observed also in TabFolder (click on tabs), CTabFolder (click on tabs), ToolBar (buttons), CoolBar (buttons), Combo (click on the drop-down button), Table (click on a table item), DateTime-Calendar, Slider (click on buttons)
Comment 1 Jordan Yakovchev CLA 2009-09-03 03:16:39 EDT
Corresponding Qooxdoo bug is opened:
http://bugzilla.qooxdoo.org/show_bug.cgi?id=2747

In Qooxdoo demo browser the bug could be reproduced with Toolbar and Combo only.
Comment 2 Jordan Yakovchev CLA 2009-09-03 05:13:07 EDT
The List widget also suffer from this issue (missed in the description).
Comment 3 Jordan Yakovchev CLA 2009-09-04 09:40:37 EDT
We found that the problem is caused by setting the style property MozUserFocus
to "ignore" in the Qooxdoo Widget.js #_applyTabIndex.
We have made extensive testing and the conclusion is that setting only the html propery tabIndex is enough.

We have proposed a patch to Qooxdoo and waiting for their comments.

This fixes all affected widgets, except the Table.
Comment 4 Jordan Yakovchev CLA 2009-09-08 05:31:35 EDT
Separate bug is opened for the Table, because the problem there is different: https://bugs.eclipse.org/bugs/show_bug.cgi?id=288807
Comment 5 Ivan Furnadjiev CLA 2009-09-08 06:10:28 EDT
Fixed by appling the proposed patch to qooxdoo. Changes are in CVS HEAD.