Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Alexander Kuppe2013-11-08 21:01:27 +0000
committerMarkus Alexander Kuppe2013-11-08 21:01:27 +0000
commit9029d04a12cc67535942c6ae3f8d57cb99346917 (patch)
tree878f8a52a1a07638b29266e6785c9689bc8afe4e
parent6ae84fb86cfb57f60ec90daca14a98cea64e64b2 (diff)
downloadorg.eclipse.e4.tools-9029d04a12cc67535942c6ae3f8d57cb99346917.tar.gz
org.eclipse.e4.tools-9029d04a12cc67535942c6ae3f8d57cb99346917.tar.xz
org.eclipse.e4.tools-9029d04a12cc67535942c6ae3f8d57cb99346917.zip
[Bug 421369]: Layout broken in part form I20131110-2200
https://bugs.eclipse.org/bugs/show_bug.cgi?id=421369 Change-Id: If64f334c7b60406afa9bef70b3de6dd2f3594265 Signed-off-by: Markus Alexander Kuppe <bugs.eclipse.org@lemmster.de>
-rw-r--r--bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/component/PartEditor.java11
1 files changed, 9 insertions, 2 deletions
diff --git a/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/component/PartEditor.java b/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/component/PartEditor.java
index 9c45ad14..2abc7d94 100644
--- a/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/component/PartEditor.java
+++ b/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/component/PartEditor.java
@@ -159,7 +159,7 @@ public class PartEditor extends AbstractComponentEditor {
protected Composite createForm(Composite parent, EMFDataBindingContext context, IObservableValue master, boolean isImport) {
CTabFolder folder = new CTabFolder(parent, SWT.BOTTOM);
- CTabItem item = new CTabItem(folder, SWT.NONE);
+ CTabItem item = new CTabItem(folder, SWT.BORDER);
item.setText(Messages.ModelTooling_Common_TabDefault);
parent = createScrollableContainer(folder);
@@ -234,6 +234,10 @@ public class PartEditor extends AbstractComponentEditor {
}
});
} else {
+ // Dispose the lnk widget, which is unused in this else branch
+ // and screws up the layout: see https://bugs.eclipse.org/421369
+ lnk.dispose();
+
Label l = new Label(parent, SWT.NONE);
l.setText(Messages.PartEditor_ClassURI);
l.setLayoutData(new GridData(GridData.HORIZONTAL_ALIGN_END));
@@ -245,7 +249,10 @@ public class PartEditor extends AbstractComponentEditor {
t.addModifyListener(new ModifyListener() {
public void modifyText(ModifyEvent e) {
- lnk.setToolTipText(((Text) (e.getSource())).getText());
+ // lnk might be disposed if else branch above taken
+ if (!lnk.isDisposed()) {
+ lnk.setToolTipText(((Text) (e.getSource())).getText());
+ }
}
});
Binding binding = context.bindValue(textProp.observeDelayed(200, t), EMFEditProperties.value(getEditingDomain(), ApplicationPackageImpl.Literals.CONTRIBUTION__CONTRIBUTION_URI).observeDetail(master), new UpdateValueStrategy().setAfterConvertValidator(new ContributionURIValidator()), new UpdateValueStrategy());

Back to the top