aboutsummaryrefslogtreecommitdiff
path: root/st.c
diff options
context:
space:
mode:
authornoname <noname@inventati.org>2015-05-03 19:28:10 +0000
committerRoberto E. Vargas Caballero <k0ga@shike2.com>2015-05-04 12:06:43 +0200
commitc990abfedf56cb8d3176fe6d5152ff65bb68bff0 (patch)
treebccbfccd50700d31c72501c9652c080c7aa190ed /st.c
parent3cb7f27afe89c33c74b51c5460b7fb16413f786b (diff)
downloadst-c990abfedf56cb8d3176fe6d5152ff65bb68bff0.tar.gz
st-c990abfedf56cb8d3176fe6d5152ff65bb68bff0.tar.bz2
Fix empty selection highlighting bug.
When user clicks LMB, one character is selected, but will not be copied to selection until the user moves cursor a bit. Therefore, the character should not be highlighted as selected yet. Before the patch, the trick was not to mark line as dirty to avoid highlighting it. However, if user has already selected something and clicks in line that contains selection, selclear sets the line as dirty and one character is highlighted when it should not. This patch replaces dirty trick with explicit check for sel.mode inside selected().
Diffstat (limited to 'st.c')
-rw-r--r--st.c12
1 files changed, 5 insertions, 7 deletions
diff --git a/st.c b/st.c
index cf6556a..3eb66fa 100644
--- a/st.c
+++ b/st.c
@@ -716,6 +716,9 @@ selnormalize(void) {
bool
selected(int x, int y) {
+ if(sel.mode == SEL_EMPTY)
+ return false;
+
if(sel.type == SEL_RECTANGULAR)
return BETWEEN(y, sel.nb.y, sel.ne.y)
&& BETWEEN(x, sel.nb.x, sel.ne.x);
@@ -921,14 +924,9 @@ bpress(XEvent *e) {
}
selnormalize();
- /*
- * Draw selection, unless it's regular and we don't want to
- * make clicks visible
- */
- if(sel.snap != 0) {
+ if(sel.snap != 0)
sel.mode = SEL_READY;
- tsetdirt(sel.nb.y, sel.ne.y);
- }
+ tsetdirt(sel.nb.y, sel.ne.y);
sel.tclick2 = sel.tclick1;
sel.tclick1 = now;
}