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

Show last unlocked level on every page #358

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbinette91
Copy link

Bug:
The menu only shows the next unlocked level if you are on the last completed level's page.
e.g. If you have unlocked Level5 and are on page Level1, you will not see Level5 in the menu.

Fix:
I modified the menu a little to show the last unlocked level no matter which page you are on.

Tested:

  • New session on Tutorial: shows nothing (same as right now)
  • Level1 completed, on page Tutorial: Shows up to Level2 (would previously show up to Level1)
  • Level1 completed, on page Level1: Shows up to Level2 (same as right now)
  • Level1 completed, on page Level2: Shows up to Level2 (would previously show up to Level1)
  • Level1 completed, on page Level20: Shows up to Level2 (would previously show up to Level1)

Fixes #295

Bug:
The menu only shows the next unlocked level if you are on the last
completed level's page.
e.g. If you have unlocked Level5 and are on page Level1, you will not
see Level5 in the menu.

Fix:
I modified the menu a little to show the last unlocked level no matter
which page you are on.

Tested:
- New session on Tutorial: shows nothing (same as right now)
- Level1 completed, on page Tutorial: Shows up to Level2 (would
previously show up to Level1)
- Level1 completed, on page Level1: Shows up to Level2 (same as right
now)
- Level1 completed, on page Level2: Shows up to Level2 (would previously
show up to Level1)
- Level1 completed, on page Level20: Shows up to Level2 (would
previously show up to Level1)

Fixes kasperpeulen#295
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.

1 participant