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

Bug 403769

Summary: Bad focus behavior when tabbing in Opera
Product: [RT] RAP Reporter: Ilya Bochkarev <tesei7>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: tbuschto, tesei7
Version: 2.0   
Target Milestone: 2.1 RC2   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Fix for Opera 12 ivan: review+

Description Ilya Bochkarev CLA 2013-03-19 09:35:38 EDT
When press TAB button in http://rap.eclipsesource.com/rapdemo/examples#input in Opera (v 12.14) focus is not correctly changed from one widget to another.
Comment 1 Ivan Furnadjiev CLA 2013-03-20 04:22:47 EDT
Reproducible with Controld Demo -> Focus tab. Focus indicator does not change and focus events are not fired. Opera highlights some parts of the widget with a blue border - Combo, Slider, Spinner.
Comment 2 Ivan Furnadjiev CLA 2013-04-05 09:12:18 EDT
In Opera 12.15 the "keypress" DOM event is not fired for non-printable keys... in Opera 11.61 the "keypress" event is fired. This is the reason why the focusing behavior changed, but have no idea when (in which Opera version) it was introduced. We should revise the EventHandlerUtil#getEventPseudoTypes (and maybe getKeyCode and getCharCode) Opera handling:
- with the current code it's working in Opera 11.61, but not in Opera 12.15
- if we remove the "opera" handling and treat Opera as "default" - Opera 12.15 is working, but not Opera 11.61
Comment 3 Tim Buschtoens CLA 2013-05-24 06:47:31 EDT
Created attachment 231446 [details]
Fix for Opera 12

This should fix the issue on Opera 12. It doesn't respect older versions and i propose we raise the minimum version of Opera for RAP to 12.
Comment 4 Ivan Furnadjiev CLA 2013-05-24 08:21:30 EDT
Comment on attachment 231446 [details]
Fix for Opera 12

> This should fix the issue on Opera 12. It doesn't respect older versions and i propose we raise the minimum version of Opera for RAP to 12.
+1 for the patch and for raising the minimum version of Opera for RAP to 12. Ralf, what do you think?
Comment 5 Tim Buschtoens CLA 2013-05-24 11:08:43 EDT
Applied patch with commit 0cc156f6367ab24d3555e80cba513116573aec0f