aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNathan Lilienthal <nathan@nixpulvis.com>2019-01-02 16:56:36 -0500
committerChristian Duerr <chrisduerr@users.noreply.github.com>2019-01-02 21:56:36 +0000
commit2d9afb9b395ea0cc71432613e0df104aefcf24c1 (patch)
treed4b7c14bab8d38c76a06c72e494fa41ab8a72e8d
parent2e968ae0add24519237b24a1ce8ed0d79f13e749 (diff)
downloadalacritty-2d9afb9b395ea0cc71432613e0df104aefcf24c1.tar.gz
alacritty-2d9afb9b395ea0cc71432613e0df104aefcf24c1.zip
Process Mouse Binding Actions Earlier
To allow for a mouse binding to properly copy text, we should process the mouse binding actions before thee built-in functionality, which will clear the selection on right click, for example.
-rw-r--r--CHANGELOG.md1
-rw-r--r--src/input.rs4
2 files changed, 3 insertions, 2 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md
index de808894..6a1d152b 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -27,6 +27,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Alt modifier is no longer sent separately from the modified key
- Various Windows issues, like color support and performance, through the new ConPTY
- Fixed rendering non default mouse cursors in terminal mouse mode (linux)
+- Fix the `Copy` `mouse_bindings` action ([#1963](https://github.com/jwilm/alacritty/issues/1963))
## Version 0.2.4
diff --git a/src/input.rs b/src/input.rs
index 23ccdd58..a4deed04 100644
--- a/src/input.rs
+++ b/src/input.rs
@@ -646,6 +646,8 @@ impl<'a, A: ActionContext + 'a> Processor<'a, A> {
MouseButton::Other(_) => None,
};
+ self.process_mouse_bindings(modifiers, button);
+
if let Some(prev_state) = prev_state {
if prev_state != state {
match state {
@@ -658,8 +660,6 @@ impl<'a, A: ActionContext + 'a> Processor<'a, A> {
if let ElementState::Released = state {
return;
}
-
- self.process_mouse_bindings(modifiers, button);
}
/// Process key input