patchgitk: fix selection color not persistent

From: ghacct <tony@viulogix.com>

Dear git community, Is it possible please for the gitk selection color setting to be persistent? Since 2015 this has been working for me. Thank you for considering!

Signed-off-by: tgomez <tgomez@usa.wtbts.net>
Signed-off-by: ghacct <tony@viulogix.com>
---
    gitk: fix selection color not persistent
    
    Dear git community, Is it possible please for the gitk selection color
    setting to be persistent? Since 2015 this has been working for me. Thank
    you for considering!

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-926%2Fghacct%2Fpatch-1-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-926/ghacct/patch-1-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/926

 gitk-git/gitk | 1 +
 1 file changed, 1 insertion(+)

On 03/04/21 02.22, gh acct via GitGitGadget wrote:

Is it anonymous contribution?

The patch/commit message seems like question asking for something,
but you should instead describe why you made this patch, as our
convention here.
  

I asked to you: Is tgomez co-authored this patch or just helping?

[CC] Junio, what do you think about this patch?

"gh acct via GitGitGadget" <gitgitgadget@gmail.com> writes:

cf. Documentation/SubmittingPatches [[describe-changes]] [[summary-section]] [[meaningful-message]] [[imperative-mood]] [[sign-off]] [[real-name]]

Also 'gitk' patches should be based on Paul's tree, not mine. cf. "Subsystems with dedicated maintainers" section in Documentation/SubmittingPatches

So the diff should look more like 

	diff --git a/gitk b/gitk
	index ...
	--- a/gitk
	+++ b/gitk
	@@ ...

As to the contents of the patch itself, I am not the best person to comment on. I no longer use gitk myself unless I am testing gitk, and I no longer speak Tcl/Tk.

Having said that, I see no existing instance of -selectforeground so in the absense of somebody who is better versed in Tk, I'd be wary of this change. I, however, see that

    text some-widget \
    -background $bgcolor -foreground $fgcolor -selectbackground $selectbgcolor

is a recurring theme for other text widgets like $cflist and $top.list, so if the change were only to add the missing "-selectbackground $selectbgcolor" there, I would imagine it would be a safe improvement (or adding the "-selectforeground $fgcolor" to other text widgets that currently lack might turn out to be a sensible change, too, but as I said, I do not speak Tcl/Tk).