aboutsummaryrefslogtreecommitdiffstats
path: root/src/messageview.cpp
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-08-24 10:40:23 +0200
committerDanilo M. <danix@danix.xyz>2026-08-24 10:40:23 +0200
commitdf5f64f16359c9e34d258758f174187ffc618101 (patch)
tree85b1a3fd49ce84f47fb80218a47ff9a383ed67b8 /src/messageview.cpp
parentfabcf080652c6e5d57bf234be5e100769a9b965b (diff)
downloadqtmaildir-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/messageview.cpp')
0 files changed, 0 insertions, 0 deletions