Thanks to visit codestin.com
Credit goes to github.com

Skip to content

refactor(tui): share hard-wrap helper across renderers#1671

Open
Aitensa wants to merge 1 commit into
Hmbown:mainfrom
Aitensa:issue-1622-wrap-helper-dedup
Open

refactor(tui): share hard-wrap helper across renderers#1671
Aitensa wants to merge 1 commit into
Hmbown:mainfrom
Aitensa:issue-1622-wrap-helper-dedup

Conversation

@Aitensa
Copy link
Copy Markdown
Contributor

@Aitensa Aitensa commented May 15, 2026

Summary

  • move the character-level hard-wrap helper into crates/tui/src/tui/ui_text.rs
  • reuse that shared helper from markdown_render, diff_render, and pager
  • remove duplicated implementations while keeping existing wrapping behavior and regression coverage intact

Why

A follow-up to the merged CJK wrapping fix in #1622. This addresses review feedback about duplicated hard-wrap helpers across TUI renderers without broadening the original bug-fix scope.

Validation

  • cargo fmt --all --check
  • cargo test -p deepseek-tui --bin deepseek-tui wrap_text_breaks_overlong_cjk_runs --locked
  • cargo test -p deepseek-tui --bin deepseek-tui render_diff_prepends_summary_and_gutter_markers --locked
  • cargo test -p deepseek-tui --bin deepseek-tui footer_hint_is_rendered_in_buffer --locked

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

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