search

MartinSGill / TodoTxtJs

A Javascript web app implementation of TodoTxt
72 stars 29 forks source link

JavaScript errors #62

Open TweePassen opened 10 years ago

TweePassen commented 10 years ago

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

MartinSGill commented 10 years ago

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

TweePassen commented 10 years ago

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.

MartinSGill commented 10 years ago

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.

TweePassen commented 10 years ago

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] FWD_4e89ZHtS@opayq.com

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/disableDisposable?fwd=FWD_4e89ZHtS@opayq.com

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 .

MartinSGill commented 10 years ago

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.

TweePassen commented 10 years ago

My file is only 30 lines.

2014-01-29 Martin Gill [Masked] FWD_4e89ZHtS@opayq.com

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/disableDisposable?fwd=FWD_4e89ZHtS@opayq.com

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 .