diff options
author | Markus Alexander Kuppe | 2013-11-08 21:01:27 +0000 |
---|---|---|
committer | Markus Alexander Kuppe | 2013-11-08 21:01:27 +0000 |
commit | 9029d04a12cc67535942c6ae3f8d57cb99346917 (patch) | |
tree | 878f8a52a1a07638b29266e6785c9689bc8afe4e | |
parent | 6ae84fb86cfb57f60ec90daca14a98cea64e64b2 (diff) | |
download | org.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.java | 11 |
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()); |