Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 326780 - ExtendedComboBoxCellEditor does not handle carriage return key from keypad
Summary: ExtendedComboBoxCellEditor does not handle carriage return key from keypad
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: Edit (show other bugs)
Version: 2.7.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Dave Steinberg CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-01 09:41 EDT by Mariot Chauvin CLA
Modified: 2010-12-14 23:29 EST (History)
1 user (show)

See Also:
Ed.Merks: iplog+


Attachments
patch for head (1.24 KB, patch)
2010-10-01 09:42 EDT, Mariot Chauvin CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mariot Chauvin CLA 2010-10-01 09:41:46 EDT
Steps to reproduce :

- Open the Sample Ecore Model Editor on an ecore file
- Select an EAttribute or an EReference
- Select the EType attribute in properties view
- Enter the first letters of the type you want to set, the EType combo value is filled with the first EType which matches.
- Hit the carriage return key from keypad

=> The EType combo value becomes empty. If you hit the standard carriage return key, it will set the EType value correctly.

It comes from the fact that carriage return key from keypad is not handle in ExtendedComboBoxCellEditor. See attached patch which fixes the problem.

Versions affected (at least) :
- 2.5
- 2.6
- 2.7
Comment 1 Mariot Chauvin CLA 2010-10-01 09:42:28 EDT
Created attachment 180054 [details]
patch for head
Comment 2 Ed Merks CLA 2010-11-05 06:15:34 EDT
The fix has been committed to CVS for 2.7.
Comment 3 Ed Merks CLA 2010-12-13 16:42:44 EST
The fixes are available in a published build.
Comment 4 Ed Merks CLA 2010-12-14 23:28:05 EST
Try again to resolve as fixed (and available in a build).
Comment 5 Ed Merks CLA 2010-12-14 23:29:43 EST
Oh, I guess I have to close them!