Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theme: Papercolor: Cleanup, linting and using inheritance #8276

Merged
merged 4 commits into from
Sep 13, 2023

Conversation

AfoHT
Copy link
Contributor

@AfoHT AfoHT commented Sep 13, 2023

Started out as just fixing the lints, ended up being a bit more substantial overhaul.

Primary changes:

  • Unify dark and light variants to share everything except palette
    • Enables using inherits feature
  • Extend original Papercolor with multi-cursor color variations
    • Add indication for primary cursor
    • Make cursor visible from selection

Minor changes:

  • Add more and align existing scopes with Vim plugin papercolor
  • Match UI elements closer to original

Light

Old

image

New

image

Vim

image

Dark

Old

image

New

image

Vim

image

For both light and dark Vim screenshots, the cursor should be filled in, just like in Helix. Due to loss of focus when taking screenshots we get the boxed in letter variant instead.

Regular Vim does not handle multiple selections,
add distinction between primary and secondary cursors.

Align dark and light variants, smaller cleanup
And cursorline/cursorcolumn
Align light and dark variants, use inherit feature
@the-mikedavis
Copy link
Member

\cc @uncomfyhalomacro what do you think of the changes?

@the-mikedavis the-mikedavis added A-theme Area: Theme and appearence related S-waiting-on-review Status: Awaiting review from a maintainer. labels Sep 13, 2023
@uncomfyhalomacro
Copy link
Contributor

@AfoHT Nice changes. I think this is good for merging @the-mikedavis 😄

@pascalkuthe pascalkuthe merged commit 764172d into helix-editor:master Sep 13, 2023
6 checks passed
@AfoHT AfoHT deleted the theme/papercolor/fix-lints branch September 15, 2023 21:03
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-theme Area: Theme and appearence related S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants