Skip to content

Conversation

@JamyGolden
Copy link

General

This is a continuation of #755. Thanks for the comments on that PR. Addressing some of the things mentioned there:

As far as I remember the current plugin doesn't actually support Base24 themes (which is a good improvement over Base16) but I am not a huge fan of adding more Vimscript plugins.

@NotAShelf I've created and stabalised tinted-nvim where we have 2 active maintainers.

What if the user sets a style that is not actually available? ? (basically hasAttr) is not type-safe either. I'm a little conflicted on adding this. Could there be a better plugin or/and a solution?

@NotAShelf in this version the styles aren't dynamic. So when new schemes come along we can just update the list manually.

About the PR

I've added tinted-nvim support for lualine. Info about that here:

All that needs to be done is to set the lualine theme value to theme = "tinted" with the tinted-nvim plugin loaded and it will use the theme set by tinted-nvim. So to get this working with nvf:

...
    statusline = {
      lualine = {
        enable = true;
        theme = "tinted";
      };
    };

    theme = {
      enable = true;
      name = "tinted-nvim";
      style = "base24-dracula";
    };
...
  • I don't know how to add anything to the manual. I searched for base16 but didn't find anything relevant in the project. nix run .#docs-html -Lv gives me error: unable to execute '/nix/store/6ny8kfmppna5f1iniw27hk0kjhkvlvqm-nvf-manual/bin/nvf-manual': No such file or directory (If this is related to the manual)
  • There are breaking changes here since vim.theme.base16-colors doesn't exist anymore. I'm not sure how to make a migration guide

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • [] I have added a section in the manual
    • [] (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

@github-actions
Copy link

github-actions bot commented Aug 9, 2025

🚀 Live preview deployed from 4b904de

View it here:

Debug Information

Triggered by: JamyGolden

HEAD at: main

Reruns: 1539

@JamyGolden JamyGolden marked this pull request as ready for review August 23, 2025 08:27
@JamyGolden
Copy link
Author

@NotAShelf ping

github-actions bot pushed a commit that referenced this pull request Oct 8, 2025
@JamyGolden JamyGolden changed the title theme: Add tinted-vim plugin theming support [skip ci] theme: Add tinted-vim plugin theming support Oct 8, 2025
@NotAShelf
Copy link
Owner

Hi, it seems that I've formulated a reply and forgot to send again. Sorry for the delayed response!

My plans for this PR is to hold it until we merge 0.8, and merge it alongside a planned theming refactor. I think the theming module is currently bearing a lot of technical debt, so it will be easier to refactor it and then add more code on it. Thank you for your work so far, but I'm going to have to ask you to be patient for a little more :)

@JamyGolden
Copy link
Author

Ok thanks, no problem :) I don't mind waiting at all. I'll update once 0.8 is merged.

@NotAShelf
Copy link
Owner

Good news, I've finally completed the theming refactor. It should be up as a PR (to v0.8 branch)) by the end of this week. I would appreciate it if you could refactor PR and consider provider feedback on the new API for theming plugins :)

github-actions bot pushed a commit that referenced this pull request Oct 31, 2025
Tinted Theming is the community successor to base16 and now supports
multiple scheme systems, including base24
@JamyGolden JamyGolden changed the base branch from main to v0.8 November 1, 2025 21:21
@JamyGolden
Copy link
Author

@NotAShelf that's great with the v0.8 branch! Which API for theming plugins has changed? I've looked through ./modules/plugins/theme/* but things seem the same as they were previously, so I'm probably looking in the wrong place.

What is the process on updating documentation in the gh-pages for the new API? (I haven't been able to find this info in the documentation)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants