-
Notifications
You must be signed in to change notification settings - Fork 14
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
fix: issues with windows displaying images #32
Conversation
Btw, I found |
Hey I find it really cool that you published a crate to solve this, if it doesn't take you too much time use winsafe, also if you need to call unsafe functions just remove the |
Done, I had to, sadly, remove the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks this is going in the right direction, it must be a nightmare to work with the windows apis and their massively undocumentend crates so I appreciate your willingness to help. hopefully in the future bill gates adds the getConsoleFontSize idk
Not adding this function, but making it works 🤷♂️ . But the issue is: instead of using actually terminals/consoles nowadays, they start an app that emulate a terminal, so the app never get access to the font size of what it's emulating (yes that's dumb asf, don't ask me why) |
…till have the app running
reminder to automate formatting process and run `cargo fmt --check` only once in workflows
I love open source, I had an issue with something, just went, fixed it, proposed the changes, and now it'll be fixed for everyone 🥳 |
This PR is intended to fix #26. I wouldn't call it a complete fix (it's still a work in progress), but at least it makes images viewable on Windows, they aren't too small. I'll still continue working on it in the future to try to get a perfect fix tho.