Skip to content
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

Fix splitscreen end of match name #1

Merged
merged 1 commit into from
Aug 24, 2017
Merged

Fix splitscreen end of match name #1

merged 1 commit into from
Aug 24, 2017

Conversation

supersmo
Copy link
Contributor

@supersmo supersmo commented Aug 23, 2017

UILayerWinner.ManialinkPage was concatenating so each time a match ended another frame with the most recent winner was added. This resulted in the end of match scene displaying all previous winners overlapping the current winner.

Reference of the bug:
https://forum.maniaplanet.com/viewtopic.php?f=470&t=22091#p182673
https://forum.maniaplanet.com/viewtopic.php?f=427&t=12535#p113598

UILayerWinner.ManialinkPage was concatenating so each time a match ended another frame with the most recent winner was added. This resulted in the end of match scene displaying all previous winners overlapping the previous winner.

Reference of the bug:
https://forum.maniaplanet.com/viewtopic.php?f=470&t=22091#p182673
https://forum.maniaplanet.com/viewtopic.php?f=427&t=12535#p113598
@supersmo
Copy link
Contributor Author

To reproduce the bug:
Start a split screen game with 2 players.
Let player 1 win the match (by winning 3 rounds in a row)
At the end of the match player 1's name is displayed.
Continue with the next match but let player 2 win the match (by winning 3 rounds in a row)
Both player 1 and player 2's name will be displayed on top of each other.

Example:
splitscreen bug

@Aessi Aessi merged commit 605f5e8 into maniaplanet:master Aug 24, 2017
@Aessi
Copy link
Member

Aessi commented Aug 24, 2017

Thanks for the fix. It will be included in the next Maniaplanet client udpate.

@supersmo
Copy link
Contributor Author

Great! :o)

@supersmo supersmo deleted the bugfix_splitscreen_overlapping_winner_names branch August 24, 2017 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants