onUpdateLinting timeout length about p5.js-web-editor HOT 8 CLOSED

@MathuraMG I think you told me a reason for the timeout being this long but I can't remember.

from p5.js-web-editor.

Yeah I figured there was probably a reason for the timeout being that long- maybe to give a screen reader time to catch up?

Personally, with my particular vision disorder (I'm legally blind only in one eye) I don't rely on a screen reader as much as I do inverting the colors of the display. I was playing with it, and setting the timeout to 800ms instead of 2000 made more sense for my particular use case.

from p5.js-web-editor.

If the timeout was 200ms, the alert beep seemed way to insistent. So we just increased the time so that it seems usable for people who are just learning to code as well.
We are yet to test this particular time. Making a note that 800ms works better!
(cc @CleezyITP)

from p5.js-web-editor.

@MathuraMG Yeah 200ms sounds way too quick- like, it wouldn't even give you time to put in a closing bracket for a function.

On the other hand, 2000ms might be more usable for people who are new to coding, not especially quick typists (maybe suffer from repetitive stress injuries?), or rely more heavily on screen readers than I do.

Would a UI element to set the timeout make sense, since this could vary from user to user?
(cc @catarak @CleezyITP)

from p5.js-web-editor.

I'd like to have as few preferences as possible, as I think having lots of configuration options can be overwhelming for beginner coders. I think our options are

1. Put in a UI slider, with some reasonable limits.
2. Put in a UI slider, do some user testing and see what most people like, then pick a value and hide the slider.
3. Pick a value, do some user testing, and decide whether it's good or needs to change.

from p5.js-web-editor.

Option no 3 seems highly practical - I agree with @catarak that too many choices can be overwhelming. Also, we will include this in our test scenarios. (+@CleezyITP)

from p5.js-web-editor.

Option 3 is probably the quickest for dev time. In the long run, if there's any kind of "advanced" preferences, maybe users could customize it in there?

from p5.js-web-editor.

I'm trying to close out stale tickets. There's not really any active work to be done on this one, and it can be reopened if anyone wants to change this feature.

from p5.js-web-editor.