Opened 16 years ago

Closed 15 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 Changed 16 years ago by fingolfin

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

comment:2 Changed 16 years ago by fingolfin

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 Changed 16 years ago by eriktorbjorn

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 Changed 16 years ago by fingolfin

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 Changed 16 years ago by eriktorbjorn

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 Changed 15 years ago by SF/monkeybutter

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

comment:7 Changed 15 years ago by SF/bigmuzzy

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 Changed 15 years ago by SF/bigmuzzy

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 Changed 15 years ago by fingolfin

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 Changed 15 years ago by fingolfin

Owner: set to fingolfin

comment:11 Changed 15 years ago by fingolfin

Resolution: fixed
Status: newclosed

comment:12 Changed 15 years ago by fingolfin

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

Note: See TracTickets for help on using tickets.