modifier keys and batch operations...

[ghost]

so, i'm typing in lowercase right now because i discovered an extremely inconvenient problem with superpng and batch operations - shift/alt keys will open input options even when photoshop is not in focus/minimized/etc, and it does not respect the batch operation option to suppress file open dialogs.

as you might have guessed, i'm running a batch operation right now, one of several i'll need to do tonight

this means that holding down shift to select text in another program can trigger it, or alt tabbing, or, yes, holding shift to capitalize things.

i beg of you to fix this

[oomek]

I second that. Very annoying. I just go for a fag or a coffee when the batch is running ;)

[ghost]

thats a bad word and im very offend

[ghost]

@oomek Any chance you have a build environment that would work for compiling this? I'd love it if you could change it to require both Ctrl and Shift to be held down (and leaving alt out entirely).

It's not ideal but it's a decent enough stopgap measure.

[oomek]

Don't be. It's how we call ciggarettes here in the UK. But I digress. I've never compiled a Photoshop plugin, but I can try. I also requested one feature a while back (premultipied alpha option on the save dialog box) Maybe now is the time.

[ghost]

Don't be.

@oomek Don't worry, I was only joking, hence the poor grammar.

I'm looking at the code now, it seems I was mistaken about it accepting the Ctrl key.

SuperPNG/src/photoshop/win/SuperPNG_InUI_Win.cpp

Line 210 in 0de1e0e

bool shift_key = ( KeyIsDown(VK_LSHIFT) || KeyIsDown(VK_RSHIFT) || KeyIsDown(VK_LMENU) || KeyIsDown(VK_RMENU) );

So this line could just be changed to...
bool shift_key = ( KeyIsDown(VK_SHIFT) && KeyIsDown(VK_CONTROL) );

Or to be on the safe side, since there might be a reason he specified the left and right variants separately (Photoshop bug maybe?)...
bool shift_key = ( ( KeyIsDown(VK_LSHIFT) || KeyIsDown(VK_RSHIFT) ) && ( KeyIsDown(VK_LCONTROL) || KeyIsDown(VK_RCONTROL) ) );

[ghost]

@oomek On second thought, I practically never see alt + shift hotkeys in any program, so let's go with that instead.

bool shift_key = ( ( KeyIsDown(VK_LSHIFT) || KeyIsDown(VK_RSHIFT) ) && ( KeyIsDown(VK_LMENU) || KeyIsDown(VK_RMENU) ) );

[ghost]

@oomek Having read your feature request for premultipled alpha, I would VERY much like this too.

[oomek]

I'll give it a go tomorrow or during the weekend.

[oomek]

@Aidolii please open an issue on my fork https://github.com/oomek/SuperPNG
so we can discuss it further. I've added premultiply alpha to the save dialog, but I'm not sure about suppressing the hotkeys as it also affects the dialog not showing up when you normally open the file.

[ghost]

https://github.com/oomek/SuperPNG/releases