zolmeister / zethos Goto Github PK

I really like your Zethos project. I've been using it to read very very long news articles.
One thing missing though is configurable WPM, I see that 500 WPM is hardcoded, so I could just edit that number to get the speed I want, but it would be better if there is some kind of text field to input the WPM.
I did fork this and add some dirty code quick change and make it look like this

Anyway, this is just some suggestion

...Unless they have a space after them.
I'm not sure why, because I see in your code you split by both the space and newline delimiter.

Just tried to use zethos and while in Chrome I'm able to paste some text copied to the clipboard, in Firefox, the text is pasted somehow above/below the textarea and the application sees the data model in undefined state.

relevant code from main.js:

o.innerHTML = Array(8 - w[1]).join('&nbsp;')+w[0].slice(0,w[1])+'<v>'+w[0][w[1]]+'</v>'+w[0].slice(w[1]+1)

@Zolmeister I like this repo, but I'm concerned that the author hasn't spent enough years in stealth mode perfecting the technology. How many years of stealth mode perfection can we expect? Can you provide a date?

Hi,
Awesome work !
Is that possible to go higher than 500 ?

I saw this on JS1K and liked it a lot. However, while it works well given just one or two paragraphs, I find that it slows down dramatically when given a larger text. This is because it evaluates parse(i.textContent) in the loop, once for every word. It should do that just once in p whenever the text changes and the play button is clicked, Something like

in function p:
newvar=parse(i.textContent);

and in function loop:
w=newvar[index++] || p();

with newvar declared globally so it is accessible in both functions.

There must surely be some way to do this while keeping things under 1k, e.g. by reusing an existing variable instead of declaring a new one, but I haven't looked deeply into that aspect.

Martin Gradwell