Re: b4 review available in master

Konstantin Ryabitsev
2026-03-20T19:37:15Z

Hi Louis,

Thanks for the thorough testing and the excellent feedback! I've committed a
number of fixes/changes in master that addresses a subset of them -- the
others will need to wait a bit, because I'm trying to get 0.15 out for wider
testing.

> When forcing drm-misc/drm-misc-next base, the check fails. I tried
> to force the acceptation (^y), but I got a git am issue during the
> branch creation

In theory, three-way merge (-3) is already enabled for all git-am operations
in the review TUI, so this should work in the current master. I don't know how
recent is the version you used for this, so hopefully the latest master is
already doing what you need. If you still see it fail, please send me the
exact error output and I'll take a look.

> - Add a way to see application logs (notably git errors)

For now, my recommendation is to press "s" to suspend the UI and drop to shell
-- you can review the b4 log output by scrolling up. Ctrl-D exits back into
the UI.

> - Copy don't work in the tui

This is a Textual framework limitation -- when mouse support is
enabled, the TUI captures mouse events for scrolling and clicking,
which prevents normal terminal text selection. Launch with
--no-mouse to get normal copy/paste, or make it permanent:

git config --global b4.review-no-mouse true

I've added a visible tip about this to the getting-started docs.

> - Add a "quick shortcut" to add trailers (t -> r/a/t/n -> enter
> instead of t -> arrows -> space -> enter)

Done! The trailer modal now accepts a/r/t/n as single-key toggles
for Acked-by, Reviewed-by, Tested-by, and NACKed-by.

> - Add a way to disable the scroll animation

The scroll animations come from Textual. You can disable them globally with an
environment variable:

export TEXTUAL_ANIMATIONS=none

or just smooth scrolling specifically:

export TEXTUAL_SMOOTH_SCROLL=0

I've also added a "Customising the TUI" section to the docs covering this and
other Textual tweaks (themes, colours, etc.).

> - Scroll using arrows/pgupdown/jk don't work in the "send email"
> confirmation list

Yeah, that was a bug. Fixed -- added j/k, arrows, space/backspace, and
pageup/pagedown bindings to the send confirmation modal.

> - Add a config to disable "autosnip"

This one is tricky -- there's a certain threshold where this more or less
becomes a "just open it and reply in your email client." :) I played a bit
with adding #!notrim and #!verbatim directives, but ended up with a bunch of
new bugs that I want to avoid in the roll-up to 0.15. So, I'm going to sit on
this for now and brainstorm about how and if we should implement such set of
features.

I've added the remaining items (auto-detect existing trailers, cancel
background processes on exit, batch trailer selection, keep trailers in-place
in reply editor) as planned for 0.16.

Thanks again!

Best,
--
KR