Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 24043 - JFace: ComboBoxCellEditor should have option of Read_ONLY
Summary: JFace: ComboBoxCellEditor should have option of Read_ONLY
Status: RESOLVED DUPLICATE of bug 12329
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows 2000
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-09-24 12:10 EDT by Richard Kulp CLA
Modified: 2002-09-25 09:35 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Richard Kulp CLA 2002-09-24 12:10:14 EDT
Currently the ComboBoxCellEditor has the text area be read-write. By default it 
should be read-only. This is because there is currently no way to retrieve the 
value of that text area, but users think they can type into that area, and it 
confuses them.

My suggestion is that it be, by default, READ_ONLY, but with the option of 
being READ_WRITE. So as not to break current users, I suggest an additional 
constructor which takes a boolean (or a style bit) where it can be indicated to 
be either read/write or read/only. Then current users would use the current 
constructor which would make it read/only. (Note: CCombo control already 
handles read/write and read/only correctly).

When it is read/write, the getValue() should then return the Integer if the 
text value equals the selection, or it should return String (text value) 
otherwise. If it is read/only, then return as now.

This way it would be identical for current users (which really expect it to be 
read/only since the text itself is useless), but any new users who want 
read/write capability can then handle the text if it is not one of the 
selections.

Thanks,
Rich Kulp
Comment 1 Sonia Dimitrov CLA 2002-09-25 09:35:34 EDT

*** This bug has been marked as a duplicate of 12329 ***