Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improving retry flow #9

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

cursedarcangel
Copy link

Makes repeating levels to get a better score a much better and more fun experience.

@cursedarcangel cursedarcangel linked an issue Mar 28, 2022 that may be closed by this pull request
@cursedarcangel cursedarcangel changed the title Retrying no longer plays full intro sequence Improving retry flow Mar 28, 2022
@cursedarcangel
Copy link
Author

golf.mp4

The retry button has been moved out of the pause menu, and the intro sequence no longer plays when retrying, it just goes straight to being playable.

@haudan
Copy link

haudan commented May 31, 2022

This looks great!

src/golf/ui.c Outdated
if (_golf_ui_button_name(layout, "pause_menu_retry_button")) {
_golf_ui_start_fade_out(false, false, 0, true);
}
//if (_golf_ui_button_name(layout, "pause_menu_retry_button")) {

Choose a reason for hiding this comment

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

(Small nitpick) - Don't leave commented out code, it confuses future contributors 😋

Copy link
Author

Choose a reason for hiding this comment

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

Apparently I forgot this. Thanks for pointing it out.

@ghost
Copy link

ghost commented Jul 18, 2022

Is there anything in place to prevent accidentally pressing retry and then losing your progress? If not, perhaps a confirmation popup would be a good idea...

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Quick-Retry
3 participants