Re: [PATCH] gitk: fix selection color not persistent

"gh acct via GitGitGadget" [off-list ref] writes:

From: ghacct <redacted>

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 <redacted>
Signed-off-by: ghacct <redacted>

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).