Comment 8 for bug 1420301

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :

Ah sorry, I read your report twice but still managed to miss your point! I thought it was about the snap delay, but its not. My fault!

This is the relevant piece of code; please note the comment at the end of the line ;-)

const int timeout_val = 1200; // TODO add preference for snap indicator timeout?

I'm not sure though if we should add more configurability here; See https://www.activecollab.com/blog/9-software-configurability.html

We could just as well set this value to 2000 ms, that would already be a big increase. Or even to 4000 ms maybe? I don't think
that we should keep it around much longer.

Why do you want to keep it longer? Is it just that the time to read is too short now?