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

On-screen messages related to settings are not modified #15

Open
c30zD opened this issue Dec 13, 2017 · 2 comments
Open

On-screen messages related to settings are not modified #15

c30zD opened this issue Dec 13, 2017 · 2 comments

Comments

@c30zD
Copy link

c30zD commented Dec 13, 2017

As I explain in #14, even though it provides a fix for the main issue (messages were originally hard-coded), there is another that remains. The steps to reproduce are as follows:

  1. After loading any table, start the game
  2. When the "press [key] to launch ball" message is displayed, return to the main menu (make sure the message doesn't disappear before returning to the menu)
  3. Change the key to launch the ball
  4. Return to the game. The message isn't modified

The same happens if instead of the "launch" message, it is the "reset" message. Nevertheless, the key bindings will work, even though the message suggests otherwise.

I'm opening this issue just so it's somewhere easily accessible, in case anyone wishes to provide a fix.

@sergiomb2
Copy link
Owner

fixed with #14

@c30zD
Copy link
Author

c30zD commented Dec 31, 2017

Perhaps I didn't explain myself properly. The current issue should be reopened, because it isn't fixed. This issue is about the following two scenarios:

  1. The user starts a new game (table is already loaded) and the message to launch the ball appears. Immediately, the user goes back to the main menu and then changes the key in the configuration menu. Then the user returns to the game and sees that the message is still the same.
  2. The user finishes the game and the message to restart appears. At this point, the user goes back to the main menu and changes the reset key in the configuration menu. When he comes back, the message is still the same.

What was fixed in #14 was that the reset key and messages are not hard-coded anymore. Therefore, the next time the user starts/ends a game, the message will be correct.

Now, the current issue might never be seen by other people because it is unlikely that the user will stop the game just to change a setting like that, but as the issue is there, I believe it should be tracked somehow. This is why I purpose to reopen the issue.

@sergiomb2 sergiomb2 reopened this Dec 31, 2017
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

No branches or pull requests

2 participants