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

Add copy/paste feature to memory view #1084

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

Conversation

Vancleeff
Copy link
Contributor

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

Enhanced user experience by providing a way to copy and paste value directly in the memory view.
@Vancleeff Vancleeff force-pushed the feature/memview-copy-paste branch from 4c1b342 to b81580b Compare April 23, 2024 23:31
@Vancleeff
Copy link
Contributor Author

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

 - 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 Vancleeff force-pushed the feature/memview-copy-paste branch from 7d01aad to 3983514 Compare April 28, 2024 22:30
@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
@Vancleeff Vancleeff force-pushed the feature/memview-copy-paste branch from 98602a9 to 9a76604 Compare May 31, 2024 18:44
@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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants