1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
|
From 3b31919427baf3da1ab1a24f682a4cefc7e5106f Mon Sep 17 00:00:00 2001
From: Christoph Cullmann <cullmann@kde.org>
Date: Sun, 16 Jan 2022 18:29:57 +0100
Subject: [PATCH] Revert "Do not cancel mouse selection when using the
keyboard"
This reverts commit 5dce70a9693f8cce37f733b4687bb9b303376742.
=> this leads to regressions
if somebody has time to properly reproduce and fix that, we can reapply
this, otherwise it is better to have some always working state without
this small usability improvement
BUG: 447525
BUG: 446189
frameworks/ktexteditor#11
---
src/view/kateview.cpp | 20 ++++++++++----------
src/view/kateviewinternal.cpp | 5 -----
src/view/kateviewinternal.h | 2 --
3 files changed, 10 insertions(+), 17 deletions(-)
diff --git a/src/view/kateview.cpp b/src/view/kateview.cpp
index 6b88eb75..56348480 100644
--- a/src/view/kateview.cpp
+++ b/src/view/kateview.cpp
@@ -3063,9 +3063,9 @@ void KTextEditor::ViewPrivate::shiftCursorRight()
void KTextEditor::ViewPrivate::wordLeft()
{
if (currentTextLine().isRightToLeft()) {
- m_viewInternal->wordNext(m_viewInternal->isUserSelecting());
+ m_viewInternal->wordNext();
} else {
- m_viewInternal->wordPrev(m_viewInternal->isUserSelecting());
+ m_viewInternal->wordPrev();
}
}
@@ -3081,9 +3081,9 @@ void KTextEditor::ViewPrivate::shiftWordLeft()
void KTextEditor::ViewPrivate::wordRight()
{
if (currentTextLine().isRightToLeft()) {
- m_viewInternal->wordPrev(m_viewInternal->isUserSelecting());
+ m_viewInternal->wordPrev();
} else {
- m_viewInternal->wordNext(m_viewInternal->isUserSelecting());
+ m_viewInternal->wordNext();
}
}
@@ -3098,7 +3098,7 @@ void KTextEditor::ViewPrivate::shiftWordRight()
void KTextEditor::ViewPrivate::home()
{
- m_viewInternal->home(m_viewInternal->isUserSelecting());
+ m_viewInternal->home();
}
void KTextEditor::ViewPrivate::shiftHome()
@@ -3108,7 +3108,7 @@ void KTextEditor::ViewPrivate::shiftHome()
void KTextEditor::ViewPrivate::end()
{
- m_viewInternal->end(m_viewInternal->isUserSelecting());
+ m_viewInternal->end();
}
void KTextEditor::ViewPrivate::shiftEnd()
@@ -3118,7 +3118,7 @@ void KTextEditor::ViewPrivate::shiftEnd()
void KTextEditor::ViewPrivate::up()
{
- m_viewInternal->cursorUp(m_viewInternal->isUserSelecting());
+ m_viewInternal->cursorUp();
}
void KTextEditor::ViewPrivate::shiftUp()
@@ -3128,7 +3128,7 @@ void KTextEditor::ViewPrivate::shiftUp()
void KTextEditor::ViewPrivate::down()
{
- m_viewInternal->cursorDown(m_viewInternal->isUserSelecting());
+ m_viewInternal->cursorDown();
}
void KTextEditor::ViewPrivate::shiftDown()
@@ -3168,7 +3168,7 @@ void KTextEditor::ViewPrivate::shiftBottomOfView()
void KTextEditor::ViewPrivate::pageUp()
{
- m_viewInternal->pageUp(m_viewInternal->isUserSelecting());
+ m_viewInternal->pageUp();
}
void KTextEditor::ViewPrivate::shiftPageUp()
@@ -3178,7 +3178,7 @@ void KTextEditor::ViewPrivate::shiftPageUp()
void KTextEditor::ViewPrivate::pageDown()
{
- m_viewInternal->pageDown(m_viewInternal->isUserSelecting());
+ m_viewInternal->pageDown();
}
void KTextEditor::ViewPrivate::shiftPageDown()
diff --git a/src/view/kateviewinternal.cpp b/src/view/kateviewinternal.cpp
index d642938a..c8d479d6 100644
--- a/src/view/kateviewinternal.cpp
+++ b/src/view/kateviewinternal.cpp
@@ -4077,11 +4077,6 @@ void KateViewInternal::viewSelectionChanged()
m_selectionCached.setStart(KTextEditor::Cursor::invalid());
}
-bool KateViewInternal::isUserSelecting() const
-{
- return m_selChangedByUser;
-}
-
KateLayoutCache *KateViewInternal::cache() const
{
return m_layoutCache;
diff --git a/src/view/kateviewinternal.h b/src/view/kateviewinternal.h
index 66b137b6..8639dcef 100644
--- a/src/view/kateviewinternal.h
+++ b/src/view/kateviewinternal.h
@@ -228,8 +228,6 @@ public:
return m_leftBorder;
}
- bool isUserSelecting() const;
-
// EVENT HANDLING STUFF - IMPORTANT
private:
void fixDropEvent(QDropEvent *event);
--
GitLab
|
