diff options
author | Nathan Lilienthal <nathan@nixpulvis.com> | 2019-01-02 16:56:36 -0500 |
---|---|---|
committer | Christian Duerr <chrisduerr@users.noreply.github.com> | 2019-01-02 21:56:36 +0000 |
commit | 2d9afb9b395ea0cc71432613e0df104aefcf24c1 (patch) | |
tree | d4b7c14bab8d38c76a06c72e494fa41ab8a72e8d | |
parent | 2e968ae0add24519237b24a1ce8ed0d79f13e749 (diff) | |
download | alacritty-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.md | 1 | ||||
-rw-r--r-- | src/input.rs | 4 |
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 |