chriskempson / tomorrow-theme

Tomorrow Theme
Other
13.73k stars 3.17k forks source link

Fix cursor colors of PuTTY #306

Closed yous closed 2 years ago

yous commented 9 years ago

Diff is here:

diff --git a/PuTTY/Tomorrow Night.reg b/PuTTY/Tomorrow Night.reg
index 53175da..ab6c611 100644
--- a/PuTTY/Tomorrow Night.reg
+++ b/PuTTY/Tomorrow Night.reg
@@ -1,11 +1,11 @@
 Windows Registry Editor Version 5.00

-[HKEY_CURRENT_USER\Software\SimonTatham\PuTTY\Sessions\Tomorrow Night]
+[HKEY_CURRENT_USER\Software\SimonTatham\PuTTY\Sessions\Tomorrow%20Night]^M
 "Colour0"="197,200,198"
 "Colour1"="234,234,234"
 "Colour2"="29,31,33"
 "Colour3"="0,0,0"
-"Colour4"="197,200,198"
+"Colour4"="29,31,33"^M
 "Colour5"="197,200,198"
 "Colour6"="29,31,33"
 "Colour7"="0,0,0"
diff --git a/PuTTY/Tomorrow.reg b/PuTTY/Tomorrow.reg
index b68f16f..f01454f 100644
--- a/PuTTY/Tomorrow.reg
+++ b/PuTTY/Tomorrow.reg
@@ -6,7 +6,7 @@ Windows Registry Editor Version 5.00
 "Colour2"="250,250,250"
 "Colour3"="0,0,0"
 "Colour4"="214,214,214"
-"Colour5"="214,214,214"
+"Colour5"="77,77,76"^M
 "Colour6"="142,144,140"
 "Colour7"="0,0,0"
 "Colour8"="255,51,52"

First, we should use %20 instead of space in session name of PuTTY. Also Colour4 is for 'Cursor Text', Colour5 is for 'Cursor Colour', so they must be different.

yous commented 9 years ago

Also this fixes #84, fixes #251.

yous commented 8 years ago

@chriskempson Would you check this one?