Skip to content

feat(extra): add active buffer styling for helix #694

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

archcorsair
Copy link

@archcorsair archcorsair commented Mar 4, 2025

Description

The helix themes currently lack styling for helix bufferline active tabs, which makes it more difficult to see which current buffer tab is active at a glance.

This PR suggests colors for the tab using the current theme's color pallet.

Screenshots

Day

image

Moon

image

Night

image

Storm

image

@archcorsair archcorsair changed the title feat: Add active buffer styling for helix feat(extra): Add active buffer styling for helix Mar 4, 2025
@archcorsair archcorsair changed the title feat(extra): Add active buffer styling for helix feat(extra): add active buffer styling for helix Mar 4, 2025
@warmwaffles
Copy link

I think you have to add this to https://github.com/folke/tokyonight.nvim/blob/main/lua/tokyonight/extra/helix.lua because those extras are generated.

@archcorsair archcorsair force-pushed the feat/helix-bufferline-active-styles branch from cb63922 to b617df7 Compare March 15, 2025 06:58
@archcorsair
Copy link
Author

archcorsair commented Mar 15, 2025

I think you have to add this to main/lua/tokyonight/extra/helix.lua because those extras are generated.

Hey, you're awesome and I totally missed that. Fixed now.

PS: Looks like I shouldn't commit the generated files since those get generated by CI. Fixed that too.

@archcorsair archcorsair force-pushed the feat/helix-bufferline-active-styles branch from b617df7 to f2336e9 Compare March 15, 2025 07:23
@archcorsair archcorsair force-pushed the feat/helix-bufferline-active-styles branch from f2336e9 to 91d4e4e Compare March 15, 2025 07:25
@warmwaffles
Copy link

warmwaffles commented Mar 17, 2025

I only noticed this because I have been trying to figure out why the ruler is rendered incorrectly (not because of your pr)

@archcorsair
Copy link
Author

archcorsair commented Mar 22, 2025

I only noticed this because I have been trying to figure out why the ruler is rendered incorrectly (not because of your pr)

I've noticed this too. Does the cursor also get hidden by the indent guide for you?

@warmwaffles
Copy link

It does. Something is wrong with this theme that is causing that. The themes that ship with helix don't suffer from these issues.

20250313-115121

Copy link
Contributor

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label Apr 22, 2025
@archcorsair
Copy link
Author

Bumping to remove stale tag

@github-actions github-actions bot removed the stale label Apr 23, 2025
Copy link
Contributor

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label May 23, 2025
@archcorsair
Copy link
Author

Bumping to remove stale tag

@warmwaffles
Copy link

@github-actions github-actions bot removed the stale label May 24, 2025
@archcorsair
Copy link
Author

warmwaffles

Thanks for letting me know! Looks like it's still missing bufferline styling though.

@warmwaffles
Copy link

@archcorsair try pushing it up stream, I'm sure other Helix users would like it. I know I would 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants