Skip to content

Conversation

@NobbZ
Copy link
Owner

@NobbZ NobbZ commented Dec 4, 2025

Summary by CodeRabbit

  • Style
    • Updated fold visual indicators with custom symbols for improved code navigation appearance and readability.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 4, 2025

Walkthrough

This change adds custom visual indicators for code folds in Neovim by configuring the fillchars option with custom glyphs for fold opening, closing, and separation. The modification is purely visual and does not alter folding behavior or existing configuration.

Changes

Cohort / File(s) Summary
Fold Visual Configuration
plugins/nobbz/lua/nobbz/ufo.lua
Adds vim.o.fillchars setting with custom fold visual glyphs (foldopen, foldclose, foldsep) and accompanying comment

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • set up UFO #41: Modifies the same UFO configuration file to adjust Neovim folding visuals and setup.

Poem

🐰 Fold the code, but make it bright,
Custom glyphs dance in the light,
Open, close, and lines divide,
Neovim folds with pretty pride!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'first attempt to nicer foldmarks' directly and clearly describes the main change: customizing fold visual indicators to be nicer in appearance.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch push-wqztqqvrqust

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the fold marks configuration in the nvim-ufo plugin setup by customizing the fold display characters to use Unicode box-drawing characters for a more visually appealing fold indicator presentation.

  • Adds custom fold character configuration using Unicode symbols
  • Documents a preferred alternative character for future consideration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

vim.o.foldenable = true

vim.o.fillchars = "foldopen:╭,foldclose:·,foldsep:│"
-- prefered "tail": ╰
Copy link

Copilot AI Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: "prefered" should be "preferred".

Suggested change
-- prefered "tail": ╰
-- preferred "tail": ╰

Copilot uses AI. Check for mistakes.
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
plugins/nobbz/lua/nobbz/ufo.lua (1)

6-8: Refine fillchars customization and fix tiny typo

Right now vim.o.fillchars is fully overwritten, which can unintentionally drop other fillchars you might have set elsewhere (e.g. eob, vert, diff). If you only want to tweak the fold-related characters, consider appending instead of replacing, and fix the spelling in the comment:

-vim.o.fillchars = "foldopen:╭,foldclose:·,foldsep:│"
--- prefered "tail": ╰
+vim.opt.fillchars:append({
+  foldopen = "╭",
+  foldclose = "·",
+  foldsep = "│",
+})
+-- preferred "tail": ╰

This keeps any existing non-fold fillchars while still giving you the custom fold markers you want, and cleans up the comment wording.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9dcd68b and c667fad.

📒 Files selected for processing (1)
  • plugins/nobbz/lua/nobbz/ufo.lua (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
plugins/**/*.lua

⚙️ CodeRabbit configuration file

plugins/**/*.lua: Please ensure all of the following:

  • Plugins are lazyloaded if possible through nobbz.lazy
  • local has to be used
  • coding follows the luarocks styleguide, but with 2 spaces indentation

Files:

  • plugins/nobbz/lua/nobbz/ufo.lua
🧠 Learnings (1)
📓 Common learnings
Learnt from: NobbZ
Repo: NobbZ/nobbz-vim PR: 0
File: :0-0
Timestamp: 2025-04-12T09:24:45.451Z
Learning: NobbZ uses a homebrewn shallow abstraction over `lz.n` rather than using lazy.nvim directly. References to Lazy.nvim's specific features should be avoided in reviews of NobbZ's repository.

@NobbZ NobbZ merged commit e4c09e5 into main Jan 7, 2026
9 checks passed
@NobbZ NobbZ deleted the push-wqztqqvrqust branch January 7, 2026 22:23
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.

2 participants