Speed Reader and Synonym Improvements, Small Fixes

pacocoursey commented 6 years ago

More pull requests for you! Getting annoying yet? :>

Add small delay of 200ms before starting the speed reader
Prevent speed reader initialization if too few words selected. This fixes a bug that prevents users from exiting the reader mode. Also early exit so it's (immeasurably) faster!
✨ Automatically scroll the text to the appropriate position when using the speed reader.

Correctly update the synonym index to wrap at end of array
✨ Improve the synonym selector behaviour so that you can actually see previously off-screen synonyms.

As always, let me know if there are any conflicting opinions about certain behaviors. Happy to rework!

neauoire commented 6 years ago

omg, it scrolls to the word!!!!

neauoire commented 6 years ago

that's so cool.

neauoire commented 6 years ago

Mhmm I'm having really weird scrolling behaviours on longer files.

pacocoursey commented 6 years ago

Not sure of the "correct" GitHub way to make a small change for this but here's the 1 line fix you can commit yourself.

// reader.js line 55

left.go.scroll_to(0, left.reader.segment.from + range);

hundredrabbits / Left