Attachment 74617 Details for Bug 195721 – Fix with clearSelection instead of setSelection

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

[patch] Fix with clearSelection instead of setSelection

clipboard.txt (text/plain), 1.57 KB, created by Frank Becker

on 2007-07-25 16:16:13 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Frank Becker

Created: 2007-07-25 16:16:13 EDT

Size: 1.57 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.mylyn.tasks.ui
>Index: src/org/eclipse/mylyn/tasks/ui/editors/AbstractRepositoryTaskEditor.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.mylyn/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/editors/AbstractRepositoryTaskEditor.java,v
>retrieving revision 1.204
>diff -u -r1.204 AbstractRepositoryTaskEditor.java
>--- src/org/eclipse/mylyn/tasks/ui/editors/AbstractRepositoryTaskEditor.java	25 Jul 2007 06:02:31 -0000	1.204
>+++ src/org/eclipse/mylyn/tasks/ui/editors/AbstractRepositoryTaskEditor.java	25 Jul 2007 20:16:41 -0000
>@@ -127,7 +127,6 @@
> import org.eclipse.swt.graphics.Color;
> import org.eclipse.swt.graphics.Cursor;
> import org.eclipse.swt.graphics.Font;
>-import org.eclipse.swt.graphics.Point;
> import org.eclipse.swt.layout.GridData;
> import org.eclipse.swt.layout.GridLayout;
> import org.eclipse.swt.layout.RowLayout;
>@@ -988,7 +987,7 @@
> 				if (attributeCombo.indexOf(value) != -1) {
> 					attributeCombo.select(attributeCombo.indexOf(value));
> 				}
>-				attributeCombo.setSelection(new Point(0,0));
>+				attributeCombo.clearSelection();
> 				attributeCombo.addSelectionListener(new SelectionAdapter() {
> 					@Override
> 					public void widgetSelected(SelectionEvent event) {
>@@ -996,7 +995,7 @@
> 							String sel = attributeCombo.getItem(attributeCombo.getSelectionIndex());
> 							attribute.setValue(sel);
> 							attributeChanged(attribute);
>-							attributeCombo.setSelection(new Point(0,0));
>+							attributeCombo.clearSelection();
> 						}
> 					}
> 				});

Actions: View | Diff

Attachments on bug 195721: 73228 | 74507 | 74508 | 74617 | 74618