Opened 9 months ago

Closed 8 months ago

#15868 closed defect (fixed)

SCUMM: MI1VGA: Load button doesn't work on default slot in original GUI

Reported by: eriktorbjorn Owned by: athrxx
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game: Monkey Island 1

Description (last modified by eriktorbjorn)

In the floppy version of MI1 (VGA), and perhaps some other games that I don't have, if you bring up the original Load dialog the first save slot will be pre-selected, like this:


But if you press the Load button, nothing happens. The button gets highlighted, and that's it:


To make it work, you have to click on the already selected save slot. When I tried to debug it, I got the impression that internal selected slot when the dialog is opened is 0, but when you click on the first slot it's 1?

Attachments (2)

scummvm-monkey-vga-1-00002.png (13.4 KB ) - added by eriktorbjorn 9 months ago.
scummvm-monkey-vga-1-00003.png (13.4 KB ) - added by eriktorbjorn 9 months ago.

Download all attachments as: .zip

Change History (10)

by eriktorbjorn, 9 months ago

by eriktorbjorn, 9 months ago

comment:1 by eriktorbjorn, 9 months ago

Description: modified (diff)

comment:2 by athrxx, 9 months ago

I have made a little fix for this which I tested on MI1 EGA. Could you please check if it works with the VGA version, too?

comment:3 by eriktorbjorn, 9 months ago

As far as I can tell, everything works like it should with the VGA floppy version.

comment:4 by eriktorbjorn, 9 months ago

Are there any other games affected, ir can this be closed?

comment:5 by athrxx, 9 months ago

It is limited to SCUMM4 non-Loom games, so that should really just be MI1 EGA and VGA, right?

in reply to:  5 comment:6 by eriktorbjorn, 9 months ago

Replying to athrxx:

It is limited to SCUMM4 non-Loom games, so that should really just be MI1 EGA and VGA, right?

Looks like it to me. Except for the demos. You can save in the "Passport To Adventure" demo, but I just tried it and it worked fine as far as I could tell.

comment:7 by eriktorbjorn, 8 months ago

Any reason not to close this as fixed?

comment:8 by athrxx, 8 months ago

Owner: set to athrxx
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.