Don't update exclude field in composite's grid data

Avatar label now correctly redraws in task editor
when assignee is added after being initially empty.

Change-Id: I44964eccbc25558336f8c4d6d2d5f45d8412211f
Signed-off-by: Kevin Sawicki <kevin@github.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

1 files changed, 2 insertions, 6 deletions

diff --git a/org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/AvatarLabel.java b/org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/AvatarLabel.java
index 924b5330..75f045b3 100644
--- a/org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/AvatarLabel.java
+++ b/org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/AvatarLabel.java
@@ -20,7 +20,6 @@ import org.eclipse.swt.events.DisposeEvent;
 import org.eclipse.swt.events.DisposeListener;
 import org.eclipse.swt.graphics.Image;
 import org.eclipse.swt.graphics.ImageData;
-import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Label;
 import org.eclipse.ui.PlatformUI;
@@ -64,11 +63,8 @@ public class AvatarLabel implements IAvatarCallback {
 	 * @return this label
 	 */
 	public AvatarLabel setVisible(boolean visible) {
-		if (!displayArea.isDisposed()) {
+		if (!displayArea.isDisposed())
 			displayArea.setVisible(visible);
-			((GridData) displayArea.getLayoutData()).exclude = !visible;
-			displayArea.getParent().getParent().layout(true, true);
-		}
 		return this;
 	}
 
@@ -79,7 +75,7 @@ public class AvatarLabel implements IAvatarCallback {
 	 */
 	public AvatarLabel layout() {
 		if (!displayArea.isDisposed())
-			displayArea.getParent().layout(true, true);
+			displayArea.getParent().getParent().layout(true, true);
 		return this;
 	}