--- ray/src/util/trad.wsh	1994/10/28 19:35:02	2.4
+++ ray/src/util/trad.wsh	1995/07/24 13:31:26	2.10
@@ -27,18 +27,18 @@ proc modified {} {	# check for radvar modification
 
 proc chksave {} {	# check if RIF is saved and accost user if not
 	global readonly rifname oldradvar curmess
-	if $readonly {
-		set warnmess "You have modified variables, but the file\
-\"[file tail $rifname]\" was opened read-only.
+	if [modified] {
+		if $readonly {
+			set warnmess "You have modified variables, but the\
+file \"[file tail $rifname]\" was opened read-only.
 Do you wish to save this information somehow?"
-		set yesmess {Go to File Screen}
-	} else {
-		set warnmess "The file \"[file tail $rifname]\" has been\
-modified since it was last saved.
+set yesmess {Go to File Screen}
+		} else {
+			set warnmess "The file \"[file tail $rifname]\" has\
+been modified since it was last saved.
 Do you wish to save it now?"
-		set yesmess {Save File}
-	}
-	if [modified] {
+			set yesmess {Save File}
+		}
 		switch [tk_dialog .dlg {File Modified} $warnmess \
 				warning 0 $yesmess {Discard Changes} \
 				{Cancel Operation}] {
@@ -46,14 +46,16 @@ Do you wish to save it now?"
 			2 { return 0 }
 		}
 		if $readonly {
-			.file invoke
 			set curmess {Rename file or uncheck read-only to save.}
+			.file invoke
 			return 0
-		} else {
-			catch {unset oldradvar}
-			save_vars $rifname
+		} elseif {[save_vars $rifname]} {
+			gotfile 1
+			return 1
 		}
-	} else { return 1 }
+		return 0
+	}
+	return 1
 }
 
 # Set global variable default values
@@ -75,17 +77,18 @@ trace variable rifname w setrname
 
 set smallscrn [expr [winfo screenwidth .] < 830]
 
+message .mess -relief ridge -font fixed -textvariable curmess -anchor nw
 if $smallscrn {
-	frame .upper -geometry 640x410
-	frame .upper.right -geometry 120x410
+	frame .upper -width 640 -height 410
+	frame .upper.right -width 120 -height 410
+	.mess configure -width 640
 } else {
-	frame .upper -geometry 830x410
-	frame .upper.right -geometry 130x410
+	frame .upper -width 830 -height 410
+	frame .upper.right -width 130 -height 410
+	.mess configure -width 830
 }
 pack .upper -side top
 place .upper.right -relx .98 -rely 0 -anchor ne
-message .mess -relief ridge -font fixed -textvariable curmess \
-		-anchor nw -width 620
 pack .mess -side top -expand yes -fill both
 helplink .mess trad trad messages
 . configure -cursor top_left_arrow
@@ -103,9 +106,9 @@ proc changescreen {} {		# switch screen according to c
 	set curscreen $curmode
 	do_$curscreen .upper.left
 	if $smallscrn {
-		.upper.left configure -geometry 520x410
+		.upper.left configure -width 520 -height 410
 	} else {
-		.upper.left configure -geometry 700x410
+		.upper.left configure -width 700 -height 410
 	}
 	place .upper.left -x 0 -y 0
 }
@@ -141,8 +144,10 @@ helplink .upper.right.quit trad trad quit
 
 if $smallscrn {
 	wm minsize . 640 460
+	wm maxsize . 640 512
 } else {
 	wm minsize . 830 460
+	wm maxsize . 830 512
 }
 wm iconbitmap . @$radlib/trad.icon
 
@@ -156,6 +161,7 @@ proc gotfile {{o 1}} {		# set file possession state
 		}
 	} else {
 		set mode disabled
+		.file invoke
 	}
 	foreach b {scene zone views options action results} {
 		.$b configure -state $mode
@@ -165,14 +171,13 @@ proc gotfile {{o 1}} {		# set file possession state
 # Decide where to go and start
 
 if {[llength $argv] == 0} {
-	gotfile 0
 	set curmess "Choose a Radiance project file."
-	.file invoke
+	gotfile 0
 } elseif {[llength $argv] == 1} {
 	cd [file dirname $argv]
 	set curfile [file tail $argv]
 	if {! [file exists $curfile]} {
-		newnew $argv
+		newnew $curfile
 		.scene invoke
 	} elseif {[newload $curfile]} {
 		if $alldone {