Re: [PATCH] git-gui: Basic dark mode support
- From
Serg Tereshchenko <serg.partizan@gmail.com>
- Date
- Oct 8, 2020, 08:24 UTC
- Message-ID
- <20201008082439.6013-1-serg.partizan@gmail.com>
- In-Reply-To
- <20201007110751.237kem2mlnb7hbrk@yadavpratyush.com>
Show 23 quoted lines
> The commit message could be improved. It should first describe the > problem it is trying to solve, why it is worth solving, and then tell > the codebase to fix it. The details of how it is done can be learned > from the contents of the patch, so they are not as important. > > How about the message below? > > The colors of some ttext widgets are hard-coded. These hard-coded > colors are okay with a light theme but with a dark theme some widgets > are dark colored and the hard-coded ones are still light. This defeats > the purpose of applying the theme and makes the UI look very awkward. > > Remove the hard-coded colors in ttext calls and use colors from the > theme for those widgets via Text.Background and Text.Foreground from > the option database. > > Similarly, the highlighting for the currently selected file(s) in the > "Staged Files" and "Unstaged Files" sections is also hard-coded. Pull > the colors for that from the current theme to make sure it is in line > with the rest of the theme colors. > > No need to resend. I'll use this message when applying unless you have > any suggestions or objections.
I have no objections, please use this message.
I'll try to write better commit messages in the future.
-- Regards, Serg Tereshchenko