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

Implement Break skipping (Spacebar) #3264

Closed
wants to merge 9 commits into from

Conversation

zachllogan
Copy link

I added the ability to skip breaks by pressing the space bar.

double startTime = -1;
foreach (var b in breaks)
{
if(Time.Current > b.StartTime && Time.Current + skip_required_cutoff + fade_duration < b.EndTime)

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@peppy
Copy link
Sponsor Member

peppy commented Sep 4, 2018

Doesn't have to be done in this PR, but at some point there should be a visual element that allows skipping without using a key binding.

@zachllogan
Copy link
Author

I was thinking of reusing the skipoverlay visual elements, I didn't think they would work well with the breakoverlay visual elements, though. But yeah, at some point there should be something. Probably would be best in a different PR. I'll ask about the design it should have if I ever want to implement it, otherwise I'll leave it to you guys.

@bdach bdach mentioned this pull request Nov 17, 2019
@peppy
Copy link
Sponsor Member

peppy commented Nov 21, 2019

Out of date, and should also show a skip button. Please re-open with an updated implementation if still interested in giving this a go! My changes in #6901 should be helpful.

@peppy peppy closed this Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants