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 selected byte information #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

rbran
Copy link

@rbran rbran commented Nov 14, 2023

This add a section that display the current byte selected information into multiple formats:
screen

TODO list:

  • Include byte information.
  • Update byte information based selected byte on main screen.
  • Update byte information based selected byte on the Hex View.
  • Draw a indicator to what byte is selected

@sharkdp
Copy link
Owner

sharkdp commented Nov 29, 2023

Very cool! Thank you for your contribution.

I'll wait with my review until you have resolved your own to-do list, unless you think this should be integrated in the current state.

@rbran
Copy link
Author

rbran commented Nov 29, 2023

I'm unable to find an elegant solution for the missing two objectives, the current implementation uses 100% of CPU. I'll study the gui lib better and search for a better solution once I get more time.

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.

None yet

2 participants