Fixes to prepare for v0.52.0#1054
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates sample configuration and documentation in preparation for the v0.52.0 release, including new word-wrap behavior and view-mode sidebar documentation.
Changes:
- Add
word_wrap_modekey binding (Alt+w) to the sample keybind configs (ov.yaml,ov-less.yaml). - Update
README.mdto document word wrap mode, add a “View Mode Sidebar” section, and standardize key-name capitalization in multiple places. - Add a new documentation screenshot (
docs/sidebar2.png) referenced from the README.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| ov.yaml | Adds word_wrap_mode key binding (Alt+w) to the default keybind list. |
| ov-less.yaml | Adds word_wrap_mode key binding (Alt+w) to the less-style keybind list. |
| README.md | Adds/updates sections for word wrap mode and view-mode sidebar; standardizes key formatting. |
| docs/sidebar2.png | New screenshot used by the README “View Mode Sidebar” section. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| * `-w` or `-w=char`(default): Wrap lines at screen width, breaking anywhere. | ||
| * `-w=word`: Wrap lines at screen width, breaking at word boundaries. | ||
| * `-w=none`: Disable line wrapping. |
There was a problem hiding this comment.
The newly added wrap mode list documents -w=none to disable wrapping, but the CLI parsing only treats values like false/no/0 as “wrap off” and word as word-wrap. Using none will be treated as an unrecognized value and end up enabling character wrap. Please update the docs to reflect the actual supported values (e.g. -w=false to disable, -w=char for character wrap, -w=word for word wrap).
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.