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

Add Second Parameter to Lost Waffle #430

Open
peterramsing opened this issue Nov 8, 2018 · 2 comments
Open

Add Second Parameter to Lost Waffle #430

peterramsing opened this issue Nov 8, 2018 · 2 comments

Comments

@peterramsing
Copy link
Owner

The issue for #423

Is this a feature request or a bug report?
Feature

What is the current behavior?
Only one param for Lost Waffle

If it's a bug please provide the steps to reproduce it and maybe some code samples.
N/A

What is the desired behavior?
lost-waffle: 1/3 3/1;
Where 3/1 is cycle/start

What's the motivation or use-case behind changing the behavior?
@Kan-Artem to populate

What version of LostGrid, browser and browser version are affected by this issue? Did this happen in previous versions?
N/A

Anything else?
I (Peter) made this issue to represent the work in the PR that @Kan-Artem made.

@peterramsing
Copy link
Owner Author

@Kan-Artem, could you fill in some of the motivation behind this?

Thanks!

@peterramsing
Copy link
Owner Author

@Kan-Artem, let's chat on this. I could see how this is cool but also don't want to add too many features not because features are bad but because sometimes there more complexity the harder things are to maintain, understand, etc. Basically, could you make a case for this?

I guess I could see having the ability to skip, but mainly want to hear from you before moving forward.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment