Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 282878 - Missing SelectionListener on ISingleChoiceRidget
Summary: Missing SelectionListener on ISingleChoiceRidget
Status: RESOLVED FIXED
Alias: None
Product: Riena
Classification: RT
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 1.2.0.M1   Edit
Assignee: Elias Volanakis CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-07-08 11:50 EDT by Christian Campo CLA
Modified: 2009-07-09 17:10 EDT (History)
1 user (show)

See Also:

Attachments
Patch: addSelectionListener in MultipleCoice and SingleChoice ridgets (12.41 KB, patch)
2009-07-09 09:11 EDT, Nataliya Sinkevych CLA 		elias: 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 Christian Campo CLA 2009-07-08 11:50:19 EDT 
There is currently no method to add a selection listener to the ISingleChoiceRidget. The only way is to add a property change listener to the property "selection".

In sync with 282534 where a selectionListener is added for Tables etc. it is also needed for SingleChoice.
Comment 1 Elias Volanakis CLA 2009-07-08 15:13:16 EDT 
Yes, we should add it to IChoiceRidget  (that is the common parent of single + multiple choice ridget).
Comment 2 Nataliya Sinkevych CLA 2009-07-09 07:08:05 EDT 
I shall implement the required feature and post the patch here...
Comment 3 Nataliya Sinkevych CLA 2009-07-09 09:11:10 EDT 
Created attachment 141198 [details]
Patch: addSelectionListener in MultipleCoice and SingleChoice ridgets
Comment 4 Elias Volanakis CLA 2009-07-09 17:04:54 EDT 
Comment on attachment 141198 [details]
Patch: addSelectionListener in MultipleCoice and SingleChoice ridgets

Marking patch with iplog+.
Comment 5 Elias Volanakis CLA 2009-07-09 17:10:13 EDT 
Patch committed -- Thanks Nataliya!