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

Fix the bad UX between your need to alert and my need to not care. #23

Closed
envygeeks opened this issue Jun 2, 2015 · 3 comments
Closed
Labels

Comments

@envygeeks
Copy link
Contributor

I understand your need to alert, I really do... I do not however appreciate the bad alert designs. It is not ever acceptable to take over and block a users ability to use their software in a non-critical situation and "paths size threshold exceeded" is not a critical message, it's something you could have clearly side alerted about while disabling it for the current session. Even Atom itself side alerts critical messages it can recover from, why are you not using that UX idea for your UI?

What do you do instead? You block my ability to use my own editor until I pay attention to something that is irrelevant, that's bad design, no matter how you try to justify it, what makes it worse is you do not allow click close on any area so I can rid of it quickly and get work done and no having a button that says "ignore this" is not good UX.

I appreciate the software it's great 👍 , I don't appreciate the bad UX 👎

@abe33
Copy link
Owner

abe33 commented Jun 2, 2015

You're completely right, it was my mistake. I hope to fix that soon by making pigments able to handle thousands of variables without requiring to parse their value every time something change (which is actually the biggest bottleneck).

I can raise the threshold to say 300 or 500 so that it won't popup unless you're in a really huge project, that would be a temporary measure until I finish working on optimizing the management of the variables collection (currently in cn-variables-collection). What do you think about it?

@envygeeks
Copy link
Contributor Author

@abe33 to fix it in this particular project I ignored the folders causing the issue and sent a pull request to default ignore those folders since they are quite common since Ruby is a popular language like JavaScript (Node/io.js)

@abe33
Copy link
Owner

abe33 commented Jun 2, 2015

Yup, I saw your PR, looks good to me, I'll merge it soon. I think I'll raise the threshold too for the time being.

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

No branches or pull requests

2 participants