github-vscode-theme icon indicating copy to clipboard operation
github-vscode-theme copied to clipboard

Contrast issue with checkbox in merge editor

Open TimonVS opened this issue 3 years ago • 3 comments

I'm using the GitHub Dark theme and I was thoroughly confused when the new three-way merge editor showed up in VS Code, I was confused because I didn't see a checkbox next to the conflict to accept a change. A colleague pointed out there was a checkbox when I shared the screenshot with them asking what to do, but they had trouble spotting it too. This issue isn't present in the default dark theme. I also noticed that the other dark variations don't have this issue, it's just GitHub Dark.

CleanShot 2022-08-29 at 15 59 06@2x

By comparison, the default dark theme:

CleanShot 2022-08-29 at 16 01 58@2x

TimonVS avatar Aug 29 '22 14:08 TimonVS

I see now that GitHub Dark has been deprecated. So we should probably close this issue? Is it possible to notify people that this theme has been deprecated and probably won't be updated to take new features like the three-way merge editor into account?

TimonVS avatar Aug 29 '22 14:08 TimonVS

Is this also a problem with the checkboxes in the settings? Seems not?

Screen Shot 2023-01-03 at 17 20 08

So maybe the merge editor checkboxes are different?

simurai avatar Jan 03 '23 08:01 simurai

Tried to reproduce it by creating a merge conflict, but there doesn't seem to be any checkboxes. Even with the GitHub Dark Default theme.

Screen Shot 2023-01-03 at 17 36 27

Maybe they got replaced with text buttons (e.g. "Accept incoming") in the meantime? 🤔

simurai avatar Jan 03 '23 08:01 simurai