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

Fix transparent sticky scroll background in the code peek panel

Open nNoidea opened this issue 2 years ago • 2 comments

peekViewEditorStickyScroll.background uses the color of peekViewEditor.background and is also on top of it. peekViewEditor.background is an rgba color, which means that the sticky scroll features also becomes transparent and the text becomes unreadable when another text is underneath it.

peekViewEditor.background color seems like a normal rgb (without the alpha channel) color, but that's because the peek panel pushes away all the text underneath itself. The resulting color is the mix of peekViewEditor.background rgba and peekViewResult.background rgb values.

Since the sticky scroll feature always uses the color of the background of the panel it is in, this fix will take peekViewEditor.background and peekViewResult.background and mix them together into solid color and assign that color to the peekViewEditorStickyScroll.background to fix the issue.

Closes #361

Screenshots

Please provide before/after screenshots for any visual changes Before: image After: image Before: image After: image

Merge checklist

  • [x] Tested in GitHub Light Default theme

Take a look at the Contribute section for more information on how test your changes locally.

nNoidea avatar Apr 21 '23 02:04 nNoidea

🦋 Changeset detected

Latest commit: 09d42d0bc129b9c7ddad43de5ba9c79d5ec365c2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
github-vscode-theme Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Apr 21 '23 02:04 changeset-bot[bot]

Did I mess up the PR progress? The problem is still in the latest version.

nNoidea avatar Sep 19 '23 20:09 nNoidea