Re: [PATCH] gitk: use user-configured background in view definition dialog

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, I suggest to apply this change to gitk.  Thanks, Gerrit.


On Fri, Dec 28, 2007 at 02:51:56PM +0000, Gerrit Pape wrote:
> Have the text fields in the view definition dialog (View->New view...)
> use the background color as configured through the preferences, instead
> of hard-coded 'white'.
> 
> This was suggested by Paul Wise through
>  http://bugs.debian.org/457124
> 
> Signed-off-by: Gerrit Pape <pape@xxxxxxxxxxx>
> ---
>  gitk |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/gitk b/gitk
> index 684e614..7d70c64 100755
> --- a/gitk
> +++ b/gitk
> @@ -1881,7 +1881,7 @@ proc editview {} {
>  
>  proc vieweditor {top n title} {
>      global newviewname newviewperm viewfiles
> -    global uifont
> +    global uifont bgcolor
>  
>      toplevel $top
>      wm title $top $title
> @@ -1895,12 +1895,12 @@ proc vieweditor {top n title} {
>  	-text [mc "Commits to include (arguments to git rev-list):"]
>      grid $top.al - -sticky w -pady 5
>      entry $top.args -width 50 -textvariable newviewargs($n) \
> -	-background white -font uifont
> +	-background $bgcolor -font uifont
>      grid $top.args - -sticky ew -padx 5
>      message $top.l -aspect 1000 -font uifont \
>  	-text [mc "Enter files and directories to include, one per line:"]
>      grid $top.l - -sticky w
> -    text $top.t -width 40 -height 10 -background white -font uifont
> +    text $top.t -width 40 -height 10 -background $bgcolor -font uifont
>      if {[info exists viewfiles($n)]} {
>  	foreach f $viewfiles($n) {
>  	    $top.t insert end $f
> -- 
> 1.5.3.7
> 
> -
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux