Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#6719 closed defect (fixed)

SCUMM: In-game help missing line of text on 2nd help screen

Reported by: BenCastricum Owned by: bluegr
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game:

Description

The SCUMM engine in-game help is missing a line of text. It seems it has disappeared since v1.5 of ScummVM. Two screenshots attached (one of a v1.4 and one of v1.7 of ScummVM) both from the official win32 distribution.

Ticket imported from: bugs/6719.

Attachments (3)

help-v1.4.png (35.4 KB ) - added by BenCastricum 10 years ago.
help.png (35.5 KB ) - added by BenCastricum 10 years ago.
help-v1.8-patched.png (53.0 KB ) - added by BenCastricum 10 years ago.

Download all attachments as: .zip

Change History (11)

by BenCastricum, 10 years ago

Attachment: help.png added

comment:1 by digitall, 10 years ago

Summary: SCUMM in-game help missing line of text on 2nd help screenSCUMM: In-game help missing line of text on 2nd help screen

comment:2 by digitall, 10 years ago

Comparing the images, this is due to the last line "or incorrect game behavior." being hidden off the bottom of the text box i.e. the amount of text is too big for the box in the font used...

comment:3 by digitall, 10 years ago

The code for this "in-game help" for SCUMM engine games is found in: https://github.com/scummvm/scummvm/blob/master/engines/scumm/help.cpp

comment:4 by digitall, 10 years ago

specifically the line: https://github.com/scummvm/scummvm/blob/master/engines/scumm/help.cpp#L110 is the responsible code.

However, none of the code in this file has been changed between v1.5 and v1.6.

I suspect the cause will be a change in the GUI or font common code as the font is notably different between the screenshots.

comment:5 by digitall, 10 years ago

Ah... This code is where the strings for this dialog are setup, but the actual display of the dialog is here: https://github.com/scummvm/scummvm/blob/master/engines/scumm/dialogs.cpp#L304

This again has no likely relevant changes, but a change in StaticTextWidget may be the cause. A simple fix would be to reduce HELP_NUM_LINES to 14: https://github.com/scummvm/scummvm/blob/master/engines/scumm/help.h#L31

However, this could break again in future if there is an issue with the reflow / overflow of contents detection for these dialogs...

by BenCastricum, 10 years ago

Attachment: help-v1.8-patched.png added

comment:6 by BenCastricum, 10 years ago

The cause of this is the use of a larger font since 1.4. https://github.com/scummvm/scummvm/blob/master/engines/scumm/dialogs.cpp#L307 get's the lineheight. This used to be 14, but is now 15.

Possible solutions to this are : - Remove the recalculation of _numLines at dialogs.cpp:315. The height of the widget is 220, 220/15=14,66666667 which gets rounded down to 14. While it used to be 220/14=15,71428571 which is rounded to 15 lines. Removing the calculation will cause the default of 15 (HELP_NUM_LINES) being used. A screenshot is attached of how that will look (on Win7) - Change the formula, since it looks like 15 lines easily fit in this widget it might need an update. - Remove one of the empty lines (ADD_LINE) from the help.cpp:103 making the help fit in 14 lines. - Change the font to one with a height of 14 again.

comment:7 by digitall, 10 years ago

Ben has submitted a fix for this as a Pull Request here: https://github.com/scummvm/scummvm/pull/503

Once this fix gets merged, this bug can be closed as fixed.

comment:8 by bluegr, 10 years ago

Owner: set to bluegr
Resolution: fixed
Status: newclosed

comment:9 by bluegr, 10 years ago

Merged Ben Castricum's pull request. The changes are minimal and fix the help dialog with the new theme. The fix will be available in the next daily version of ScummVM.

Thanks Ben for all the work on this issue! :)

Closing as fixed.

Note: See TracTickets for help on using tickets.