TodoTxtJs icon indicating copy to clipboard operation
TodoTxtJs copied to clipboard

JavaScript errors

Open TweePassen opened this issue 11 years ago • 6 comments

Hi Martin,

This may be the stupidest question ever, but, though I like TodoTxtJs a lot, Firefox has a lot of trouble executing the javascript and it asks me to stop it each minute.

Would there be a way around that?

Thanks, Marc

TweePassen avatar Jan 27 '14 09:01 TweePassen

I don't get any errors in Firefox. Can you provide some examples please? Also which version of Firefox and on what operating system.

MartinSGill avatar Jan 27 '14 09:01 MartinSGill

I have been getting these errors in Firefox 26 on Windows 8.1 and Internet Explorer 11 on Windows 7 Professional.

I don't get these errors continously. Sometimes I can add or edit taks effortlessly, but sometimes the scripts get stuck. I had the impression that adding due dates makes it worses, but I can't reproduce that.

TweePassen avatar Jan 29 '14 08:01 TweePassen

Firefox and IE 11 for me don't give any errors. Firefox does issue some warnings, but none of those should cause Firefox to prompt.

Could you please provide me with the error messages you are seeing? Without knowing what the actual errors are there's nothing I can do to help.

MartinSGill avatar Jan 29 '14 11:01 MartinSGill

I can understand you need me to be more specific. Unfortunately all I get are message from my browser that javascript is stuck and a prompt to either terminate the script or wait some more.

2014-01-29 Martin Gill [Masked] [email protected]

Preview: Firefox and IE 11 for me don't give any errors. Firefox does Safely received through a Masked Email. IF THIS IS SPAM, CLICK HERE TO BLOCK THIS MASKED EMAIL.https://emails.abine.com/[email protected]

Firefox and IE 11 for me don't give any errors. Firefox does issue some warnings, but none of those should cause Firefox to prompt.

Could you please provide me with the error messages you are seeing? Without knowing what the actual errors are there's nothing I can do to help.

Reply to this email directly or view it on GitHubhttps://github.com/MartinSGill/TodoTxtJs/issues/62#issuecomment-33574815 .

TweePassen avatar Jan 29 '14 12:01 TweePassen

That's already a lot more useful. How big is your todo.txt file?

That type of warning means that the script is spending a very long time processing all your Todos. Due dates and the like will make this worse, as they mean more processing, the more complex (due dates, meta-data, projects, contexts etc.) your todo is, and the more todos you have in the file the worse performance will get.

I've not really spent much time optimizing for performance as generally I wouldn't expect a todo file larger than say 50 entries, and modern browsers/computers/tablets will run that easily.

If people are running much larger TODO.txt files than I expected, I'll need to work on improving performance.

MartinSGill avatar Jan 29 '14 13:01 MartinSGill

My file is only 30 lines.

2014-01-29 Martin Gill [Masked] [email protected]

Preview: That's already a lot more useful. How big is your todo.txt fi

Safely received through a Masked Email. IF THIS IS SPAM, CLICK HERE TO BLOCK THIS MASKED EMAIL.https://emails.abine.com/[email protected]

That's already a lot more useful. How big is your todo.txt file?

That type of warning means that the script is spending a very long time processing all your Todos. Due dates and the like will make this worse, as they mean more processing, the more complex (due dates, meta-data, projects, contexts etc.) your todo is, and the more todos you have in the file the worse performance will get.

I've not really spent much time optimizing for performance as generally I wouldn't expect a todo file larger than say 50 entries, and modern browsers/computers/tablets will run that easily.

If people are running much larger TODO.txt files than I expected, I'll need to work on improving performance.

Reply to this email directly or view it on GitHubhttps://github.com/MartinSGill/TodoTxtJs/issues/62#issuecomment-33584396 .

TweePassen avatar Jan 29 '14 14:01 TweePassen