Opened 4 months ago

Closed 12 days ago

#15262 closed defect (fixed)

GUI: Options dialog show incorrect default options

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

Description (last modified by eriktorbjorn)

When not overriding the graphics settings for a game, it still shows the settings grayed out like this:


Aspect ratio correction and V-SYNC are both checked. But this seems confusing to me, because my default settings are to have them both unchecked:


It seems that ScummVM displays the default setting, not the actual global setting?

Attachments (2)

scummvm-00032.png​ (45.9 KB ) - added by eriktorbjorn 4 months ago.
scummvm-00033.png​ (43.4 KB ) - added by eriktorbjorn 4 months ago.

Download all attachments as: .zip

Change History (6)

by eriktorbjorn, 4 months ago

Attachment: scummvm-00032.png​ added

by eriktorbjorn, 4 months ago

Attachment: scummvm-00033.png​ added

comment:1 by eriktorbjorn, 4 months ago

Description: modified (diff)

comment:2 by somaen, 2 weeks ago

Priority: normal → high

Would be good to resolve this for the 2.9.0 release.

comment:3 by sev-, 12 days ago

In 52b7b319:

COMMON: Return application-level value for ConfMan if exists. Bug #15262

This has been since forever. As it is mentioned in the config-manager.cpp,
we need to get rid of these methods. So far, they are used in GUI as supposed
and by usage, the devs were always relying on this behavior.

Thus, changing it should be safe as it seems, however, I plan to remove
these methods completely, so it stays clean without side effects.

comment:4 by sev-, 12 days ago

Owner: set to sev-
Resolution: → fixed
Status: new → closed

Fixed.

Note: See TracTickets for help on using tickets.