From 8aa1cae911d5323f2e435c37ef5132c90f16ed55 Mon Sep 17 00:00:00 2001 From: Neil Date: Fri, 21 Jun 2019 07:16:45 +1000 Subject: Bug [#2106]. Place line caret in correct location for line caret with block OVR. That is for CARETSTYLE_LINE | CARETSTYLE_BLOCK mode. --- src/EditView.cxx | 4 +++- src/ViewStyle.cxx | 6 ++++++ src/ViewStyle.h | 1 + 3 files changed, 10 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/EditView.cxx b/src/EditView.cxx index f3185ecc8..887e7754e 100644 --- a/src/EditView.cxx +++ b/src/EditView.cxx @@ -1424,7 +1424,9 @@ void EditView::DrawCarets(Surface *surface, const EditModel &model, const ViewSt for (size_t r = 0; (r model.sel.Range(r).anchor) { + if ((vsDraw.DrawCaretInsideSelection(model.inOverstrike, imeCaretBlockOverride)) && + !drawDrag && + posCaret > model.sel.Range(r).anchor) { if (posCaret.VirtualSpace() > 0) posCaret.SetVirtualSpace(posCaret.VirtualSpace() - 1); else diff --git a/src/ViewStyle.cxx b/src/ViewStyle.cxx index 6b24e15fb..65c4f2f8e 100644 --- a/src/ViewStyle.cxx +++ b/src/ViewStyle.cxx @@ -549,6 +549,12 @@ bool ViewStyle::IsBlockCaretStyle() const noexcept { return (caretStyle == CARETSTYLE_BLOCK) || (caretStyle & CARETSTYLE_OVERSTRIKE_BLOCK) != 0; } +bool ViewStyle::DrawCaretInsideSelection(bool inOverstrike, bool imeCaretBlockOverride) const noexcept { + return ((caretStyle & CARETSTYLE_INS_MASK) == CARETSTYLE_BLOCK) || + (inOverstrike && (caretStyle & CARETSTYLE_OVERSTRIKE_BLOCK) != 0) || + imeCaretBlockOverride; +} + ViewStyle::CaretShape ViewStyle::CaretShapeForMode(bool inOverstrike) const noexcept { if (inOverstrike) { return (caretStyle & CARETSTYLE_OVERSTRIKE_BLOCK) ? CaretShape::block : CaretShape::bar; diff --git a/src/ViewStyle.h b/src/ViewStyle.h index 587eb976a..0235b3651 100644 --- a/src/ViewStyle.h +++ b/src/ViewStyle.h @@ -201,6 +201,7 @@ public: enum class CaretShape { invisible, line, block, bar }; bool IsBlockCaretStyle() const noexcept; + bool DrawCaretInsideSelection(bool inOverstrike, bool imeCaretBlockOverride) const noexcept; CaretShape CaretShapeForMode(bool inOverstrike) const noexcept; private: -- cgit v1.2.3