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

Bug 323605

Summary: ToggleButtonRidget.setOutputOnly() does not work properly
Product: [RT] Riena Reporter: Steffen Kriese <steffen.kriese>
Component: ridgetAssignee: Project Inbox <riena.core-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: nobody
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows All   
Whiteboard:
Attachments:
Description Flags
Screenshot showing the result of the 2 snippets
none
Patch for the SWT-ExampleClient to reproduce the bug
none
Possible Fix none

Description Steffen Kriese CLA 2010-08-25 08:38:26 EDT 
The visual appearance of the ToggleButtons outputmarker depends on the order in which setOutputOnly() and setSelected() is called.

The following two snippets lead to different result:

Case 1:
choiceFirst.setSelected(true);
choiceFirst.setOutputOnly(true);


Case 2:
choiceLast.setOutputOnly(true);
choiceLast.setSelected(true);
Comment 1 Steffen Kriese CLA 2010-08-25 08:39:50 EDT 
Created attachment 177411 [details]
Screenshot showing the result of the 2 snippets
Comment 2 Steffen Kriese CLA 2010-08-25 08:41:48 EDT 
Created attachment 177412 [details]
Patch for the SWT-ExampleClient to reproduce the bug
Comment 3 Elias Volanakis CLA 2010-08-25 19:23:49 EDT 
Reproducible, thanks!

Interesting side note: it works if the setters are not called from inside the listener, but from configureRigets()   (that's why I didn't catch it earlier).
Comment 4 Elias Volanakis CLA 2010-08-26 01:49:41 EDT 
Created attachment 177491 [details]
Possible Fix

Here is a patch that fixes this.

I'm investigating one more alternative, so I haven't committed this yet.
Comment 5 Elias Volanakis CLA 2011-08-03 14:35:12 EDT 
Ressigning to default assignee. Not working on Riena right now.
Comment 6 Elias Volanakis CLA 2011-08-03 14:35:16 EDT 
Reassigning to default assignee. Not working on Riena right now.
Comment 7 Nobody - feel free to take it CLA 2015-06-30 10:09:41 EDT 
Fixed for single radio buttons, check boxes, and ChoiceComposite.