git-gui: Remove the loaded column from the blame viewer
[gitweb.git] / lib / blame.tcl
index 02e439cbc6a2c6aec238d1eb656ce26c594cbb3f..ca73ba7d14f608dd705d25093b8c6d312b9d1642 100644 (file)
@@ -3,17 +3,23 @@
 
 class blame {
 
-field commit  ; # input commit to blame
-field path    ; # input filename to view in $commit
-
-field w
-field w_line
-field w_cgrp
-field w_load
-field w_file
-field w_cmit
-field status
-
+image create photo ::blame::img_back_arrow -data {R0lGODlhGAAYAIUAAPwCBEzKXFTSZIz+nGzmhGzqfGTidIT+nEzGXHTqhGzmfGzifFzadETCVES+VARWDFzWbHzyjAReDGTadFTOZDSyRDyyTCymPARaFGTedFzSbDy2TCyqRCyqPARaDAyCHES6VDy6VCyiPAR6HCSeNByWLARyFARiDARqFGTifARiFAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAACH5BAEAAAAALAAAAAAYABgAAAajQIBwSCwaj8ikcsk0BppJwRPqHEypQwHBis0WDAdEFyBIKBaMAKLBdjQeSkFBYTBAIvgEoS6JmhUTEwIUDQ4VFhcMGEhyCgoZExoUaxsWHB0THkgfAXUGAhoBDSAVFR0XBnCbDRmgog0hpSIiDJpJIyEQhBUcJCIlwA22SSYVogknEg8eD82qSigdDSknY0IqJQXPYxIl1dZCGNvWw+Dm510GQQAh/mhDcmVhdGVkIGJ5IEJNUFRvR0lGIFBybyB2ZXJzaW9uIDIuNQ0KqSBEZXZlbENvciAxOTk3LDE5OTguIEFsbCByaWdodHMgcmVzZXJ2ZWQuDQpodHRwOi8vd3d3LmRldmVsY29yLmNvbQA7}
+
+field commit    ; # input commit to blame
+field path      ; # input filename to view in $commit
+field history {}; # viewer history: {commit path}
+
+field w          ; # top window in this viewer
+field w_back     ; # our back button
+field w_path     ; # label showing the current file path
+field w_line     ; # text column: all line numbers
+field w_cgrp     ; # text column: abbreviated commit SHA-1s
+field w_file     ; # text column: actual file data
+field w_cmit     ; # pane showing commit message
+field status     ; # text variable bound to status bar
+field old_height ; # last known height of $w.file_pane
+
+field current_fd       {} ; # background process running
 field highlight_line   -1 ; # current line selected
 field highlight_commit {} ; # sha1 of commit selected
 
@@ -51,28 +57,62 @@ constructor new {i_commit i_path} {
 
        make_toplevel top w
        wm title $top "[appname] ([reponame]): File Viewer"
-       set status "Loading $commit:$path..."
 
-       label $w.path -text "$commit:$path" \
+       frame $w.header -background orange
+       label $w.header.commit_l \
+               -text {Commit:} \
+               -background orange \
                -anchor w \
-               -justify left \
-               -borderwidth 1 \
-               -relief sunken \
-               -font font_uibold
-       pack $w.path -side top -fill x
-
-       frame $w.out
-       set w_load $w.out.loaded_t
-       text $w_load \
-               -background white -borderwidth 0 \
+               -justify left
+       set w_back $w.header.commit_b
+       label $w_back \
+               -image ::blame::img_back_arrow \
+               -borderwidth 0 \
+               -relief flat \
                -state disabled \
-               -wrap none \
-               -height 40 \
-               -width 1 \
-               -font font_diff
-       $w_load tag conf annotated -background grey
-
-       set w_line $w.out.linenumber_t
+               -background orange \
+               -activebackground orange
+       bind $w_back <Button-1> "
+               if {\[$w_back cget -state\] eq {normal}} {
+                       [cb _history_menu]
+               }
+               "
+       label $w.header.commit \
+               -textvariable @commit \
+               -background orange \
+               -anchor w \
+               -justify left
+       label $w.header.path_l \
+               -text {File:} \
+               -background orange \
+               -anchor w \
+               -justify left
+       set w_path $w.header.path
+       label $w_path \
+               -background orange \
+               -anchor w \
+               -justify left
+       pack $w.header.commit_l -side left
+       pack $w_back -side left
+       pack $w.header.commit -side left
+       pack $w_path -fill x -side right
+       pack $w.header.path_l -side right
+
+       panedwindow $w.file_pane -orient vertical
+       frame $w.file_pane.out
+       frame $w.file_pane.cm
+       $w.file_pane add $w.file_pane.out \
+               -sticky nsew \
+               -minsize 100 \
+               -height 100 \
+               -width 100
+       $w.file_pane add $w.file_pane.cm \
+               -sticky nsew \
+               -minsize 25 \
+               -height 25 \
+               -width 100
+
+       set w_line $w.file_pane.out.linenumber_t
        text $w_line \
                -background white -borderwidth 0 \
                -state disabled \
@@ -82,7 +122,7 @@ constructor new {i_commit i_path} {
                -font font_diff
        $w_line tag conf linenumber -justify right
 
-       set w_cgrp $w.out.commit_t
+       set w_cgrp $w.file_pane.out.commit_t
        text $w_cgrp \
                -background white -borderwidth 0 \
                -state disabled \
@@ -90,21 +130,30 @@ constructor new {i_commit i_path} {
                -height 40 \
                -width 4 \
                -font font_diff
-
-       set w_file $w.out.file_t
+       $w_cgrp tag conf curr_commit
+       $w_cgrp tag conf prior_commit \
+               -foreground blue \
+               -underline 1
+       $w_cgrp tag bind prior_commit \
+               <Button-1> \
+               "[cb _load_commit @%x,%y];break"
+
+       set w_file $w.file_pane.out.file_t
        text $w_file \
                -background white -borderwidth 0 \
                -state disabled \
                -wrap none \
                -height 40 \
                -width 80 \
-               -xscrollcommand [list $w.out.sbx set] \
+               -xscrollcommand [list $w.file_pane.out.sbx set] \
                -font font_diff
 
-       scrollbar $w.out.sbx -orient h -command [list $w_file xview]
-       scrollbar $w.out.sby -orient v \
+       scrollbar $w.file_pane.out.sbx \
+               -orient h \
+               -command [list $w_file xview]
+       scrollbar $w.file_pane.out.sby \
+               -orient v \
                -command [list scrollbar2many [list \
-               $w_load \
                $w_line \
                $w_cgrp \
                $w_file \
@@ -112,33 +161,22 @@ constructor new {i_commit i_path} {
        grid \
                $w_cgrp \
                $w_line \
-               $w_load \
                $w_file \
-               $w.out.sby \
+               $w.file_pane.out.sby \
                -sticky nsew
-       grid conf $w.out.sbx -column 3 -sticky we
-       grid columnconfigure $w.out 3 -weight 1
-       grid rowconfigure $w.out 0 -weight 1
-       pack $w.out -fill both -expand 1
-
-       label $w.status \
-               -textvariable @status \
-               -anchor w \
-               -justify left \
-               -borderwidth 1 \
-               -relief sunken
-       pack $w.status -side bottom -fill x
+       grid conf $w.file_pane.out.sbx -column 2 -sticky we
+       grid columnconfigure $w.file_pane.out 2 -weight 1
+       grid rowconfigure $w.file_pane.out 0 -weight 1
 
-       frame $w.cm
-       set w_cmit $w.cm.t
+       set w_cmit $w.file_pane.cm.t
        text $w_cmit \
                -background white -borderwidth 0 \
                -state disabled \
                -wrap none \
                -height 10 \
                -width 80 \
-               -xscrollcommand [list $w.cm.sbx set] \
-               -yscrollcommand [list $w.cm.sby set] \
+               -xscrollcommand [list $w.file_pane.cm.sbx set] \
+               -yscrollcommand [list $w.file_pane.cm.sby set] \
                -font font_diff
        $w_cmit tag conf header_key \
                -tabs {3c} \
@@ -148,12 +186,24 @@ constructor new {i_commit i_path} {
                -background $active_color \
                -font font_ui
        $w_cmit tag raise sel
-       scrollbar $w.cm.sbx -orient h -command [list $w_cmit xview]
-       scrollbar $w.cm.sby -orient v -command [list $w_cmit yview]
-       pack $w.cm.sby -side right -fill y
-       pack $w.cm.sbx -side bottom -fill x
+       scrollbar $w.file_pane.cm.sbx \
+               -orient h \
+               -command [list $w_cmit xview]
+       scrollbar $w.file_pane.cm.sby \
+               -orient v \
+               -command [list $w_cmit yview]
+       pack $w.file_pane.cm.sby -side right -fill y
+       pack $w.file_pane.cm.sbx -side bottom -fill x
        pack $w_cmit -expand 1 -fill both
-       pack $w.cm -side bottom -fill x
+
+       frame $w.status \
+               -borderwidth 1 \
+               -relief sunken
+       label $w.status.l \
+               -textvariable @status \
+               -anchor w \
+               -justify left
+       pack $w.status.l -side left
 
        menu $w.ctxm -tearoff 0
        $w.ctxm add command \
@@ -162,17 +212,15 @@ constructor new {i_commit i_path} {
 
        foreach i [list \
                $w_cgrp \
-               $w_load \
                $w_line \
                $w_file] {
                $i conf -cursor $cursor_ptr
                $i conf -yscrollcommand \
                        [list many2scrollbar [list \
                        $w_cgrp \
-                       $w_load \
                        $w_line \
                        $w_file \
-                       ] yview $w.out.sby]
+                       ] yview $w.file_pane.out.sby]
                bind $i <Button-1> "
                        [cb _hide_tooltip]
                        [cb _click $i @%x,%y]
@@ -192,7 +240,6 @@ constructor new {i_commit i_path} {
 
        foreach i [list \
                $w_cgrp \
-               $w_load \
                $w_line \
                $w_file \
                $w_cmit] {
@@ -210,8 +257,88 @@ constructor new {i_commit i_path} {
 
        bind $w_cmit <Button-1> [list focus $w_cmit]
        bind $top <Visibility> [list focus $top]
-       bind $top <Destroy> [list delete_this $this]
+       bind $w_file <Destroy> [list delete_this $this]
+
+       grid configure $w.header -sticky ew
+       grid configure $w.file_pane -sticky nsew
+       grid configure $w.status -sticky ew
+       grid columnconfigure $top 0 -weight 1
+       grid rowconfigure $top 0 -weight 0
+       grid rowconfigure $top 1 -weight 1
+       grid rowconfigure $top 2 -weight 0
+
+       set req_w [winfo reqwidth  $top]
+       set req_h [winfo reqheight $top]
+       if {$req_w < 600} {set req_w 600}
+       if {$req_h < 400} {set req_h 400}
+       set g "${req_w}x${req_h}"
+       wm geometry $top $g
+       update
+
+       set old_height [winfo height $w.file_pane]
+       $w.file_pane sash place 0 \
+               [lindex [$w.file_pane sash coord 0] 0] \
+               [expr {int($old_height * 0.70)}]
+       bind $w.file_pane <Configure> \
+       "if {{$w.file_pane} eq {%W}} {[cb _resize %h]}"
+
+       _load $this
+}
+
+method _load {} {
+       _hide_tooltip $this
+
+       if {$total_lines != 0 || $current_fd ne {}} {
+               if {$current_fd ne {}} {
+                       catch {close $current_fd}
+                       set current_fd {}
+               }
+
+               set highlight_line -1
+               set highlight_commit {}
+               set total_lines 0
+               set blame_lines 0
+               set commit_count 0
+               set commit_list {}
+               array unset order
+               array unset line_commit
+               array unset line_file
+
+               $w_cgrp conf -state normal
+               $w_line conf -state normal
+               $w_file conf -state normal
+
+               $w_cgrp delete 0.0 end
+               $w_line delete 0.0 end
+               $w_file delete 0.0 end
+
+               $w_cgrp conf -state disabled
+               $w_line conf -state disabled
+               $w_file conf -state disabled
+       }
+
+       if {[winfo exists $w.status.c]} {
+               $w.status.c coords bar 0 0 0 20
+       } else {
+               canvas $w.status.c \
+                       -width 100 \
+                       -height [expr {int([winfo reqheight $w.status.l] * 0.6)}] \
+                       -borderwidth 1 \
+                       -relief groove \
+                       -highlightt 0
+               $w.status.c create rectangle 0 0 0 20 -tags bar -fill navy
+               pack $w.status.c -side right
+       }
 
+       if {$history eq {}} {
+               $w_back conf -state disabled
+       } else {
+               $w_back conf -state normal
+       }
+       lappend history [list $commit $path]
+
+       set status "Loading $commit:[escape_path $path]..."
+       $w_path conf -text [escape_path $path]
        if {$commit eq {}} {
                set fd [open $path r]
        } else {
@@ -220,10 +347,55 @@ constructor new {i_commit i_path} {
        }
        fconfigure $fd -blocking 0 -translation lf -encoding binary
        fileevent $fd readable [cb _read_file $fd]
+       set current_fd $fd
+}
+
+method _history_menu {} {
+       set m $w.backmenu
+       if {[winfo exists $m]} {
+               $m delete 0 end
+       } else {
+               menu $m -tearoff 0
+       }
+
+       for {set i [expr {[llength $history] - 2}]
+               } {$i >= 0} {incr i -1} {
+               set e [lindex $history $i]
+               set c [lindex $e 0]
+               set f [lindex $e 1]
+
+               if {[regexp {^[0-9a-f]{40}$} $c]} {
+                       set t [string range $c 0 8]...
+               } else {
+                       set t $c
+               }
+               if {![catch {set summary $header($c,summary)}]} {
+                       append t " $summary"
+                       if {[string length $t] > 70} {
+                               set t [string range $t 0 66]...
+                       }
+               }
+
+               $m add command -label $t -command [cb _goback $i $c $f]
+       }
+       set X [winfo rootx $w_back]
+       set Y [expr {[winfo rooty $w_back] + [winfo height $w_back]}]
+       tk_popup $m $X $Y
+}
+
+method _goback {i c f} {
+       set history [lrange $history 0 [expr {$i - 1}]]
+       set commit $c
+       set path $f
+       _load $this
 }
 
 method _read_file {fd} {
-       $w_load conf -state normal
+       if {$fd ne $current_fd} {
+               catch {close $fd}
+               return
+       }
+
        $w_cgrp conf -state normal
        $w_line conf -state normal
        $w_file conf -state normal
@@ -232,7 +404,6 @@ method _read_file {fd} {
                incr total_lines
 
                if {$total_lines > 1} {
-                       $w_load insert end "\n"
                        $w_cgrp insert end "\n"
                        $w_line insert end "\n"
                        $w_file insert end "\n"
@@ -241,7 +412,6 @@ method _read_file {fd} {
                $w_line insert end "$total_lines" linenumber
                $w_file insert end "$line"
        }
-       $w_load conf -state disabled
        $w_cgrp conf -state disabled
        $w_line conf -state disabled
        $w_file conf -state disabled
@@ -249,7 +419,7 @@ method _read_file {fd} {
        if {[eof $fd]} {
                close $fd
                _status $this
-               set cmd [list git blame -M -C --incremental]
+               set cmd {nice git blame -M -C --incremental}
                if {$commit eq {}} {
                        lappend cmd --contents $path
                } else {
@@ -259,12 +429,18 @@ method _read_file {fd} {
                set fd [open "| $cmd" r]
                fconfigure $fd -blocking 0 -translation lf -encoding binary
                fileevent $fd readable [cb _read_blame $fd]
+               set current_fd $fd
        }
 } ifdeleted { catch {close $fd} }
 
 method _read_blame {fd} {
        variable group_colors
 
+       if {$fd ne $current_fd} {
+               catch {close $fd}
+               return
+       }
+
        $w_cgrp conf -state normal
        while {[gets $fd line] >= 0} {
                if {[regexp {^([a-z0-9]{40}) (\d+) (\d+) (\d+)$} $line line \
@@ -295,7 +471,12 @@ method _read_blame {fd} {
 
                        if {[regexp {^0{40}$} $cmit]} {
                                set commit_abbr work
+                               set commit_type curr_commit
+                       } elseif {$cmit eq $commit} {
+                               set commit_abbr this
+                               set commit_type curr_commit
                        } else {
+                               set commit_type prior_commit
                                set commit_abbr [string range $cmit 0 4]
                        }
 
@@ -322,17 +503,17 @@ method _read_blame {fd} {
 
                        set first_lno $lno
                        while {
-                       ![catch {set ncmit $line_commit([expr {$first_lno - 1}])}]
+                          ![catch {set ncmit $line_commit([expr {$first_lno - 1}])}]
+                       && ![catch {set nfile $line_file([expr {$first_lno - 1}])}]
                        && $ncmit eq $cmit
+                       && $nfile eq $file
                        } {
                                incr first_lno -1
                        }
 
                        while {$n > 0} {
                                set lno_e "$lno.0 lineend + 1c"
-                               if {[catch {set g g$line_commit($lno)}]} {
-                                       $w_load tag add annotated $lno.0 $lno_e
-                               } else {
+                               if {![catch {set g g$line_commit($lno)}]} {
                                        $w_cgrp tag remove g$g $lno.0 $lno_e
                                        $w_line tag remove g$g $lno.0 $lno_e
                                        $w_file tag remove g$g $lno.0 $lno_e
@@ -347,7 +528,7 @@ method _read_blame {fd} {
 
                                $w_cgrp delete $lno.0 "$lno.0 lineend"
                                if {$lno == $first_lno} {
-                                       $w_cgrp insert $lno.0 $commit_abbr
+                                       $w_cgrp insert $lno.0 $commit_abbr $commit_type
                                } elseif {$lno == [expr {$first_lno + 1}]} {
                                        $w_cgrp insert $lno.0 $author_abbr
                                } else {
@@ -376,12 +557,16 @@ method _read_blame {fd} {
                                incr blame_lines
                        }
 
-                       while {![catch {set ncmit $line_commit($lno)}]
-                               && $ncmit eq $cmit} {
+                       while {
+                          ![catch {set ncmit $line_commit($lno)}]
+                       && ![catch {set nfile $line_file($lno)}]
+                       && $ncmit eq $cmit
+                       && $nfile eq $file
+                       } {
                                $w_cgrp delete $lno.0 "$lno.0 lineend"
 
                                if {$lno == $first_lno} {
-                                       $w_cgrp insert $lno.0 $commit_abbr
+                                       $w_cgrp insert $lno.0 $commit_abbr $commit_type
                                } elseif {$lno == [expr {$first_lno + 1}]} {
                                        $w_cgrp insert $lno.0 $author_abbr
                                } else {
@@ -398,7 +583,9 @@ method _read_blame {fd} {
 
        if {[eof $fd]} {
                close $fd
+               set current_fd {}
                set status {Annotation complete.}
+               destroy $w.status.c
        } else {
                _status $this
        }
@@ -413,6 +600,7 @@ method _status {} {
        set status [format \
                "Loading annotations... %i of %i lines annotated (%2i%%)" \
                $have $total $pdone]
+       $w.status.c coords bar 0 0 $pdone 20
 }
 
 method _click {cur_w pos} {
@@ -421,6 +609,16 @@ method _click {cur_w pos} {
        _showcommit $this $lno
 }
 
+method _load_commit {pos} {
+       set lno [lindex [split [$w_cgrp index $pos] .] 0]
+       if {[catch {set cmit $line_commit($lno)}]} return
+       if {[catch {set file $line_file($lno)  }]} return
+
+       set commit $cmit
+       set path $file
+       _load $this
+}
+
 method _showcommit {lno} {
        global repo_config
        variable active_color
@@ -590,6 +788,13 @@ method _open_tooltip {cur_w} {
 $author_name $author_email  $author_time
 $summary"
 
+       set file $line_file($lno)
+       if {$file ne $path} {
+               append tooltip_text "
+
+Original File: $file"
+       }
+
        if {$tooltip_wm ne "$cur_w.tooltip"} {
                _hide_tooltip $this
 
@@ -633,4 +838,19 @@ method _hide_tooltip {} {
        }
 }
 
+method _resize {new_height} {
+       set diff [expr {$new_height - $old_height}]
+       if {$diff == 0} return
+
+       set my [expr {[winfo height $w.file_pane] - 25}]
+       set o [$w.file_pane sash coord 0]
+       set ox [lindex $o 0]
+       set oy [expr {[lindex $o 1] + $diff}]
+       if {$oy < 0}   {set oy 0}
+       if {$oy > $my} {set oy $my}
+       $w.file_pane sash place 0 $ox $oy
+
+       set old_height $new_height
+}
+
 }