-
Notifications
You must be signed in to change notification settings - Fork 12
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
Memory Table Editability #110
base: main
Are you sure you want to change the base?
Memory Table Editability #110
Conversation
…esn't send an memory event
TODO: Set up state setting
3b51517
to
61a7599
Compare
Thanks, @colin-grant-work ! |
Actually did a little testing now.... :-) |
Yes, because this allows editing that spans groups, which is the unit at which we apply endianness currently, it could end up behaving a bit strangely there.
This one addresses the early separation of data and UI, so basically the feedback on the |
Setting as draft for now pending reconciliation with recently merged code. |
What it does
An alternative implementation of #108 that allows both editing by group (by double clicking on the group) and editing of sub- and supra-group editing by selection and context menu.
How to test
Edit currently selected memory
Review checklist
Reminder for reviewers