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.

Sign up for GitHub

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

Jump to bottom

Add copy/paste feature to memory view #1084

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add copy/paste feature to memory view #1084

wants to merge 5 commits into from

Conversation

Vancleeff
Copy link
Contributor

Enhanced user experience by providing a way to copy and paste value directly in the memory view.

@Vancleeff
Add copy/paste feature to memory view
b81580b 
Enhanced user experience by providing a way to copy and paste value directly in the memory view.
@Vancleeff
Copy link
Contributor Author

Thanks for the review @Jamiras !
I commited these improvements !

@Vancleeff
Refactor copy/paste feature following review
3983514 
 - New HandleShortcut function implemented to ViewModel
 - Improve code to match repository naming convention
 - Prevent from using paste feature in the pointer finder
 - Using of ParseHex instead of iswxdigit and refactoring logic based on this change
@Vancleeff
Copy link
Contributor Author

Thanks @Jamiras, commited !

@Vancleeff
Copy link
Contributor Author

Fix CI checks

@Vancleeff Vancleeff force-pushed the feature/memview-copy-paste branch 2 times, most recently from db6e263 to 98602a9 Compare May 24, 2024 21:23
@Jamiras
Copy link
Member

Jamiras commented Jul 27, 2024

This has conflicts from the +/- PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jamiras Jamiras Jamiras left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Vancleeff @Jamiras