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

fix: mini.statusline colors, cursorcolumn and signcolumn linking with cursorline and cursorlinenr #544

Closed
wants to merge 2 commits into from

Conversation

231tr0n
Copy link
Contributor

@231tr0n 231tr0n commented Jun 23, 2024

Changes for ministatusline and linking cursorcolumn and signcolumn to LineNr

on_highlights = function(highlights, colors)
        -- Ministatusline changes
        highlights.MiniStatuslineDevinfo = { fg = colors.fg_dark, bg = colors.fg_gutter }
	highlights.MiniStatuslineFileinfo = { fg = colors.fg_dark, bg = colors.fg_gutter }
	highlights.MiniStatuslineFilename = { fg = colors.fg_dark, bg = colors.bg_highlight }
        -- cursorcolumn and signcolumn linking to lineNr
	highlights.SignColumn = { link = "LineNr" }
	highlights.FoldColumn = { link = "LineNr" }
        highlights.CursorLineFold = { link = "CursorLineNr" }
        highlights.CursorLineSign = { link = "CursorLineNr" }
end

MiniStatusline

Without the Ministatusline changes the statusline looks something like this
image
With the changes of Ministatusline it looks something like this
image

SignColumn and FoldColumn changes

Without the signcolumn and foldcolumn changes the column looks something like this
image
With the signcolumn and foldcolumn changes the column looks something like this
image

@231tr0n
Copy link
Contributor Author

231tr0n commented Jun 30, 2024

@folke, sorry for this ping and hope I am not being rude but I wanted to know if there was any chance of this PR getting merged since this fixes issues with sign and fold column highlighting and mini.statusline.

@231tr0n 231tr0n closed this Jul 2, 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.

1 participant