-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
Made preview display same format as output #112
base: main
Are you sure you want to change the base?
Conversation
Moved the lowercase hex from a bool to a output format and also made the preview match the output format, still need to make it fit in the box
Also, I am not used to contributing to projects with forks so I am not sure what I did but for some reason it has the commits from the previous PR I made, I don't think this affects anything though. Just let me know if its a problem. |
uhh, it has some stuff removed. |
Resolved the conflict, I removed the --lowercase-hex since it makes more sense to me for lowercase hex to be a format, let me know if I should undo this. |
it should be delegated to a separate MR as it's completely unrelated to this |
What I did:
Video of it working:
https://github.com/user-attachments/assets/8cb7b184-28c9-4d3c-8679-61744d1a523d
Please let me know if I need to do or change anything.