search

futurepress / epubjs-angular-reader

ePub.js Reader
Other
122 stars 37 forks source link

Searching for "whale" returns first result in Ch. 32 #25

Closed dwhly closed 10 years ago

dwhly commented 10 years ago

http://epubjs-reader.appspot.com/

Using new search feature on Moby Dick. Type "whale". First result returns from ch. 32 "Cetology". But whale is used throughout. :)

D

fchasen commented 10 years ago

Great catch. Scrolling down it looks like results from the entire book are returned, but they are unordered.

@AJRenold - We need to update the results order to match position in epub spine.

dwhly commented 10 years ago

ah… good. scrolled down a bit but clearly not enough!

On Dec 23, 2013, at 9:31 AM, Fred Chasen notifications@github.com wrote:

Great catch. Scrolling down it looks like results from the entire book are returned, but they are unordered.

Need to update order to match position in epub spine.

— Reply to this email directly or view it on GitHub.

AJRenold commented 10 years ago

Search results are now sorted by chapter https://github.com/futurepress/epubjs-search/commit/995e824d94a58127a72d682438682dc738e332d9