Opened 15 years ago
Closed 6 years ago
#7769 closed feature request (fixed)
DRASCULA: Add pause mode
Reported by: | SF/simsaw | Owned by: | criezy |
---|---|---|---|
Priority: | normal | Component: | Engine: Drascula |
Version: | Keywords: | ||
Cc: | Game: | Drascula |
Description
In the translated versions of Drascula, the subtitles are often hard to read in the short time they are displayed on the screen. Therefore, I would be glad if someone could add a pause mode activated and disabled by the pause key so that one can take as many time as he wants to read the text above the characters' heads.
Ticket imported from: #3009133. Ticket imported from: feature-requests/585.
Change History (6)
comment:1 by , 7 years ago
Component: | → Engine: Drascula |
---|---|
Game: | → Drascula |
comment:2 by , 7 years ago
comment:3 by , 6 years ago
@criezy: Since you have a patch for this, could you push this to a branch in your fork? It can maybe be rebased / fixed up and then open a PR for this.
comment:4 by , 6 years ago
Pushed: https://github.com/criezy/scummvm/commit/5667ded
I am still not sure that is the best solution, as explained in my previous comment, but it should work (at least it was working back when I wrote that code).
comment:5 by , 6 years ago
@criezy: Thanks for pushing that public. It looks pretty self-explanatory and self-contained. I think it would be worth opening this as a Pull Request for discussion / merging... at least this bug is now 8 years old, so maybe add at least some fix and it can always be improved / reworked in future if a better solution emerges.
comment:6 by , 6 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I have decided to push my commit. We can always improve the implementation later if we feel the need for it.
I actually implemented this back in march, but didn't merge it or even push it to my fork back then for a few reasons, and then forgot about it.
One of the reason was that it was the GSoC application period and a student was considering this task for his application PR. This was actually the reason why I implemented (I wanted to check that I was understanding the code correctly and that my ideas would work in case he had questions). But I didn't push it to let the student have the opportunity to work on it himself (in the end he didn't do it).
That reason is now obsolete, but the other two reasons are still valid and are:
One difficulty with the text speed setting is that currently extra engine gui options can only be booleans (true or false). So we would either have to only allow setting it by editing the scummvm.ini file (which would make it more difficult to know about, set or change) or we would need to turn it into a "slower subtitles" boolean that would control whether to use a hardcoded delay or none. Or possibly the two could be combined with the "slower subtitles" option having a default delay that could be changed in the scummvm.ini.
What do you think of all this?