This breaks the still image used when the game is paused or behind the savestate select, see Window::updateFrame. The correct thing to do would be to call m_screenWidget->filter(false) in the constructor and Window::gameStopped. That said, that code should also be refactored out because it's common between the two functions. You'll also need to gate the m_screenWidget->filter(value.toBool()); line in setupMenu behind an if (m_controller) check like the ones above it.

Ah, thanks! Would have been too easy :D

EDIT: I tried to find out what exactly breaks so I have something to compare. At least the still image on pausing seems to work fine - could you give me another hint what's broken in this case?

Anything that calls Window::updateFrame expects that code you changed to have the same filtering as the game. It might only be noticeable if you use the Software display driver. Not sure.

Are you still looking into this? I can probably fix it myself if you're having trouble reproducing the case I described.

I was pretty busy in the past days, so this issue was a bit 'on hold' - however, I'd like to look at it in the next couple of days, so I can start gaining some experience :-)

Any update?

Sorry, I'm afraid I won't be able to take a look at this issue soon :/

Do you mind if I just fix it on my end then? I'd like to include a fix for this in 0.8.4, since it should be a safe fix, and I'd like to release that soon.

It wouldn't make any sense to hold back a release just waiting for me trying to fix stuff while it seems that you already know the solution to the problem, so please, go ahead 😃

It wouldn't be holding back the release, it would be skipping fixing it for the release, but yeah I'll go ahead and do that soon then. Thanks.