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 lobbyTitleTextPos to be properly cast as float #228

Merged
merged 1 commit into from
Oct 22, 2020

Conversation

okonech
Copy link
Contributor

@okonech okonech commented Oct 16, 2020

Fix for this error trying to play any songs or using preview:

Traceback (most recent call last):
File "FoFiX.py", line 171, in <module>
main.run()
File "/opt/fofix-git/fofix/game/Main.py", line 131, in run
while self.engine.run():
File "/opt/fofix-git/fofix/core/GameEngine.py", line 649, in run
rtn = self.mainloop()
File "/opt/fofix-git/fofix/core/GameEngine.py", line 633, in main
self.view.render()
File "/opt/fofix-git/fofix/core/View.py", line 263, in render
layer.render(self.visibility[layer], layer == self.layers[-1])
File "/opt/fofix-git/fofix/game/Lobby.py", line 457, in render
self.engine.theme.themeLobby.renderPanels(self)
File "/opt/fofix-git/fofix/core/Theme.py", line 871, in renderPanels
lobby.fontDict[self.theme.lobbyTitleTextFont].render(self.theme.lobbyTitleText, self.theme.lobbyTitleTextPos, scale=self.theme.lobbyTitleTextScale, align=self.theme.lobbyTitleTextAlign)
File "/opt/fofix-git/fofix/core/Font.py", line 199, in render
x -= (w / 2)
TypeError: unsupported operand type(s) for -=: 'str' and 'float'

@Linkid Linkid added this to the 4.0 milestone Oct 17, 2020
Copy link
Member

@Linkid Linkid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi !
Nice catch ! Thank you for your contribution :).

@Linkid Linkid merged commit aed6aa2 into fofix:master Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants