| Summary: | Bad focus behavior when tabbing in Opera | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [RT] RAP | Reporter: | Ilya Bochkarev <tesei7> | ||||
| Component: | RWT | Assignee: | 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
Ilya Bochkarev
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. 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 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 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? Applied patch with commit 0cc156f6367ab24d3555e80cba513116573aec0f |