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

Optional step after console command, closes #1884 #1910

Merged
merged 2 commits into from
Aug 21, 2016
Merged

Optional step after console command, closes #1884 #1910

merged 2 commits into from
Aug 21, 2016

Conversation

MSGhero
Copy link
Member

@MSGhero MSGhero commented Aug 21, 2016

Adds a boolean to see if the user wants to step after inputting a single console command (default behavior) or input multiple commands in a single frame. Adds the step() console command so you don't have to mouse over to the little stepper arrow each time you do want to advance. The boolean is regularly accessible from the console if it needs to be changed at runtime.

Cleaned up some defines that didn't seem to be relevant any longer. The entire ConsoleCommands class is wrapped in #if FLX_DEBUG, so there's no need to repeat those internally. Neko seemed fine with respect to taking out the #if flash defines.

image
Ugh, I'll get this right one day.

@Gama11 Gama11 added this to the 4.2.0 milestone Aug 21, 2016
@Gama11
Copy link
Member

Gama11 commented Aug 21, 2016

Wow, the VCR step actually seems to work as expected on Neko now. I think there were a lot of issues with it back in the day related to text field focus.

Ugh, I'll get this right one day.

Doesn't matter commits get squased anyway. ;)

@Gama11 Gama11 merged commit e802c61 into HaxeFlixel:dev Aug 21, 2016
Gama11 added a commit that referenced this pull request Aug 21, 2016
@MSGhero MSGhero deleted the console-step branch August 21, 2016 20:59
Aurel300 pushed a commit to larsiusprime/haxeflixel that referenced this pull request Apr 18, 2018
Aurel300 pushed a commit to larsiusprime/haxeflixel that referenced this pull request Apr 18, 2018
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.

2 participants