Skip to content

feat: support snacks picker highlights#341

Open
techwizrd wants to merge 1 commit into
dracula:masterfrom
techwizrd:snacks-picker-highlights
Open

feat: support snacks picker highlights#341
techwizrd wants to merge 1 commit into
dracula:masterfrom
techwizrd:snacks-picker-highlights

Conversation

@techwizrd
Copy link
Copy Markdown

@techwizrd techwizrd commented May 11, 2026

Summary

  • Add Dracula highlight links for folke/snacks.nvim picker UI groups.
  • Improve picker row selection, match text, file/path labels, and git status colors.
  • Keep the integration consistent by linking to existing Dracula highlight groups.

Verification

  • Loaded the colorscheme successfully in Neovim:
nvim --headless -u NONE +'set rtp^=.' +'set termguicolors' +'colorscheme dracula' +'quit'
  • Verified Snacks picker highlight links with a reproducible local config using folke/snacks.nvim and nvim-mini/mini.icons:
SnacksPickerListCursorLine -> DraculaSelection
SnacksPickerMatch -> DraculaCyan
SnacksPickerSelected -> DraculaPink
SnacksPickerFile -> DraculaFg
SnacksPickerDirectory -> Directory
SnacksPickerGitStatusModified -> DiagnosticWarn
  • Manually inspected the picker UI with icons enabled.
image

@benknoble
Copy link
Copy Markdown
Member

I don't have any objections to this implementation, though I may tweak the commit message when merging.

I don't use the relevant plugins, though, so feedback from @figmads is very welcome.

I wonder if the lua/ and doc/ that are made subtle in the screenshot are too low-contrast (esp. when the highlighting from the current selection is overlaid)? Idk.

@techwizrd
Copy link
Copy Markdown
Author

I can bump up the contrast and amend the commit message. It is a little terse. Do you have suggestions on which colors might be a little better fit?

@benknoble
Copy link
Copy Markdown
Member

I can bump up the contrast and amend the commit message. It is a little terse.

Looking back, I'm not sure what I meant about the commit. A few thoughts

  • any thing "non-standard" worth calling out ?
  • I prefer imperative Close <link> with a full URL.

Do you have suggestions on which colors might be a little better fit?

Honestly no. It may not be an issue. 🤷

Thanks!

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants