Skip to content

feat(config): add options visual appearance options#80

Merged
dlyongemallo merged 1 commit intomainfrom
add_visual_appearance_options
Apr 8, 2026
Merged

feat(config): add options visual appearance options#80
dlyongemallo merged 1 commit intomainfrom
add_visual_appearance_options

Conversation

@dlyongemallo
Copy link
Copy Markdown
Owner

Relates to #70.

Copilot AI review requested due to automatic review settings April 3, 2026 08:32
Copy link
Copy Markdown

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

Adds new configuration knobs to tune Diffview’s UI styling/visual noise, aligning the fork’s appearance closer to the original sindrets/diffview.nvim expectations (Issue #70).

Changes:

  • Add file_panel.tree_options.folder_count_style and file_panel.tree_options.folder_trailing_slash to control collapsed-folder counts and folder name formatting.
  • Add file_history_panel.subject_highlight to control ref-aware vs plain commit subject highlighting.
  • Document the new options in doc/diffview.txt and doc/diffview_defaults.txt.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
lua/diffview/scene/views/file_history/render.lua Gates commit subject highlight behavior behind new subject_highlight option.
lua/diffview/scene/views/diff/render.lua Adds new tree rendering options for folder trailing slash and count style.
lua/diffview/config.lua Introduces new default config values for the added UI options.
doc/diffview.txt Documents the new configuration options.
doc/diffview_defaults.txt Updates the default-config documentation output with the new options.

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

Comment thread lua/diffview/config.lua
Comment thread lua/diffview/scene/views/diff/render.lua
Comment thread lua/diffview/scene/views/file_history/render.lua
@dlyongemallo dlyongemallo force-pushed the add_visual_appearance_options branch 3 times, most recently from 8392730 to 60a196e Compare April 7, 2026 02:33
@dlyongemallo dlyongemallo requested a review from Copilot April 7, 2026 02:34
Copy link
Copy Markdown

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

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.


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

@dlyongemallo dlyongemallo force-pushed the add_visual_appearance_options branch from 60a196e to 7d40e20 Compare April 7, 2026 02:59
Copy link
Copy Markdown

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

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.


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

@dlyongemallo dlyongemallo merged commit 74424bf into main Apr 8, 2026
6 checks passed
@dlyongemallo dlyongemallo deleted the add_visual_appearance_options branch April 8, 2026 02:31
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