Opened 6 years ago

Closed 4 years ago

#10875 closed defect (fixed)

BACKENDS: WIN32 - In-build Add Game Screen doesn't work

Reported by: GermanTribun Owned by: SupSuper
Priority: normal Component: Port: Win32
Version: Keywords:
Cc: Game:

Description

Using the newest snapshot, when I try to add a new game, the GUI for selecting the directory with the game files doesn't render and it snaps back to Windows to open up a standard window from which I can choose properly. After that, things are back to normal.

This does not depend on the theme and looking at the program window shows garbage in it while the Windows GUI is open. I parked the snapshot in a different directory from my main SCUMMVM, so that it doesn't mix.

Attachments (1)

GUI Bug.png (48.1 KB ) - added by GermanTribun 6 years ago.

Download all attachments as: .zip

Change History (9)

by GermanTribun, 6 years ago

Attachment: GUI Bug.png added

comment:1 by digitall, 6 years ago

Component: GUIPort: Win32

This is not a bug. The ability to use the system native file chooser was added by and was made default true, which might have been a mistake given that this is not yet an option in the GUI to set this:
https://github.com/scummvm/scummvm/pull/1414

You can disable this by adding:
gui_browser_native=false

to the [scummvm] section of your scummvm.ini file...

comment:2 by digitall, 6 years ago

Owner: set to criezy

The bug really should be to set this to false by default, add an option in the GUI to change this and look at if we can stop the main scummvm window getting "garbage" as the rendering loop is suspended during the native dialog display.

Will assign to criezy who was going to look at this.

comment:3 by criezy, 6 years ago

I already added the option to the GUI (commits 319b90f and e7ba15f). Does it not show up on Windows? It should be in the Misc tab.

And it would indeed be better if the ScummVM window still rendered properly while the system directory chooser is shown (and by the way it does on macOS I believe).

comment:4 by digitall, 6 years ago

Ah, Mea Culpa. I hadn't realised that that feature was not compiled for my platform i.e. Linux x86_64 and thought that the GUI option for setting it was missing in all cases.

Will wait on the submitter giving feedback on if the option fixes things for them.

comment:5 by GermanTribun, 6 years ago

New build now has the option for native GUI instead of SCUMMVM GUI for file select. To make things less confusing, it should be turned off by default.

comment:6 by SupSuper, 6 years ago

Note that the native file picker was the default for OSX users, so you might need different defaults for different platforms.

comment:7 by raziel-, 4 years ago

Summary: In-build Add Game Screen doesn't workBACKENDS: WIN32 - In-build Add Game Screen doesn't work

comment:8 by SupSuper, 4 years ago

Owner: changed from criezy to SupSuper
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.