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

Bug 148888

Summary: Probable incorrect state on AccessibleEditPart
Product: [Tools] GEF Reporter: Barry Feigenbaum <feigenba>
Component: GEF-Legacy GEF (MVC)Assignee: Anthony Hunter <ahunter.eclipse>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ahunter.eclipse
Version: 3.1   
Target Milestone: 3.4.1 (Ganymede SR1)   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Barry Feigenbaum CLA 2006-06-27 16:32:54 EDT 
Probable bug in GEF AbstractEditPart's AbstractAccessibleEditPart inner class - need to use |= (vs =) on FOCUS assignment:
	public void getState(AccessibleControlEvent e) {
		e.detail = ACC.STATE_SELECTABLE | ACC.STATE_FOCUSABLE;
		if (getSelected() != EditPart.SELECTED_NONE)
			e.detail |= ACC.STATE_SELECTED;
		if (getViewer().getFocusEditPart() == AbstractGraphicalEditPart.this)
			e.detail = /* should be |= */ ACC.STATE_FOCUSED;
	}
Comment 1 Anthony Hunter CLA 2006-08-11 13:15:15 EDT 
Randy, can you confirm that this is indeed a bug?
Comment 2 Randy Hudson CLA 2006-08-11 14:21:58 EDT 
It probably is.
Comment 3 Anthony Hunter CLA 2008-07-11 17:12:58 EDT 
Committed to HEAD and R3_4_maintenance