Opened 2 years ago

Closed 23 months ago

Last modified 22 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 2 years ago.
scummvm-00001.png (53.9 KB ) - added by eriktorbjorn 2 years ago.
scummvm-00002.png (29.9 KB ) - added by eriktorbjorn 2 years ago.

Download all attachments as: .zip

Change History (8)

by eriktorbjorn, 2 years ago

Attachment: scummvm-00000.png added

by eriktorbjorn, 2 years ago

Attachment: scummvm-00001.png added

by eriktorbjorn, 2 years ago

Attachment: scummvm-00002.png added

comment:1 by digitall, 2 years 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, 23 months ago

Priority: normalhigh

comment:3 by sev-, 23 months ago

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

Fixed. Please test comprehensively :D

comment:4 by sluicebox, 23 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, 22 months ago

Also possibly related: #13541

Note: See TracTickets for help on using tickets.