Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix mouse mode bindings with multiple actions #7959

Merged
merged 1 commit into from May 16, 2024

Conversation

EBADBEEF
Copy link
Contributor

@EBADBEEF EBADBEEF commented May 7, 2024

The following config was broken:

  [mouse]
  bindings = [
  { mouse = "Right", mods = "Shift", action = "Copy"            },
  { mouse = "Right", mods = "Shift", action = "ClearSelection"  },
  ]

Only the first action was applied. Change to allow more than one exact match in mouse mode with shift held, but keep the logic to not allow fallback search if any exact match was found.

Fixes: 1a143d1 (Fix trigger of normal bindings in mouse mode)

Copy link
Member

@chrisduerr chrisduerr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a changlog entry?

The code looks good to me.

The following config was broken:
```
  [mouse]
  bindings = [
  { mouse = "Right", mods = "Shift", action = "Copy"            },
  { mouse = "Right", mods = "Shift", action = "ClearSelection"  },
  ]
```

Only the first action was applied. Change to allow more than one exact
match in mouse mode with shift held, but keep the logic to not allow
fallback search if any exact match was found.

Fixes: 1a143d1 (Fix trigger of normal bindings in mouse mode)
@chrisduerr chrisduerr merged commit 38fed9a into alacritty:master May 16, 2024
5 checks passed
@EBADBEEF EBADBEEF deleted the fix-mouse-mode-fallback branch May 17, 2024 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants