Code review comment for lp:~vthompson/urps/fix-label
Revision history for this message
| Max Tither (11ubuntucg) wrote | # |
review:
Needs Fixing
(runtime review)
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
Your fix is excellent. The only issue I have is in the runtime, when a user click the number of rounds they want for the game and moves to the next page, it will display the round specified the last time the game was played(as it was saved in the db). This is the major setback I have. for the record, I want to display the rounds selected in the game title under the variable "gameLength".
The reason at first I said that the name was an issue was because I thought that the two would be similar, but that is not the case. I tried to apply the fix you did with the label to the round numbers, but to no success. I did update a new verson of the app with your fix. Should you fix this last thing, then I will merge everything into the code.
Thank you,
Max Tither