diff options
| author | Danilo M. <danix@danix.xyz> | 2026-08-24 10:40:23 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-08-24 10:40:23 +0200 |
| commit | df5f64f16359c9e34d258758f174187ffc618101 (patch) | |
| tree | 85b1a3fd49ce84f47fb80218a47ff9a383ed67b8 /src/mainwindow.h | |
| parent | fabcf080652c6e5d57bf234be5e100769a9b965b (diff) | |
| download | qtmaildir-df5f64f16359c9e34d258758f174187ffc618101.tar.gz qtmaildir-df5f64f16359c9e34d258758f174187ffc618101.zip | |
fix(compose): put the reply cursor on blank space, not on the quote
quote_position names where the QUOTE goes, so the reply belongs on the
other side of it and the cursor has to follow the reply rather than the
buffer. seedBody() moved the cursor to Start under both positions, which
is correct only for Below: under Above, the shipped default, it landed on
the "On ... wrote:" attribution line, so every reply had to have room made
before it could be typed.
End under Above, Start under Below.
The existing theQuotePositionDecidesWhereTheQuoteLands passed throughout
the defect and still does, because the quote was in the right place all
along; only the cursor was not. The new test asserts the cursor's block is
blank, and that typing lands on the correct side of the quote, so a fix
that freed the cursor by inverting the position would not satisfy it.
Also re-measures item 136 in the backlog, which is not the intermittent
race it was filed as: undoMovesTheMessageBack fails 6 runs in 6 when named
alone and passes in the full suite, on a clean tree, so it depends on the
tests before it.
Diffstat (limited to 'src/mainwindow.h')
0 files changed, 0 insertions, 0 deletions
