Attachment 3614 Details for Bug 30646 – Removed the hardcoded initial size for the shell

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Removed the hardcoded initial size for the shell

30646.patch (text/plain), 1.61 KB, created by Boris Shingarov

on 2003-02-20 19:03:15 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Boris Shingarov

Created: 2003-02-20 19:03:15 EST

Size: 1.61 KB

patch

obsolete

>Index: TagConfigurationDialog.java
>===================================================================
>RCS file: /home/eclipse/org.eclipse.team.cvs.ui/src/org/eclipse/team/internal/ccvs/ui/TagConfigurationDialog.java,v
>retrieving revision 1.21
>diff -u -r1.21 TagConfigurationDialog.java
>--- TagConfigurationDialog.java	20 Feb 2003 23:18:48 -0000	1.21
>+++ TagConfigurationDialog.java	21 Feb 2003 00:02:17 -0000
>@@ -96,12 +96,6 @@
> 	// enable selecting auto-refresh files
> 	private boolean allowSettingAutoRefreshFiles = true;
> 	
>-	// sizing default hints
>-	private final int ALLOWREFRESH_WIDTH = 500;
>-	private final int ALLOWREFRESH_HEIGHT = 625;
>-	private final int NOREFRESH_WIDTH = 500;
>-	private final int NOREFRESH_HEIGHT = 550;
>-	
> 	// preference keys
> 	private final String ALLOWREFRESH_WIDTH_KEY = "AllowRefreshWidth"; //$NON-NLS-1$
> 	private final String ALLOWREFRESH_HEIGHT_KEY = "AllowRefreshHeight"; //$NON-NLS-1$
>@@ -746,16 +740,18 @@
> 				height = settings.getInt(ALLOWREFRESH_HEIGHT_KEY);
> 				width = settings.getInt(ALLOWREFRESH_WIDTH_KEY);
> 			} catch(NumberFormatException e) {
>-				height = ALLOWREFRESH_HEIGHT;
>-				width = ALLOWREFRESH_WIDTH;
>+				Point is = super.getInitialSize();
>+				height = is.x;
>+				width = is.y;
> 			}
> 		} else {
> 			try {
> 				height = settings.getInt(NOREFRESH_HEIGHT_KEY);
> 				width = settings.getInt(NOREFRESH_WIDTH_KEY);
> 			} catch(NumberFormatException e) {
>-				height = NOREFRESH_HEIGHT;
>-				width = NOREFRESH_WIDTH;
>+				Point is = super.getInitialSize();
>+				height = is.x;
>+				width = is.y;
> 			}
> 		}
> 		return new Point(width, height);

Actions: View | Diff

Attachments on bug 30646: 3614 | 3999