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

Support for multiple value edits. #106

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

cristian64
Copy link
Collaborator

When multiple watch entries are selected, and an edit is applied to one of the values, the new value is set in all addresses (provided the watch entry has the same type).

To edit a value without losing the multi selection, F2 can be pressed while the focus is set on a value cell.

Fixes #48.

When multiple watch entries are selected, and an edit is applied to one
of the values, the new value is set in all addresses (provided the
watch entry has the same type).

To edit a value without losing the multi selection, `F2` can be pressed
while the focus is set on a value cell.

Fixes aldelaro5#48.
@dreamsyntax
Copy link
Collaborator

In a future PR if not this one,
I think making it a Context Right Click action too should be done so its exposed besides F2

Copy link
Collaborator

@dreamsyntax dreamsyntax left a comment

Choose a reason for hiding this comment

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

Tested, works good. I think we are good to merge unless you wanted to add the right click context menu addition

@cristian64
Copy link
Collaborator Author

cristian64 commented Mar 7, 2024

Tested, works good. I think we are good to merge unless you wanted to add the right click context menu addition

I'll look into it later. It can be a different PR. (It's certainly going to be a separate commit anyway.)

EDIT: Done in #108.

@dreamsyntax dreamsyntax merged commit b2b2074 into aldelaro5:master Mar 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Edit multiple values at once
2 participants