Skip to content

Comments

Allow theming the diff view's diagonal fill with diffEditor.diagonalFill#94780

Merged
alexdima merged 2 commits intomicrosoft:masterfrom
robertrossmann:feat/diff-diagonal-fill-colour
Apr 27, 2020
Merged

Allow theming the diff view's diagonal fill with diffEditor.diagonalFill#94780
alexdima merged 2 commits intomicrosoft:masterfrom
robertrossmann:feat/diff-diagonal-fill-colour

Conversation

@robertrossmann
Copy link
Contributor

Themes can now customise the colour and opacity of the diff editor's diagonal fill.

I have replaced the diagonal fill image (png) with a CSS version which allows us to customise the fill's colour. Then I added a new colour contribution point, diffEditor.diagonalFill with default values very closely matching the original look. The light themes, however, look considerably better with this new version (screenshots attached).

Benefits

  • we can style the stripes' colour
  • themes can provide their own colour
  • the stripes remain sharp even at high editor zoom levels (the image-based stripes start to be visibly blurred at zoom level 3-4).
  • Dunno, maybe it now requires less memory to render...? 🤷‍♂️

Screenshots

Click on a screenshot to see the full image.

old new diffEditor.diagonalFill
remedy dark
remedy dark old remedy dark new remedy dark custom
remedy bright
remedy light old remedy light new
Light+
light-plus-old light-plus-new
Abyss
abyss-dark-old abyss-dark-new
Kimbie Dark
kimbie-dark-old kimbie-dark-new
Solarized Dark
solarized-dark-old solarized-dark-new

@joaomoreno joaomoreno removed their assignment Apr 14, 2020
@robertrossmann
Copy link
Contributor Author

I just rebased this on top of current master. Is there anything I can do to get this merged in? 🙏 ❤️ @alexdima

Thanks in advance!

@alexdima
Copy link
Member

This is great! Thank you!

@alexdima alexdima added this to the April 2020 milestone Apr 27, 2020
@alexdima alexdima merged commit 982ce26 into microsoft:master Apr 27, 2020
@robertrossmann robertrossmann deleted the feat/diff-diagonal-fill-colour branch April 28, 2020 07:00
robertrossmann added a commit to robertrossmann/vscode-remedy that referenced this pull request Apr 28, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jun 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants