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: add rose-pine #219

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Theme: add rose-pine #219

merged 1 commit into from
Feb 13, 2024

Conversation

elijahimmer
Copy link
Contributor

Description

Added rose-pine as a theme.

I don't think I missed anything, but please tell me if I did.

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist

  • My code follows the style and contributing guidelines of this project.
  • I ran Alejandra to format my code (nix fmt).
  • I have performed a self-review of my own code and tested it.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generate no new warnings.
  • This change requires a documentation update.
  • I have updated the documentation accordingly.

Screenshots & Logs
neovim messages
logs
(the git tree was only dirty because I was testing the theme)
the theme
docs image
I couldn't figure out how to get the docs to update the styles list, but the themes work.

Simply added the rose-pine input and added it as a theme.
@NotAShelf
Copy link
Owner

NotAShelf commented Feb 13, 2024

Looks good! I'll review it in a bit, but probably a quick merge.

I couldn't figure out how to get the docs to update the styles list, but the themes work.

Did you build docs-html package locally?

Copy link
Owner

@NotAShelf NotAShelf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!


styles = {
bold = false,
italic = false, -- I would like to add more options for this
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you're more than welcome to do so! I can't think of a backwards-compatible way of, but we can introduce a freeform settings attribute to be shared scross modules so that they are able to freely define their own settings :)

@NotAShelf
Copy link
Owner

swappy-20240214_011443

Theme name properly visible in the latest doc build, merging. Thanks!

@NotAShelf NotAShelf merged commit c0841e0 into NotAShelf:main Feb 13, 2024
7 checks passed
bloxx12 pushed a commit to bloxx12/nvf that referenced this pull request Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants