Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 283545 - [Radio Buttons] Key events also affects scrollable parent
Summary: [Radio Buttons] Key events also affects scrollable parent
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.3 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 283944
  Show dependency tree
 
Reported: 2009-07-15 08:49 EDT by Jordan Yakovchev CLA
Modified: 2009-08-12 06:07 EDT (History)
0 users

See Also:

Attachments
Proposed patch (955 bytes, patch)
2009-07-15 09:31 EDT, Jordan Yakovchev CLA 		ivan: iplog+
Details | Diff
Proposed patch (1.67 KB, patch)
2009-07-22 03:36 EDT, Jordan Yakovchev CLA 		ivan: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jordan Yakovchev CLA 2009-07-15 08:49:16 EDT 
Build ID: 20090619-0625

Steps To Reproduce:
When group of radio buttons is placed in scrollable composite, its key events appart from current functionality also makes parrent to scroll.

More information:
Comment 1 Jordan Yakovchev CLA 2009-07-15 09:31:03 EDT 
Created attachment 141641 [details]
Proposed patch
Comment 2 Ralf Sternberg CLA 2009-07-17 10:52:16 EDT 
Applied patch to CVS HEAD
Comment 3 Ralf Sternberg CLA 2009-07-19 04:03:15 EDT 
The problem is only partly fixed. Keys that are not consumed by the radio button (Home, End, PageUp, PageDown) still affect the scrolled composite. Mouse wheel events also end up in the parent. There is no such event bubbling in SWT, thus all these events need to be catched.
Comment 4 Jordan Yakovchev CLA 2009-07-22 03:36:25 EDT 
Created attachment 142228 [details]
Proposed patch

Meets the requirements of this blocker:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=283944
Comment 5 Ivan Furnadjiev CLA 2009-08-11 07:10:32 EDT 
The second patch is adopted and applyed to RadioButtonUtil.js.