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

Bug 268602

Summary: Cell editors not accessible
Product: [Modeling] GMF-Runtime Reporter: Wayne <wdiu>
Component: GeneralAssignee: Lidija Grahek <lgrahek>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ahunter.eclipse, lgrahek
Version: unspecified   
Target Milestone: 2.2   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch to fix problems
none
Modified patch none

Description Wayne CLA 2009-03-13 15:54:43 EDT
Created attachment 128784 [details]
Patch to fix problems

Build ID: org.eclipse.gmf.runtime.common.ui.services.properties_1.2.0.v20090114-0940.jar

Steps To Reproduce:
Problems with cell editors:
1. Choose something from a cell editor and focus is lost (ExtendedComboBoxCellEditor).
2. Tab doesn't work at all (ExtendedComboBoxCellEditor).
3. Tab only works between buttons, not cell editors (MultiButtonCellEditor).

Please see attached patch which fixes the problems.

A07BZ156Q8T2R
Comment 1 Anthony Hunter CLA 2009-04-22 20:32:03 EDT
The patch does not solve the scenario in Bug 193757 .

so the basic accessibility issue still does not work.

We could probably make this a duplicate of Bug 193757
	

Comment 2 Anthony Hunter CLA 2009-07-27 13:48:10 EDT

*** This bug has been marked as a duplicate of bug 193757 ***
Comment 3 Lidija Grahek CLA 2009-09-17 12:18:09 EDT
The problem described does sound similar to that of bug 193757, but after looking into both of them more closely, it turns out they are caused by different things. So reopening this defect.
Comment 4 Lidija Grahek CLA 2009-09-17 12:25:13 EDT
Created attachment 147456 [details]
Modified patch

Attached modified patch:
- Left the change to MultiButtonCellEditor as in the original patch.
- Modified change to GMF's ExtendedComboBoxCellEditor: removed adding traversal listener to the parent - it shouldn't be done here, some parent controls already have traversal listener. Also, I don't feel comfortable about removing the selection listener. Instead, I modified its widgetSelected method so it doesn't cause accessibility problem, but is still useful for those who depend on it being there.
Comment 5 Lidija Grahek CLA 2009-11-05 13:51:20 EST
Comitted to HEAD, R2_2_maintenance and R2_1_maintenance.
Comment 6 Anthony Hunter CLA 2009-11-24 11:28:58 EST
(In reply to comment #5)
> Comitted to HEAD, R2_2_maintenance and R2_1_maintenance.

We did not want this in R2_1_maintenance, so undoing the change here.
Comment 7 Eclipse Webmaster CLA 2010-07-16 23:35:46 EDT
[target cleanup] 2.2 M7 was the original target milestone for this
bug
Comment 8 Eclipse Webmaster CLA 2010-07-19 21:55:50 EDT
[GMF Restructure] Bug 319140 : product GMF and component
Runtime Common was the original product and component for this bug