Skip to content

Conversation

Ircama
Copy link
Contributor

@Ircama Ircama commented Dec 27, 2022

Fix a number of hex editor issues and add some keyboard management:

  • fix crash when selecting or extending selection before the beginning of the hex editor using the shift LEFT and UP arrow keys
  • Add Del key to remove a single byte and Ins key to add a single byte
  • Appropriately position the cursor after group deletion
  • Add basic BACK and DELETE key in hex cell editor
  • Add basic arrow keys in hex cell editor doing the same as Escape
  • Change "len" variable [which was the same as len()] to "length"
  • Catch possible exception on int(value, 16)

- fix crash when selecting or extending selection before the beginning of
  the hex editor using the shift LEFT and UP arrow keys
- Add Del key to remove a single byte and Ins key to add a single byte
- Appropriately position the cursor after group deletion
- Add basic BACK and DELETE key in hex cell editor
- Add basic arrow keys in hex cell editor doing the same as Escape
- Change "len" variable [which was the same as len()] to "length"
- Catch possible exception on int(value, 16)
timrid added a commit that referenced this pull request Jan 2, 2023
…of the hex editor using the shift LEFT and UP arrow keys (see PR #20)
timrid added a commit that referenced this pull request Jan 2, 2023
@timrid
Copy link
Owner

timrid commented Jan 3, 2023

Thanks for your contribution. I will merge this PR. But I am also working on an different branch where I refactored many files in this repository. But I applied your changes to the other branch manually, with a few modifications. The next release will be based on the refactored branch.

@timrid timrid merged commit eda474e into timrid:main Jan 3, 2023
@timrid timrid mentioned this pull request Jan 3, 2023
@Ircama Ircama deleted the ircama-fix-null-select branch January 8, 2023 15:55
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