Opened 22 months ago

Closed 16 months ago

Last modified 16 months ago

#13096 closed defect (fixed)

GUI: Wrong list items are disabled in the SCUMM launcher

Reported by: eriktorbjorn Owned by: sev-
Priority: high Component: GUI
Version: Keywords:
Cc: Game:

Description

If a game has been added to ScummVM and the directory for it no longer exists, it's supposed to be grayed out in the ScummVM launcher.

However, the colour and the list entry are apparently kept in different data structures, and now group headers apparently throw off the numbering of the entries.

E.g. I had moved a game to a different directory. ScummVM grayed out the game that was listed above it, presumably because of some (invisible?) main header.

If I changed the grouping so that it added a multitude of headers, the grayed out line was way out of place.

Also, applying a search filter seems to remove the graying completely?

In the attached screenshots, only "[LucasArts] Maniac Mansion (GOG Broken Version)" is supposed to be grayed out.

(Don't mind that the search box looks ugly. I usually run with a 4x scaler, where I can safely use large text. I changed to 2x to keep down the size of the screenshots, and didn't bother to make the text smaller.)

Attachments (3)

scummvm-00000.png (64.8 KB ) - added by eriktorbjorn 22 months ago.
scummvm-00001.png (53.9 KB ) - added by eriktorbjorn 22 months ago.
scummvm-00002.png (29.9 KB ) - added by eriktorbjorn 22 months ago.

Download all attachments as: .zip

Change History (8)

by eriktorbjorn, 22 months ago

Attachment: scummvm-00000.png added

by eriktorbjorn, 22 months ago

Attachment: scummvm-00001.png added

by eriktorbjorn, 22 months ago

Attachment: scummvm-00002.png added

comment:1 by digitall, 21 months ago

Responsible code for this behaviour is in gui/launcher.cpp line 1052 to 1070:
https://github.com/scummvm/scummvm/blob/master/gui/launcher.cpp#L1052

comment:2 by lotharsm, 16 months ago

Priority: normalhigh

comment:3 by sev-, 16 months ago

Owner: set to sev-
Resolution: fixed
Status: newclosed

Fixed. Please test comprehensively :D

comment:4 by sluicebox, 16 months ago

Possibly related: #13525

All of my disabled games are now highlighting correctly, thanks! That's been confusing me at least once a week since it was introduced.

comment:5 by sluicebox, 16 months ago

Also possibly related: #13541

Note: See TracTickets for help on using tickets.