Bug 120020

Description cole harris 2005-12-08 17:27:26 EST

I recently upgraded to SWT 3.1 and it no longer supports multiple displays in Linux.  My program creates a splash screen with a progress bar that updates as the main application launches.  To fix the multiple display issue I pass the main UI display to the splash screen class to use.  The issue arises in that I must create a separate thread to run the splash screen while the main app loads.  SWT/UI calls can not be made in any thread except the main UI thread which is running.  This is an easy fix using the call ‘display.asyncExec()’, however this seems to queue its newly created thread and only runs it after the main app is finished loading.  I have been mucking around with this and scanning the internet for a few days with no luck.  Has anyone experienced any similar issues?  Thanks   

Further more the attached code works on windows and gives a multiple display error on Linux
-Cole 

import org.eclipse.swt.SWT;
import org.eclipse.swt.events.SelectionAdapter;
import org.eclipse.swt.events.SelectionEvent;
import org.eclipse.swt.graphics.Color;
import org.eclipse.swt.graphics.Image;
import org.eclipse.swt.graphics.Rectangle;
import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Dialog;
import org.eclipse.swt.widgets.Display;
import org.eclipse.swt.widgets.Label;
import org.eclipse.swt.widgets.ProgressBar;
import org.eclipse.swt.widgets.Shell;

import com.enterasys.dragon.ems.client.ui.Util;
import com.swtdesigner.ResourceManager;
import com.swtdesigner.SWTResourceManager;

public class SplashScreen extends Dialog implements Splash , Runnable{
	Thread T;
	public ProgressBar progressBar;
	public Display display;
	public SplashScreen(Shell parent) {
		super(parent);
		T = new Thread(this);
		T.start();
		
	}

	public void run() 
	{
		final Image image;
		Display display = new Display();
		final Shell dragonStartupScreenShell = new Shell(display, SWT.NO_TRIM);
		dragonStartupScreenShell.setText("Dragon Startup Screen");
		dragonStartupScreenShell.setImage(SWTResourceManager.getImage(SplashScreen.class, "/images/enterasysLogo.gif"));
		
		dragonStartupScreenShell.setSize(445, 287);
		dragonStartupScreenShell.setBackground(new Color(display,255,255,255));

//		 Center the splash screen
		Rectangle splashRect = dragonStartupScreenShell.getBounds();
		Rectangle displayRect = display.getMonitors()[0].getBounds();
		int x = (displayRect.width - splashRect.width) / 2;
		int y = (displayRect.height - splashRect.height) / 2;
		dragonStartupScreenShell.setLocation(x, y);
		
		Label label = new Label(dragonStartupScreenShell, SWT.None);
		label.setBounds(4, -6, 440, 280);
		image = new Image(display, 470, 310);
		label.setImage(ResourceManager.getImage(this.getClass(),"/images/Dragon_Splash470x265.jpg"));
		
		final Button _Button = new Button(dragonStartupScreenShell, SWT.NONE);
		_Button.addSelectionListener(new SelectionAdapter() {
			public void widgetSelected(SelectionEvent e) 
			{
				dragonStartupScreenShell.setMinimized(true);
			}
		});
		_Button.setFont(SWTResourceManager.getFont("Wide Latin", 10, SWT.BOLD));
		_Button.moveAbove(label);
		_Button.setText("-");
		_Button.setBounds(428, 1, 16, 13);

		T.setPriority(9);
		final ProgressBar progressBar = new ProgressBar(dragonStartupScreenShell, SWT.NONE);
		progressBar.setBounds(8, 270, 431, 15);
		progressBar.setMinimum(0);
		progressBar.setMaximum(100);
		progressBar.moveAbove(null);
		this.progressBar = progressBar; 
		this.display = display;
		dragonStartupScreenShell.open();
		while(!dragonStartupScreenShell.isDisposed())
		{
			if(!display.readAndDispatch())
			{
				display.sleep();
				progressBar.setSelection(PB);
				// Once PB equals 100 dispose screen bc Dragon is loaded.
				if (PB == 100)
				{
					dragonStartupScreenShell.dispose();
					image.dispose();
					label.dispose();
				}
			}
		}
	}
	
	public int PB;
	public void setProgress(int progress)
	{
		PB = progress;	
		//  display.wake is needed to retore focus control to SPlashScreen Thread once finished
		//  so it may destroy correctly.  However it is not needed nor handled in !windows
		if(Util.isWindowsOS())
		{
			display.wake();
		}
	}

}