Change Screenshot Functionality

[kphoenix137]

Removes:

• Red screen
• 300 ms pause
• Gamma adjustment from output png/pcx

Adds:

• Sound effect

[StephenCWills]

The other downside is that the chat message will linger on screen much longer than the red palette.

EDIT: The other other downside is that it may interfere when taking screenshots of conversations in chat.

EDIT2: And potentially exposing personal info in the screenshot.

[AJenbo]

Cutting it down to just the file name or relative path, or no file name is other options.

We could also use the info box that's also used for shrines and loading rather then the chat, that way it's not affecting the chat.

[kphoenix137]

Cutting it down to just the file name or relative path, or no file name is other options.

We could also use the info box that's also used for shrines and loading rather then the chat, that way it's not affecting the chat.

Absolutely. I think that DiabloMsg might not be a good idea because it's kind of annoying. I think anything new introduced text based to the user should be chat based rather than DiabloMsg for that reason.

[kphoenix137]

Removed the file path from the chat message

[StephenCWills]

I think that DiabloMsg might not be a good idea because it's kind of annoying.

Maybe so, but at least it can be dismissed. It does the player no good to keep the screenshot message hanging around in their chat log.

[kphoenix137]

I think that DiabloMsg might not be a good idea because it's kind of annoying.

Maybe so, but at least it can be dismissed. It does the player no good to keep the screenshot message hanging around in their chat log.

We could add a bool that prevents it from being added to the chat log (the one you get with the hotkey)

[FitzRoyX]

Instead of displaying text, you could play an unused sound effect like vtheft.wav

[kphoenix137]

Instead of displaying text, you could play an unused sound effect like vtheft.wav

Yeah we could do that.

[kphoenix137]

Updated

[StephenCWills]

There's a warning here says you're not using the palette variable anymore in CaptureScreen().

[StephenCWills]

Since we decided to remove the on-screen message, I was kind of hoping maybe we could use a camera shutter sound effect to make it more obvious. Here is one I found that sounds pretty good to me. It might work with a bit of editing. Released under the cc0 license so it's public domain.

https://freesound.org/people/mrrap4food/sounds/619062/