Opened 17 years ago

Closed 16 years ago

#1077 closed defect (fixed)

COMI: Text on scrolling backgrounds

Reported by: eriktorbjorn Owned by: fingolfin
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Monkey Island 3

Description

I don't know if this should be termed a bug or a missing feature, but when the background scrolls while text is drawn on it, the text scrolls along with the background. With the original interpreter the text remained in place.

Eventually we will probably have to implement this in ScummVM as well, because there are several places where text could easily be scrolled right off the screen, which looks silly.

Ticket imported from: #777999. Ticket imported from: bugs/1077.

Change History (12)

comment:1 by fingolfin, 17 years ago

Summary: CMI: Text on scrolling backgroundsCOMI: Text on scrolling backgrounds

comment:2 by fingolfin, 17 years ago

Can you give some specific examples as to where this happens? I just fixed a bug somewhat related to this, but only for vertical scrolling. Without a test case for this happening with horizontal scrolling, it's hard to do anything about it...

comment:3 by eriktorbjorn, 17 years ago

The earliest example I can think of would be to examine, for instance, the statue of Elaine (or anything else on that beach, I guess), and then start walking to the right before the subtitle text is removed.

comment:4 by fingolfin, 17 years ago

When you say "the text remains in place", that's a bit ambigous. It remains in place relative to the actore / the real screen in the original, I assume. While in ScummVM, it remains in place relative to the background / the room ;-).

I think this might be related to bug #744650.

Apparently, then, COMI (and maybe some V6/V7 games?) is drawing conversation text quite differently, then. The current code draws the text once. Period. Obviously, the text then can't "stay fixed" - it's part of the room graphics, so scrolling *will* affect it. Hence, text would have to be redrawn when scrolling is (or simply, be redrawn every frame, like blast text)...

comment:5 by eriktorbjorn, 17 years ago

I meant that the text remains on the same screen coordinates, regardless of how the actors move or the background scrolls. As if you had physically drawn it yourself on the glass of your monitor. (Not a very practical way of doing it, of course. :-)

> Hence, text would have to be redrawn when scrolling is (or > simply, be redrawn every frame, like blast text)...

Yes, that was my thought too.

comment:6 by SF/monkeybutter, 17 years ago

Bug 902568 has a save game that should get you to a diagonal scrolling scenario to work with.

comment:7 by SF/bigmuzzy, 17 years ago

I'm not sure my problem is connected with this.

(Subtitles set to ON). When Guybrush squeezes through cannon window and talks to Murray, he can choose line "You know, you'd look great with a melting candle on your forehead". This phrase appears partly transparent (in the middle) and so do the part of Guybrush body behind it. More than that, rest of the text doesn't vanish and new subtitles appear right above it.

(Since any other (shorter) subtitles in this room doesn't cause such effect, I guess problem somehow could be connected with fact, that this particulary phrase long enough to be displayed in two lines.)

I could attach the savegame with Guybgush's ready to select this phrase, if needed.

[English version of CoMI; Windows XP SP1; ScummVM 0.6.0pre (Feb 24 2004 12:10:45)]

comment:8 by SF/bigmuzzy, 17 years ago

Hmmm, think my very smart guess about two-lines prases was... err... too smart. At least two-lines phrase "Well, at least now you never have to worry about what to wear" is displayed correclty.

Anyway, the problem still remains.

comment:9 by fingolfin, 16 years ago

The problem described here is treated by patch #1003866 (https:// sourceforge.net/tracker/index.php? func=detail&aid=1003866&group_id=37116&atid=418822)

Everybody feel free to test it :-)

comment:10 by fingolfin, 16 years ago

Owner: set to fingolfin

comment:11 by fingolfin, 16 years ago

Resolution: fixed
Status: newclosed

comment:12 by fingolfin, 16 years ago

The patch mentioned below is in CVS now, fixing the issue.

Note: See TracTickets for help on using tickets.