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

Warn when too many guides are going to be created #11

Open
AriaMinaei opened this issue May 22, 2014 · 5 comments
Open

Warn when too many guides are going to be created #11

AriaMinaei opened this issue May 22, 2014 · 5 comments

Comments

@AriaMinaei
Copy link
Owner

As suggested by @dbox, there should be a warning when the user enters something like "1" for the griddify command.

(Btw, until we implement that, you can cancel Griddify's operation by hitting the Esc button a few times)

@AriaMinaei
Copy link
Owner Author

Under what condition should we warn the user when too many guides are going to be drawn?

Should we simply set a maximum number of guides limit? How many?

@PetePixxx
Copy link

Since I don't know exactly how users will need to Griddify some files I would lhink many 25 or so might be a possible starting number. I had a small number in one setup and it placed a gazillion grids. Easy to clear, but...

Along the same thought line, a warning to say something like no grid available on visible screen may be a good idea if your selected grid falls outside of the picture borders. Say you have a 4x6 picture and try to place a grid at 7.

@AriaMinaei
Copy link
Owner Author

Along the same thought line, a warning to say something like no grid available on visible screen may be a good idea if your selected grid falls outside of the picture borders. Say you have a 4x6 picture and try to place a grid at 7.

That's interesting. Have you found yourself needing this feature?

@PetePixxx
Copy link

Yes! That's how it came to mind. I was testing to see if grids were placed properly and it was confusing to me when I put in a number and no grid appeared. Found out it would be placed outside the visible area. Don't know how practical this is on a regular basis.

@AriaMinaei
Copy link
Owner Author

Ah, yes, we should add this.

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

No branches or pull requests

2 participants