Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Pranav-FP opened this issue Sep 14, 2022 · 6 comments
Closed

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

Pranav-FP opened this issue Sep 14, 2022 · 6 comments
Labels
A-rendering Area: Rendering & Graphics bug Something isn't working text Issues relating to text rendering/input

Comments

@Pranav-FP
Copy link

Pranav-FP commented Sep 14, 2022

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 Pranav-FP added the bug Something isn't working label Sep 14, 2022
@Pranav-FP Pranav-FP changed the title All text show as "test" in Phit All text show as "test" in Phit and some rendering issues Sep 14, 2022
@Pranav-FP
Copy link
Author

Related to #5887 I guess

@n0samu n0samu added A-rendering Area: Rendering & Graphics text Issues relating to text rendering/input labels Sep 14, 2022
@n0samu
Copy link
Member

n0samu commented Sep 14, 2022

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

@hatal175
Copy link
Contributor

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.

@n0samu
Copy link
Member

n0samu commented Sep 19, 2022

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.

@waspennator
Copy link

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

@Lord-McSweeney
Copy link
Collaborator

Fixed by #12541.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rendering Area: Rendering & Graphics bug Something isn't working text Issues relating to text rendering/input
Projects
None yet
Development

No branches or pull requests

5 participants