Opened 6 years ago
Closed 3 years ago
Last modified 3 years ago
#9690 closed feature request (fixed)
SCI: PQ4CD: Needs "dual" audio mode selection
|Reported by:||csnover||Owned by:||bluegr|
|Cc:||Game:||Police Quest 4|
Like KQ6 and LB2, PQ4CD’s in-game settings only allow toggling between text or speech, instead of allowing both at the same time.
Change History (11)
comment:1 by , 6 years ago
|Priority:||low → normal|
comment:2 by , 6 years ago
comment:3 by , 3 years ago
@m-kiewitz , what do you mean by 'injected view is missing'?
And what is implemented?
comment:4 by , 3 years ago
I patched the game so that people can select "dual" in the game menu.
For this game there is no graphic included saying "dual", so right now there are 3 selections in the game menu called "text", "text" followed by "speech". The first "text" selection is the dual mode, which means text + speech combined, the second "text" is really just that - text only and the 3rd "speech" is well only speech.
What needs to be done is inject such a 3rd graphic into the game. We already do this for SCI 16-bit games, but SCI32 has totally different code for loading these graphics. That code needs to be adjusted too.
by , 3 years ago
comment:5 by , 3 years ago
Thanks for your detailed answer.
We're talking about the configuration menu invoked during game play "from inside", right?
I see only SPEECH and TEXT buttons, without a 3rd option. (see my attachment)
With SPEECH it's only speech, but with TEXT is both text and speech.
The "Text and speech" configuration from ScummVM behaves as expected - "Both" gives both of them, "Subtitles" gives only text.
ScummVM 2.1.0 (Oct 5 2019 01:07:22)
Features compiled in: Vorbis FLAC MP3 RGB zLib MPEG2 FluidSynth Theora AAC A/52 FreeType2 JPEG PNG cloud (servers, local)
PQ4: Aug 12, 1994
comment:6 by , 3 years ago
When you click on it, you will have to click on it 2 times to get "Speech" enabled.
Without the script patch only 1 click is required. That's because the 3rd "dual" option is hidden in there atm.
comment:7 by , 3 years ago
|Status:||new → closed|
comment:8 by , 3 years ago
I would have preferred the resource patch to apply to the existing resource. All the SCI16 view patches work that way. It's weird that the resource was patched, but the script patch was not modified accordingly before.
comment:9 by , 3 years ago
m-kiewitz: that is a nice enhancement. We got a base patch to work on, so we can change the patched resource and append it to the end of the other view resource. We should use the resource patcher for this, as its functionality will suit us in other cases as well, compared to a hardcoded array of bytes.
comment:10 by , 3 years ago
bluegr: I don't have a problem with the resource patcher. It's nice to have and should be very useful in certain situations. My issue is with the new view, instead of modifying the already existing view with the resource patcher (unless I'm mistaken on this, but it looks to me like it would add a new view). In my opinion, the "dual" cel should be the 3rd cel of that existing view.
Now implemented, injected view is missing right now though.