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

Bug 346471

Summary:	LayoutComposite takes focus (tab)
Product:	[RT] RAP	Reporter:	Andreas Hoegger <Andreas.Hoegger>
Component:	RWT	Assignee:	Project Inbox <rap-inbox>
Status:	RESOLVED DUPLICATE	QA Contact:
Severity:	critical
Priority:	P3	CC:	spinel74
Version:	1.4
Target Milestone:	---
Hardware:	All
OS:	All
Whiteboard:

Description Andreas Hoegger

2011-05-19 11:16:52 EDT

By creating a Composite using FormToolkit.createComposite(..) a 'org.eclipse.ui.forms.widgets.LayoutComposite' is returned. 
Tabbing over the controls the LayoutComposite grabs the focus for some reason. To reproduce use the following example:

---
  Composite c1 = new FormToolkit(parent.getDisplay()).createComposite(parent, SWT.NONE);
  Text t = new Text(c1, SWT.BORDER);
  
  Composite c2 = new Composite(parent, SWT.NONE);
  Text t1 = new Text(c2, SWT.BORDER);

  parent.setLayout(new GridLayout(2, true));
  c1.setLayout(new FillLayout());
  c2.setLayout(new FillLayout());
---

Comment 1 Andreas Hoegger

2011-05-19 11:26:47 EDT

The behavior is reproducible with every class ? extends Composite.

Comment 2 Ivan Furnadjiev

2011-05-20 10:58:11 EDT

Andreas, could you elaborate a little bit what do you mean by "grab" focus. What do you expect and how RAP behaves? What is the difference?

Comment 3 Sebastien Pinel

2011-07-08 08:36:45 EDT

(In reply to comment #0)
> By creating a Composite using FormToolkit.createComposite(..) a
> 'org.eclipse.ui.forms.widgets.LayoutComposite' is returned. 
> Tabbing over the controls the LayoutComposite grabs the focus for some reason.
> To reproduce use the following example:
> 
> ---
>   Composite c1 = new FormToolkit(parent.getDisplay()).createComposite(parent,
> SWT.NONE);
>   Text t = new Text(c1, SWT.BORDER);
> 
>   Composite c2 = new Composite(parent, SWT.NONE);
>   Text t1 = new Text(c2, SWT.BORDER);
> 
>   parent.setLayout(new GridLayout(2, true));
>   c1.setLayout(new FillLayout());
>   c2.setLayout(new FillLayout());
> ---

I think this is linked to https://bugs.eclipse.org/bugs/show_bug.cgi?id=321274

Comment 4 Ivan Furnadjiev

2011-11-07 07:59:50 EST

(In reply to comment #3)
> I think this is linked to https://bugs.eclipse.org/bugs/show_bug.cgi?id=321274
I think that this issue is the same with bug 321274. That's why I will close it as a duplicate of it.

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