Opened 21 years ago
Closed 21 years ago
#1588 closed defect (fixed)
FM-TOWNS: Wrong text position (regression)
| Reported by: | (none) | Owned by: | fingolfin |
|---|---|---|---|
| Priority: | high | Component: | Engine: SCUMM |
| Version: | Keywords: | ||
| Cc: | Game: | Loom |
Description
ScummVM 0.7.0CVS (Apr 11 2004 11:40:40) Features compiled in: Vorbis MP3 zLib MPEG2 Runnig under WinXP
In Loom (floppy version and FM-Towns version, but not Talkie version), the text used in close-ups appears too much on the loft of the screen, making it partly unreadable. The notes on the distaff appear too much on the left as well (I checked against the original interpreter that it is indeed the case).
The problem is recent and didn't occur in version 0.6.0. I first noticed it a few days (maybe a week) ago, but didn't report it then as I thought that it was the same as # , which was also occurring. Yet, # is now fixed and the text position remains in current CVS.
I'm attaching two screenshots showing the bug, as well as a save for Loom EGA (even though it is not really necessary: just start the game and go straight to the Elders' building to notice the bug).
Ticket imported from: #933168. Ticket imported from: bugs/1588.
Attachments (4)
Change History (22)
by , 21 years ago
| Attachment: | closeup.PNG added |
|---|
comment:1 by , 21 years ago
When I say "I thought that it was the same as # ", I meant: "I thought that it was the same as # 930385" (forgot to paste the number before submitting)
comment:2 by , 21 years ago
Hm, odd. First I thought that maybe this is caused by a change in ScummEngine::CHARSET_1(), where I added a check for the actor's room: if (a && a->isInCurrentRoom() && _string[0].overhead != 0) { But changing this back to if (a && _string[0].overhead != 0) { doesn't help.
Hm.
comment:3 by , 21 years ago
| Priority: | normal → high |
|---|
comment:4 by , 21 years ago
The problem is also occurring in Indy 3 EGA and Indy 3 Towns (but NOT Indy 3 256). The bug is noticeable during fights (the names etc. at the bottom of the screen are too much on left) and when reading the Grail Diary (which makes it rather impossible to read).
comment:5 by , 21 years ago
This is probably caused by one of the text changes I made since 0.6.0 ?!? Just guessing, actually, so far I have no idea...
Erik, maybe you have any insights on this... ? If not, feel free to re- assign this to me!
comment:6 by , 21 years ago
| Owner: | set to |
|---|
comment:7 by , 21 years ago
I think I see what's happening. The text is printed at the correct *position*, but it's centered rather than left-justified.
In 0.6.0 we stored the string's 'xpos', 'ypos' and 'color' to use as default values for the next message (I think). With the CVS version, we save all the string's parameters. Including 'center'.
If I make sure that 'center' is cleared, it works for me again.
comment:8 by , 21 years ago
Ahhh, that sounds sensible. Feel free to apply that fix (or if you prefer, I can do it later this week, depending on when I have time).
comment:9 by , 21 years ago
Hmm... I'm a bit uncertain now, because when I looked at 0.6.0 a bit closer I noticed that in most cases we do save everything, including the 'center' flag, but this one case is different: not only don't we backup the 'center' flag, we don't touch it at all. Not even to clear it.
To preserve that behaviour, I guess we have to manipulate _default directly, like in the patch I just attached. Is that acceptable, or is it too ugly?
comment:10 by , 21 years ago
I've applied my suggested, somewhat hacky fix. Feel free to make a better fix later, but this is the best I can think of.
comment:11 by , 21 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:12 by , 21 years ago
| Status: | closed → new |
|---|---|
| Summary: | LOOM: Wrong text position (regression) → FM-TOWNS: Wrong text position (regression) |
comment:13 by , 21 years ago
Sorry to be a spoilsport, but... Using latest CVS, the bug is indeed fixed in LoomEGA and Indy3EGA, but still occurs in LoomTowns and Indy3Towns.
comment:14 by , 21 years ago
I don't have any of the FM TOWNS games. (Much too rare and expensive for my taste.) Fingolfin, I seem to recall that you got the Indy3Towns that someone donated? Could you look into this?
comment:15 by , 21 years ago
| Owner: | changed from to |
|---|
comment:16 by , 21 years ago
I'll try to reproduce this if somebody tells me where and how (ideally, give me a savegame and precise instructions what to do). Manwe21 said that the problem is visible in fights. At least when doing the sparring at the very start of Indy3 (FM-TOWNS version), no problems are visible...
comment:17 by , 21 years ago
| Status: | new → closed |
|---|
comment:18 by , 21 years ago
Eriktorbjorn's fix seems to have worked after all: I still see the bug using my old savegames, but can't reproduce it with a fresh start. Sorry about that.
Screenshot of the bug (close-up)