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

add toggleterm highlights and initial bufferline highlights #77

Merged
merged 3 commits into from
Apr 15, 2022

Conversation

mvllow
Copy link
Member

@mvllow mvllow commented Apr 14, 2022

closes #76

@mvllow
Copy link
Member Author

mvllow commented Apr 14, 2022

This adds a starting point for supporting plugins in which the user sets their own highlights. Included are bufferline and toggleterm:

use({
	'akinsho/bufferline.nvim',
	event = 'ColorScheme',
	config = function()
		local highlights = require('rose-pine.plugins.bufferline')
		require('bufferline').setup({
			highlights = highlights,
		})
	end,
})
use({
	'akinsho/toggleterm.nvim',
	event = 'ColorScheme',
	config = function()
		local highlights = require('rose-pine.plugins.toggleterm')
		require('toggleterm').setup({
			highlights = highlights,
		})
	end,
})

Notice that event = 'ColorScheme' is necessary when Rosé Pine uses the dawn variant.

For anyone wanting to contribute, please reach out here. Ideally we would have a PR per plugin – simply use this PR for reference 😌

@mvllow mvllow marked this pull request as ready for review April 15, 2022 13:08
@mvllow mvllow changed the title wip: add plugin highlights add toggleterm highlights and initial bufferline highlights Apr 15, 2022
@mvllow mvllow force-pushed the add-plugin-highlights branch from 8bb9c0d to ef47d4b Compare April 15, 2022 13:15
@mvllow mvllow merged commit 3f6586e into main Apr 15, 2022
@mvllow mvllow deleted the add-plugin-highlights branch April 15, 2022 13:16
@will
Copy link
Contributor

will commented Apr 15, 2022

Thanks!

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.

Support toggleterm highlights config
2 participants