prati0100 / git-gui

Tcl/Tk based UI for Git. I am currently acting as the project's maintainer.
160 stars 87 forks source link

commit message text cursor missing in dark theme for macOS #39

Closed lormanlau closed 3 years ago

lormanlau commented 4 years ago

The text cursor when typing in a commit message is not visible while in dark theme. In light theme the cursor blinks and shows normally. I suspect that the cursor is blinking white.

macOS Catalina 10.15.6 git-gui version 0.21.0.72.g46972 git version 2.28.0

Tcl/Tk version 8.6.10

Screen Shot 2020-07-29 at 10 01 43 AM Screen Shot 2020-07-29 at 10 02 06 AM
prati0100 commented 4 years ago

Seems related to #30. I unfortunately don't have a lot of free time so I can't say for sure when I can get around to looking at it.

last-partizan commented 4 years ago

I don't know about workaround, but proper way to fix this would be to support theming.

Tcl/Tk will have (i hope), proper dark themes by the 8.7 version, and git citool must not hardcode color values.

Gitk does have color configuration on its own, and with some effort i can get it to look like this:

image

But git citool just uses plain hardcoded color values, so it looks horrible when themed.

@prati0100 i can try to make things just little better, how do i make merge requests? I see this project uses maillist instead of gitlab tools, so i just make patch and send it to maillist?

prati0100 commented 4 years ago

Yes, you can make a patch and send it to git@vger.kernel.org. Or if you prefer to use GitHub pull requests, you can use GitGitGadget. See the readme for more info.

last-partizan commented 4 years ago

@lormanlau

https://github.com/last-partizan/git-gui/tree/dark-colors

Please check if this branch fixed bug for you.

prati0100 commented 3 years ago

Should be fixed with https://github.com/prati0100/git-gui/commit/01121d6132135e62d4bf7c2e67e749ae3037d0ba. If you still see the issue please feel free to reopen.