SWT Shell: Auslagerung von Code verändert Verhalten

[hdi]

Hey, ich hab nach 2 min überlegen leider keinen besseren Titel gefunden...

Ich will mir ein Eclipse Plug-In programmieren, und nutze dafür SWT, da Eclipse ja darauf aufbaut. Ich bin da ein absoluter Anfänger.
Nun hab ich mir in meiner Plugin Action einfach mal einen leeren Dialog gemacht, der beim Start des Plugins angezeigt wird:

public class MyAction implements IWorkbenchWindowActionDelegate {

	private IWorkbenchWindow window;

	@Override
	public void init(IWorkbenchWindow window) {
		this.window = window;
	}

	@Override
	public void run(IAction action) {
		Shell eclipse = window.getShell();
		Shell dialog = new Shell(eclipse, SWT.APPLICATION_MODAL | SWT.DIALOG_TRIM);
		Display display = eclipse.getDisplay();
		dialog.open();
		while (!dialog.isDisposed()) {
			if (!display.readAndDispatch()) {
				display.sleep();
			}
		}
	}

	@Override
	public void dispose() {
	}
	@Override
	public void selectionChanged(IAction arg0, ISelection arg1) {
	}
}

Das funktioniert. Nun wollte ich den Dialog in eine eigene Klasse auslagern:

public class MyDialog extends Shell {

	public MyDialog(Shell owner) {
		super(owner, SWT.APPLICATION_MODAL | SWT.DIALOG_TRIM);
	}
}

und in der obigen run-Methode die Änderung:

//Shell dialog = new Shell(eclipse, SWT.APPLICATION_MODAL | SWT.DIALOG_TRIM);
Shell dialog = new MyDialog(eclipse);

Meiner Meinung nach ist das doch jetzt noch immer der selbe Code?! Auf jeden Fall funktioniert das jetzt nicht mehr. Beim Klick auf den Button der mein Plugin startet passiert einfach gar nix, es erscheint kein Dialog. Was mach ich hier falsch?

Merci

 

[hdi]

..oder mal anders gefragt: Ist das ein SWT Problem/Fehler von mir oder hat das eher was mit dem Plugin zu tun? Kennt sich jmd von euch mit Plugins für Eclipse aus? Vllt kann er nicht auf andere Klassen zugreifen oder weiss der Teufel... ?! Ich meine da konfiguriert man ja auch eine plugin.xml usw.. ich hab das jetzt aus einem Bsp übernommen, da hatten die aber nur 1 Klasse...

 

[Sonecc]

Shell darf nicht extended werden

IMPORTANT: This class is not intended to be subclassed.

steht in der Javadoc zu shell und normalerweise führt das extenden zu einem SWT-Error

 

[hdi]

Achso okay danke. Ich seh halt keine Fehlermeldungen weil ich das Plugin in einer anderen Eclipse Instanz einfach starte (Debug As -> Eclipse Application).
Aber jetzt hab ich eine andere Frage: Ich vermisse eine setLocationRelativeTo() Methode bei der Shell. Irgendwie erscheint der dialog nämlich immer woanders, scheinbar bei jedem Öffnen 10px weiter unten und weiter rechts oder so ???:L

Also wie kann ich es einstellen dass sich die Shell in der Mitte der Eclipse Umgebung öffnet? Hier nochmal der Code:

// eclipse ist die Shell die mir das Plugin-Fenster beim Start mitgibt, und dialog ist meine neue Shell die ich anzeigen will

@Override
	public void run(IAction action) {
		Shell eclipse = window.getShell();
		dialog = new Shell(eclipse, SWT.APPLICATION_MODAL | SWT.DIALOG_TRIM);
		setupDialog();
		dialog.open();
		Display display = eclipse.getDisplay();
		while (!dialog.isDisposed()) {
			if (!display.readAndDispatch()) {
				display.sleep();
			}
		}
	}

	private void setupDialog() {
		// elemente erstellen und zur shell adden
		dialog.pack();
	}

 

[Sonecc]

Edit:
setLocation bestimmt die Position der Shell

 

[hdi]

Okay gut, dann muss man das bei SWT per Hand ausrechnen scheinbar. Dachte es gäbe sowas wie relative locations. Danke

 

[Sonecc]

soweit ich weiß gibt es das nicht, sollte aber kein Problem sein, so etwas in einer Utility Klasse auszulagern.