-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
More screenshot options #601
Comments
👍 for camera click noise!!! |
Started to work on it :) |
Additional related ideas / discussion http://forum.movingblocks.net/threads/screenshot-enhancments.873/ @NowNewStart - if you've still got that working camera click noise and beginning options and can base them off the restructure branch (with nothing else in the same PR) ... gogogo! ;-) |
Downloading it right now, will start working on it when I fetched all files for it ;) |
I found something stupid as I played a normal Terasology build and wanted to take a screenshot, and also had the out of memory problem we talked about @Cervator. I think there are common problems with it. |
Well, yeah, the default screenshot size is mega-big. That's why we need the options to choose the size and default to a more manageable size :-) |
The memory allocatef by the standard exe is too low in the first instance. |
Bought a new PC. |
Closing as complete! Referenced the remaining minor WIBNIFs in the backlog for any future interest |
Thanks to @begla we have crazy detailed screenshots now :-)
Having 15MB screenshot files might call for some options though :D
I figure a little "Screenshot Settings" button under Graphic Settings would be appropriate. Something like this:
Multiple options can be enabled so users don't have to manually resize
Could default all sizes enabled unless that lags too much. Or just super
Bonus: Implement auto-upload to image hosting sites / social sites (should drop the upload process into a background thread / queue). Settings screen would be where you put the information of what site to use and any authentication needed (ideally none or a fancy OAuth type setup to not expose passwords).
If auto-upload is enabled make a set of radio buttons appear next to the size settings so the user can select which one to upload.
Double bonus: Add a camera click noise when taking screenshot
Edit way later: Confirmed that the out of memory issues I hit locally while attempting to take screenshots were due to the new crazy size. Switching to a 64-bit JDK fixed it and allows me to take screenies again, but we probably should default to lower quality so users using 32-bit won't crash + put a warning by the super sized option. Also about how long it can take to then generate said super screenies ;-)
The text was updated successfully, but these errors were encountered: