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

It's somewhat difficult to use at smaller screen and window sizes #33

Open
cyrozap opened this issue Nov 8, 2021 · 1 comment
Open

Comments

@cyrozap
Copy link
Contributor

cyrozap commented Nov 8, 2021

First I'd like to say, thanks for making this tool! It's a lot more convenient than writing a bespoke Python script every time I want to visualize a new binary in a different way.

The only complaint I have about it right now is that it assumes the user has a fairly large monitor. My laptop's built-in screen is 1366x768, so not only does the window default to a size larger than my screen, I can't shrink it to fit my screen. This means that the lower part of the controls get cut off, so I can't access the endianness setting or any other settings below it without expanding the controls really far to the left. In addition, when viewing the controls, the left size of the binary view gets cut off, meaning I have to drag the window to the right in order to try and see that part of the binary. And finally, there's some weird part in the middle of the binary that appears "stretched" for some reason, with non-square pixels, though I'm not sure if this is related--it might be a Vulkan driver issue since Mesa's support for Vulkan on Bay Trail is incomplete and that's what I'm running this on.

1366x768 is a popular resolution for inexpensive laptops, some users set their screens to lower resolutions for accessibility reasons, and others may simply want to run two instances of binocle side-by-side, so it'd be nice if binocle supported smaller window sizes.

Here are some screenshots so you can see what I mean:



@sharkdp
Copy link
Owner

sharkdp commented Nov 9, 2021

Thank you very much for your feedback. I definitely agree that this should be supported. For now, the only way to fix this is to change the hard-coded HEIGHT/WIDTH settings, requiring a recompile.

I think this might be fairly easy to fix though by using something like min(WIDTH, actual_screen_width) for the window width (and similar for the height).

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

No branches or pull requests

2 participants