ruffle icon indicating copy to clipboard operation
ruffle copied to clipboard

All text show as "test" in Phit and some rendering issues

Open Pranav-FP opened this issue 3 years ago • 4 comments

Describe the bug

Menu

image All the menu text show up as "test" and also rendering is not perfect.

Level

image Only the outlibes are rendered properly

Expected behavior

Menu

image

Level

image

Affected platform

Desktop app

Operating system

Windows 11

Browser

NA

Additional information

SWF Link - https://www.addictinggames.com/sites/default/files/flash_games/phit_0.swf

Version - ruffle-nightly-2022_09_14 (latest as of writing this)

Pranav-FP avatar Sep 14 '22 07:09 Pranav-FP

Related to #5887 I guess

Pranav-FP avatar Sep 14 '22 07:09 Pranav-FP

Looks like it's showing the correct number of menu options now, at least! I wonder if the text issue can be fixed soon.

n0samu avatar Sep 14 '22 21:09 n0samu

So the issue here is that there's a 'button sprite' -> button -> text heirarchy. In the button sprite code CPhitButton it assigns a text to this._buttonText which is the variable name for the text. In flash this works fine but ruffle doesn't make that connection, I assume because of scoping issues (assigning to this._button._buttonText fixes this but ruins flash). I assume #5492 will help? Not sure what is the pull request currently working on this.

hatal175 avatar Sep 18 '22 21:09 hatal175

Oh interesting, sounds very similar to #2181 then. As far as I understand, this would take quite a big refactor to fix and no one is working on it at the moment.

n0samu avatar Sep 19 '22 04:09 n0samu

Blocks are rendering properly now, but the text is still somewhat buggy. I do notice that during gameplay, most of the button text shows up normally save for the quit button, but as soon as the buttons become active as you move pieces around, they start changing text to test.

Screenshot_20230727_013323_Chrome Screenshot_20230727_013400_Chrome

waspennator avatar Jul 27 '23 07:07 waspennator

Fixed by #12541.

Lord-McSweeney avatar Aug 06 '23 05:08 Lord-McSweeney