IrTcl now works with both tk4.0b4/tcl7.4b4 and tk3.6/tcl7.3
[ir-tcl-moved-to-github.git] / client.tcl
index 6ce4644..a99e741 100644 (file)
@@ -4,7 +4,24 @@
 # Sebastian Hammer, Adam Dickmeiss
 #
 # $Log: client.tcl,v $
-# Revision 1.53  1995-06-26 12:40:09  adam
+# Revision 1.58  1995-06-29 12:34:06  adam
+# IrTcl now works with both tk4.0b4/tcl7.4b4 and tk3.6/tcl7.3
+#
+# Revision 1.57  1995/06/29  09:20:30  adam
+# Target entries in cascade menus are sorted.
+#
+# Revision 1.56  1995/06/27  19:03:48  adam
+# Bug fix in do_present in ir-tcl.c: p->set_child member weren't set.
+# nextResultSetPosition used instead of setOffset.
+#
+# Revision 1.55  1995/06/27  17:10:37  adam
+# Bug fix: install procedure didn't work on some systems.
+# Error turned up when clientrc.tcl was't present.
+#
+# Revision 1.54  1995/06/27  14:41:03  adam
+# Bug fix in search-response. Didn't always observe non-surrogate diagnostics.
+#
+# Revision 1.53  1995/06/26  12:40:09  adam
 # Client defines its own tkerror.
 # User may specify 'no preferredRecordSyntax'.
 #
 #
 #
 
+set tk4 0
+if {$tk4} {
+    proc configure-enable-e {w n} {
+        incr n
+        $w entryconfigure $n -state normal
+    }
+    proc configure-disable-e {w n} {
+        incr n
+        $w entryconfigure $n -state disabled
+    }
+    set noFocus [list -takefocus 0]
+} else {
+    proc configure-enable-e {w n} {
+        $w enable $n
+    }
+    proc configure-disable-e {w n} {
+        $w disable $n
+    }
+    set noFocus {}
+}
+
+if {! $tk4} {
+    if {[tk colormodel .] == "color"} {
+        set monoFlag 0
+    } else {
+        set monoFlag 1
+    }
+} else {
+    set monoFlag 0
+}
+
 set libdir LIBDIR
-if {[file readable clientrc.tcl]} {
+if {[file readable bitmaps/book2]} {
        set libdir .
 }
+if {! [file readable ${libdir}/bitmaps/book2]} {
+    puts "Cannot locate system files in ${libdir}. You must either run this"
+    puts "program from the source directory root of ir-tcl or you must assure"
+    puts "that it is installed - normally in /usr/local/lib/irtcl"
+    exit 1
+}
+
 set hotTargets {}
 set hotInfo {}
 set busy 0
 
-set profile(Default) {{} {} {210} {} 16384 8192 tcpip {} 1 {} {} Z39}
+set profile(Default) {{} {} {210} {} 16384 8192 tcpip {} 1 {} {} Z39 1}
 set hostid Default
 set settingsChanged 0
 set setNo 0
-set lastSetNo 0
+set setNoLast 0
 set cancelFlag 0
 set scanEnable 0
 set fullMarcSeq 0
@@ -220,6 +275,7 @@ wm minsize . 0 0
 set setOffset 0
 set setMax 0
 
+if {0} {
 proc tkerror err {
     set w .tkerrorw
 
@@ -237,7 +293,8 @@ proc tkerror err {
             -font -Adobe-Helvetica-Bold-R-Normal-*-240-*
     pack $w.top.b $w.top.t -side left -padx 10 -pady 10
 
-    bottom-buttons $w [list {Close} [list destroy $w]] 0
+    bottom-buttons $w [list {Close} [list destroy $w]] 1
+}
 }
 
 proc read-formats {} {
@@ -263,7 +320,7 @@ proc set-wrap {m} {
 }
 
 proc dputs {m} {
-#    puts $m
+#   puts $m
 }
 
 proc set-display-format {f} {
@@ -280,9 +337,6 @@ proc set-display-format {f} {
     }
     update idletasks
     add-title-lines -1 10000 1
-    if {!$busy} {
-        .bot.a.status configure -text "Ready"
-    }
 }
 
 proc initBindings {} {
@@ -335,8 +389,12 @@ proc toplevelG {w} {
     bind $w <Destroy> [list destroyGW $w]
 }
 
-if {[file readable "${libdir}/clientrc.tcl"]} {
-    source "${libdir}/clientrc.tcl"
+if {[file readable "clientrc.tcl"]} {
+    source "clientrc.tcl"
+} else {
+    if {[file readable "${libdir}/clientrc.tcl"]} {
+        source "${libdir}/clientrc.tcl"
+    }
 }
 
 if {[file readable "~/.clientrc.tcl"]} {
@@ -466,12 +524,14 @@ proc show-status {status b sb} {
             .mid.scan configure -state normal
         }
         if {$setNo == 0} {
-            .top.service.m disable 1
-        } elseif {$setOffset > 0 && $setOffset <= [z39.$setNo resultCount]} {
-            .top.service.m enable 1
+            configure-disable-e .top.service.m 1
+        } elseif {[z39.$setNo nextResultSetPosition] > 0 && 
+            [z39.$setNo nextResultSetPosition] <= [z39.$setNo resultCount]} {
+            configure-enable-e .top.service.m 1
             .mid.present configure -state normal
         } else {
-            .top.service.m disable 1
+            configure-disable-e .top.service.m 1
+            .mid.present configure -state disabled
         }
         if {[winfo exists .scan-window]} {
             .scan-window.bot.2 configure -state normal
@@ -541,6 +601,7 @@ proc about-target {} {
     toplevel $w
 
     wm title $w "About target"
+    place-force $w .
     top-down-window $w
 
     frame $w.top.a -relief ridge -border 2
@@ -652,7 +713,8 @@ proc popup-marc {sno no b df} {
                 -yscrollcommand [list $w.top.s set]
         scrollbar $w.top.s -command [list $w.top.record yview]
 
-        if {[tk colormodel .] == "color"} {
+        global monoFlag
+        if {! $monoFlag} {
             $w.top.record tag configure marc-tag -foreground blue
             $w.top.record tag configure marc-id -foreground red
         } else {
@@ -693,8 +755,10 @@ proc popup-marc {sno no b df} {
         }
     } else {
         set i 0
+        $w.bot.formats.m delete 0 last
         foreach f $displayFormats {
-            $w.bot.formats.m entryconfigure $i \
+            $w.bot.formats.m add radiobutton -label $f \
+                    -variable popupMarcdf -value $i \
                     -command [list display-$f $sno $no $w.top.record 0]
             incr i
         }
@@ -707,10 +771,15 @@ proc popup-marc {sno no b df} {
 
 proc update-target-hotlist {target base} {
     global hotTargets
+    global tk4
 
     set len [llength $hotTargets]
     if {$len > 0} {
-        .top.target.m delete 6 [expr 6+[llength $hotTargets]]
+        if {$tk4} {
+            .top.target.m delete 7 [expr 7+[llength $hotTargets]]
+        } else {
+            .top.target.m delete 6 [expr 6+[llength $hotTargets]]
+        }
     }
     set i 0
     foreach e $hotTargets {
@@ -814,22 +883,25 @@ proc open-target {target base} {
         } errorMessage]
     if {$err} {
         tkerror $errorMessage
-        show-status Ready 0 {}
+        show-status "Not connected" 0 {}
         return
     }
     set hostid $target
-    .top.target.m disable 0
-    .top.target.m enable 1
-    .top.target.m enable 2
+    configure-disable-e .top.target.m 0
+    configure-enable-e .top.target.m 1
+    configure-enable-e .top.target.m 2
 }
 
 proc close-target {} {
     global hostid
     global cancelFlag
     global setNo
+    global setNoLast
+    global tk4
 
     set cancelFlag 0
     set setNo 0
+    set setNoLast 0
     .bot.a.set configure -text ""
     set hostid Default
     z39 disconnect
@@ -837,30 +909,36 @@ proc close-target {} {
     show-status {Not connected} 0 0
     init-title-lines
     show-message {}
-    .top.target.m disable 1
-    .top.target.m disable 2
-    .top.target.m enable 0
+    configure-disable-e .top.target.m 1
+    configure-disable-e .top.target.m 2
+    if {$tk4} {
+        .top.rset.m delete 2 last
+    } else {
+        .top.rset.m delete 1 last
+    }
+    .top.rset.m add separator
+    configure-enable-e .top.target.m 0
 }
 
 proc load-set-action {} {
-    global setNo
+    global setNoLast
 
-    incr setNo
-    ir-set z39.$setNo z39
+    incr setNoLast
+    ir-set z39.$setNoLast z39
 
     set fname [.load-set.top.filename.entry get]
     destroy .load-set
     if {$fname != ""} {
         show-status {Loading} 1 {}
         update
-        z39.$setNo loadFile $fname
+        z39.$setNoLast loadFile $fname
 
-        set no [z39.$setNo numberOfRecordsReturned]
-        add-title-lines $setNo $no 1
+        set no [z39.$setNoLast numberOfRecordsReturned]
+        add-title-lines $setNoLast $no 1
     }
-    set l [format "%-4d %7d" $setNo $no]
+    set l [format "%-4d %7d" $setNoLast $no]
     .top.rset.m add command -label $l \
-            -command [list add-title-lines $setNo 10000 1]
+            -command [list add-title-lines $setNoLast 10000 1]
     show-status {Ready} 0 {}
 }
 
@@ -885,7 +963,6 @@ proc load-set {} {
 }
 
 proc init-request {} {
-    global setNo
     global cancelFlag
 
     if {$cancelFlag} {
@@ -926,6 +1003,7 @@ proc init-response {} {
 
 proc search-request {bflag} {
     global setNo
+    global setNoLast
     global profile
     global hostid
     global busy
@@ -953,7 +1031,8 @@ proc search-request {bflag} {
     if {$query==""} {
         return
     }
-    incr setNo
+    incr setNoLast
+    set setNo $setNoLast
     ir-set z39.$setNo z39
 
     if {[lindex $profile($target) 10] == 1} {
@@ -1033,23 +1112,19 @@ proc scan-request {} {
         entry $w.top.entry -relief sunken 
         pack $w.top.entry -fill x -padx 4 -pady 2
         bind $w.top.entry <KeyRelease> [list scan-term-h $attr]
-        if {1} {
-            listbox $w.top.list -yscrollcommand [list $w.top.scroll set] \
-                    -font fixed 
-            scrollbar $w.top.scroll -orient vertical -border 1
-            pack $w.top.list -side left -fill both -expand yes
-            pack $w.top.scroll -side right -fill y
-            $w.top.scroll config -command [list $w.top.list yview]
-        } else {
-            listbox $w.top.list -font fixed -geometry 60x14
-            pack $w.top.list -side left -fill both -expand yes
-        }
+        listbox $w.top.list -yscrollcommand [list $w.top.scroll set] \
+                -font fixed 
+        scrollbar $w.top.scroll -orient vertical -border 1
+        pack $w.top.list -side left -fill both -expand yes
+        pack $w.top.scroll -side right -fill y
+        $w.top.scroll config -command [list $w.top.list yview]
         
         bottom-buttons $w [list {Close} [list destroy $w] \
                 {Up} [list scan-up $attr] \
                 {Down} [list scan-down $attr]] 0
         bind $w.top.list <Up> [list scan-up $attr]
         bind $w.top.list <Down> [list scan-down $attr]
+        focus $w.top.entry
     }
     bind $w.top.list <Double-Button-1> [list scan-copy %y $curIndexEntry]
     wm title $w "Scan $title"
@@ -1273,29 +1348,32 @@ proc search-response {} {
         }
         return
     }
+    set setOffset 0
     set delayRequest {}
     init-title-lines
     set setMax [z39.$setNo resultCount]
+    show-status {Ready} 0 1
+    set status [z39.$setNo responseStatus]
+    if {[lindex $status 0] == "NSD"} {
+        z39.$setNo nextResultSetPosition 0
+        set code [lindex $status 1]
+        set msg [lindex $status 2]
+        set addinfo [lindex $status 3]
+        tkerror "NSD$code: $msg: $addinfo"
+        return
+    }
     show-message "${setMax} hits"
+    if {$setMax == 0} {
+        return
+    }
+    set setOffset 1
+    show-status {Ready} 0 1
     set l [format "%-4d %7d" $setNo $setMax]
     .top.rset.m add command -label $l \
             -command [list add-title-lines $setNo 10000 1]
-    if {$setMax <= 0} {
-        show-status {Ready} 0 1
-        set status [z39.$setNo responseStatus]
-        if {[lindex $status 0] == "NSD"} {
-            set code [lindex $status 1]
-            set msg [lindex $status 2]
-            set addinfo [lindex $status 3]
-            tkerror "NSD$code: $msg: $addinfo"
-        }
-        return
-    }
     if {$setMax > 20} {
         set setMax 20
     }
-    set setOffset 1
-    show-status {Ready} 0 1
     z39 callback {present-response}
     z39.$setNo present $setOffset 1
     show-status {Retrieving} 1 0
@@ -1321,10 +1399,14 @@ proc present-more {number} {
         dputs "setNo=$setNo"
        return
     }
+    set setOffset [z39.$setNo nextResultSetPosition]
+    dputs "setOffest=${setOffset}"
+    dputs "setNo=${setNo}"
     set max [z39.$setNo resultCount]
-    if {$max <= $setOffset} {
+    if {$max < $setOffset} {
         dputs "max=$max"
         dputs "setOffset=$setOffset"
+        show-status Ready 0 1
         return
     }
     if {$number == ""} {
@@ -1359,12 +1441,13 @@ proc title-press {y setno} {
 proc add-title-lines {setno no offset} {
     global displayFormats
     global displayFormat
-    global lastSetNo
+    global setNo
+    global busy
 
-    if {$setno == -1} {
-        set setno $lastSetNo
+    if {$setno != -1} {
+        set setNo $setno
     } else {
-        set lastSetNo $setno
+        set setno $setNo
     }
     if {$offset == 1} {
         .bot.a.set configure -text $setno
@@ -1388,6 +1471,9 @@ proc add-title-lines {setno no offset} {
                 [list popup-marc $setno $o 0 0]
         update idletasks
     }
+    if {!$busy} {
+        show-status Ready 0 1
+    }
 }
 
 proc present-response {} {
@@ -1437,28 +1523,38 @@ proc left-cursor {w} {
         incr i -1
         $w icursor $i
     }
+    dputs left
 }
 
 proc right-cursor {w} {
     set i [$w index insert]
     incr i
+    dputs right
     $w icursor $i
 }
 
 proc bind-fields {list returnAction escapeAction} {
+    global tk4
     set max [expr [llength $list]-1]
     for {set i 0} {$i < $max} {incr i} {
         bind [lindex $list $i] <Return> $returnAction
         bind [lindex $list $i] <Escape> $escapeAction
-        bind [lindex $list $i] <Tab> [list focus [lindex $list [expr $i+1]]]
-        bind [lindex $list $i] <Left> [list left-cursor [lindex $list $i]]
-        bind [lindex $list $i] <Right> [list right-cursor [lindex $list $i]]
+        if {!$tk4} {
+            bind [lindex $list $i] <Tab> \
+                    [list focus [lindex $list [expr $i+1]]]
+            bind [lindex $list $i] <Left> \
+                    [list left-cursor [lindex $list $i]]
+            bind [lindex $list $i] <Right> \
+                    [list right-cursor [lindex $list $i]]
+        }
     }
     bind [lindex $list $i] <Return> $returnAction
     bind [lindex $list $i] <Escape> $escapeAction
-    bind [lindex $list $i] <Tab>    [list focus [lindex $list 0]]
-    bind [lindex $list $i] <Left> [list left-cursor [lindex $list $i]]
-    bind [lindex $list $i] <Right> [list right-cursor [lindex $list $i]]
+    if {!$tk4} {
+        bind [lindex $list $i] <Tab>    [list focus [lindex $list 0]]
+        bind [lindex $list $i] <Left> [list left-cursor [lindex $list $i]]
+        bind [lindex $list $i] <Right> [list right-cursor [lindex $list $i]]
+    }
     focus [lindex $list 0]
 }
 
@@ -1635,9 +1731,9 @@ proc protocol-setup {target} {
     dputs target
     dputs $profile($target)
 
+    frame $w.top.description
     frame $w.top.host
     frame $w.top.port
-    frame $w.top.description
     frame $w.top.idAuthentication
     frame $w.top.maximumRecordSize
     frame $w.top.preferredMessageSize
@@ -1686,8 +1782,14 @@ proc protocol-setup {target} {
             -command [list add-database $target]
     button $w.top.databases.delete -text "Delete" \
             -command [list delete-database $target]
-    listbox $w.top.databases.list -geometry 20x6 \
-            -yscrollcommand "$w.top.databases.scroll set"
+    global tk4
+    if {! $tk4} {
+        listbox $w.top.databases.list -geometry 20x6 \
+                -yscrollcommand "$w.top.databases.scroll set"
+    } else {
+        listbox $w.top.databases.list -width 20 \
+                -yscrollcommand "$w.top.databases.scroll set"
+    }
     scrollbar $w.top.databases.scroll -orient vertical -border 1
     pack $w.top.databases.label -side top -fill x \
             -padx 2 -pady 2
@@ -1797,7 +1899,7 @@ proc cascade-target-list {} {
         destroy $sub
     }
     .top.target.m.clist delete 0 last
-    foreach n [array names profile] {
+    foreach n [lsort [array names profile]] {
         if {$n != "Default"} {
             set nl [lindex $profile($n) 12]
             if {[llength [lindex $profile($n) 7]] > 1} {
@@ -1815,7 +1917,7 @@ proc cascade-target-list {} {
         }
     }
     .top.target.m.slist delete 0 last
-    foreach n [array names profile] {
+    foreach n [lsort [array names profile]] {
         if {$n != "Default"} {
             .top.target.m.slist add command -label $n \
                     -command [list protocol-setup $n]
@@ -1892,7 +1994,7 @@ proc query-delete {queryNo} {
 
     label $w.top.warning -bitmap warning
     message $w.top.quest -text "Are you sure you want to delete the \
-query type $n ?"  -aspect 200
+query type $n ?"  -aspect 300
     pack $w.top.warning $w.top.quest -side left -expand yes -padx 10 -pady 5
     bottom-buttons $w [list {Ok} [list query-delete-action $queryNo] \
                             {Cancel} [list destroy $w]] 1
@@ -1957,9 +2059,15 @@ proc save-settings {} {
     global queryInfo
    
     if {![file writable "${libdir}/clientrc.tcl"]} {
-       return
+        set a [alert "Cannot open ${libdir}/clientrc.tcl for writing. Do you \
+                wish to save clientrc.tcl in the current directory instead?"]
+        if {! $a} {
+            return
+        }
+        set f [open "clientrc.tcl" w]
+    } else {
+        set f [open "${libdir}/clientrc.tcl" w]
     }
-    set f [open "${libdir}/clientrc.tcl" w]
     puts $f "# Setup file"
 
     foreach n [array names profile] {
@@ -1992,7 +2100,7 @@ proc alert {ask} {
     top-down-window $w
 
     label $w.top.warning -bitmap warning
-    message $w.top.message -text $ask -aspect 200 \
+    message $w.top.message -text $ask -aspect 300 \
             -font -Adobe-Times-Medium-R-Normal-*-180-*
 
     pack $w.top.warning $w.top.message -side left -pady 5 -padx 10 -expand yes
@@ -2309,6 +2417,7 @@ proc use-attr {init} {
         {Content type}                 1034 
         {Anywhere}                     1035 
     }
+    global tk4
     set w .index-setup
     global useTmpValue
     set l [llength $attr]
@@ -2324,8 +2433,13 @@ proc use-attr {init} {
             }
             incr lno
         }
-        $w.top.use.list select from $s
-        $w.top.use.list select to $s
+        if {$tk4} {
+            $w.top.use.list selection clear 0 end
+            $w.top.use.list selection set $s $s
+        } else {
+            $w.top.use.list select from $s
+            $w.top.use.list select to $s
+        }
         incr s -3
         if {$s < 0} {
             set s 0
@@ -2387,6 +2501,7 @@ proc index-setup {attr queryNo indexNo} {
     global completenessTmpValue
     global positionTmpValue
     global useTmpValue
+    global tk4
     set relationTmpValue 0
     set truncationTmpValue 0
     set structureTmpValue 0
@@ -2441,8 +2556,13 @@ proc index-setup {attr queryNo indexNo} {
     pack $w.top.use -side left -pady 6 -padx 6 -fill y
 
     label $w.top.use.label -text "Use"
-    listbox $w.top.use.list -geometry 26x10 \
-            -yscrollcommand "$w.top.use.scroll set"
+    if {$tk4} {
+        listbox $w.top.use.list -width 26 \
+                -yscrollcommand "$w.top.use.scroll set"
+    } else {
+        listbox $w.top.use.list -geometry 26x10 \
+                -yscrollcommand "$w.top.use.scroll set"
+    }
     scrollbar $w.top.use.scroll -orient vertical -border 1
     pack $w.top.use.label -side top -fill x \
             -padx 2 -pady 2
@@ -2532,6 +2652,7 @@ proc query-setup {queryNo} {
     global queryButtonsTmp
     global queryInfoTmp
     global queryIndexTmp
+    global tk4
     
     set queryIndexTmp 0
     set queryName [lindex $queryTypes $queryNo]
@@ -2574,8 +2695,13 @@ proc query-setup {queryNo} {
     pack $w.top.index.list -side left -fill both -expand yes -padx 2 -pady 2
     pack $w.top.index.scroll -side right -fill y -padx 2 -pady 2
 
-    $w.top.index.list select from 0
-    $w.top.index.list select to 0
+    if {$tk4} {
+        $w.top.index.list selection clear 0 end
+        $w.top.index.list selection set 0 0
+    } else {
+        $w.top.index.list select from 0
+        $w.top.index.list select to 0
+    }
 
     foreach x $queryInfoTmp {
         $w.top.index.list insert end [lindex $x 0]
@@ -2648,16 +2774,24 @@ proc index-query {} {
 
 proc index-focus-in {w i} {
     global curIndexEntry
+    global tk4
 
-    $w.$i configure -background red
+    if {! $tk4} {
+        $w.$i configure -background red
+    }
     set curIndexEntry $i
 }
 
 proc index-lines {w realOp buttonInfo queryInfo handle} {
+    global tk4
     set i 0
     foreach b $buttonInfo {
         if {! [winfo exists $w.$i]} {
-            frame $w.$i -background white -border 1
+            if {$tk4} {
+                frame $w.$i -border 0
+            } else {
+                frame $w.$i -background white -border 1
+            }
         }
         listbuttonx $w.$i.l [lindex $b 1] $queryInfo $handle $i
 
@@ -2670,8 +2804,10 @@ proc index-lines {w realOp buttonInfo queryInfo handle} {
                 pack $w.$i.l -side left
                 pack $w.$i.e -side left -fill x -expand yes
                 pack $w.$i -side top -fill x -padx 2 -pady 2
-                bind $w.$i.e <Left> [list left-cursor $w.$i.e]
-                bind $w.$i.e <Right> [list right-cursor $w.$i.e]
+                if {!$tk4} {
+                    bind $w.$i.e <Left> [list left-cursor $w.$i.e]
+                    bind $w.$i.e <Right> [list right-cursor $w.$i.e]
+                }
                 bind $w.$i.e <Return> {search-request 0}
             }
         } else {
@@ -2688,15 +2824,19 @@ proc index-lines {w realOp buttonInfo queryInfo handle} {
     if {! $realOp} {
         return
     }
-    set j 0
-    incr i -1
-    while {$j < $i} {
-        set k [expr $j+1]
-        bind $w.$j.e <Tab> "focus $w.$k.e"
-        set j $k
+    if {! $tk4} {
+        set j 0
+        incr i -1
+        while {$j < $i} {
+            set k [expr $j+1]
+            bind $w.$j.e <Tab> "focus $w.$k.e"
+            set j $k
+        }
     }
     if {$i >= 0} {
-        bind $w.$i.e <Tab> "focus $w.0.e"
+        if {! $tk4} {
+            bind $w.$i.e <Tab> "focus $w.0.e"
+        }
         focus $w.0.e
     }
 }
@@ -2769,8 +2909,8 @@ menu .top.target.m
 .top.target.m add separator
 set-target-hotlist
 
-.top.target.m disable 1
-.top.target.m disable 2
+configure-disable-e .top.target.m 1
+configure-disable-e .top.target.m 2
 
 menu .top.target.m.clist
 menu .top.target.m.slist
@@ -2881,11 +3021,15 @@ pack .mid.search .mid.scan .mid.present .mid.clear -side left \
 text .data.record -height 2 -width 20 -wrap none \
         -yscrollcommand [list .data.scroll set] -wrap $textWrap
 scrollbar .data.scroll -command [list .data.record yview]
+if {$tk4} {
+    .data.record configure -takefocus 0
+    .data.scroll configure -takefocus 0
+}
 pack .data.scroll -side right -fill y
 pack .data.record -expand yes -fill both
 initBindings
 
-if {[tk colormodel .] == "color"} {
+if {! $monoFlag} {
     .data.record tag configure marc-tag -foreground blue
     .data.record tag configure marc-id -foreground red
 } else {
@@ -2895,6 +3039,9 @@ if {[tk colormodel .] == "color"} {
 .data.record tag configure marc-data -foreground black
 
 button .bot.logo  -bitmap @${libdir}/bitmaps/book1 -command cancel-operation
+if {$tk4} {
+    .bot.logo configure -takefocus 0
+}
 frame .bot.a
 pack .bot.a -side left -fill x
 pack .bot.logo -side right -padx 2 -pady 2