Search viewLines rather than lines

So far, search was only enabled in views with Wrap=false, so it didn't make a
difference. If Wrap is on, we need to work with viewLines rather than lines,
because that's what the code that highlights the search results expects. Note
that a viewLine index is also passed back to the onSelectItem callback, so
clients need to be able to deal with that.

We don't touch the modelSearchResults code path, as it's a bit unclear how to
support Wrap=true in that case, and we don't need it right now.

Author: stefanhaller

view.go

@@ -1087,6 +1087,8 @@ func (v *View) updateSearchPositions() {
 
 		if v.searcher.modelSearchResults != nil {
 			for _, result := range v.searcher.modelSearchResults {
+				// This code only works when v.Wrap is false.
+
 				if result.Y >= len(v.lines) {
 					break
 				}
@@ -1115,8 +1117,9 @@ func (v *View) updateSearchPositions() {
 				}
 			}
 		} else {
-			for y, line := range v.lines {
-				v.searcher.searchPositions = append(v.searcher.searchPositions, searchPositionsForLine(line, y)...)
+			v.refreshViewLinesIfNeeded()
+			for y, line := range v.viewLines {
+				v.searcher.searchPositions = append(v.searcher.searchPositions, searchPositionsForLine(line.line, y)...)
 			}
 		}
 	}