180977 - [typeahead find] shift-F3/shift-accel-G doesn't cycle warp backwards to the bottom of the document

Status: VERIFIED FIXED

(SeaMonkey :: Find In Page, defect)

Description

[sairuh (rarely reading bugmail)]

shift-F3/shift-accel-G doesn't cycle backwards when i'm using it in conjuction
with typeahead find. on the other hand, F3/accel-G works fine to cycle forwards
when using it with typeahead find.

found using 2002.11.19 comm trunk builds on all platforms.

here's a testcase, using with non-linked text --although this is problem with
typeahead find for either linked or non-linked text.

    this is a sentence ending with this
    here is this line
    go visit this place

steps:

1. hit /
2. enter 'this' (without the quotes).
3. once TAF finds the 1st instance, hit F3 (or accel-G) to cycle fwd to the 4th
instance on the 3rd line in the sample above.
4. move backwards: hit shift-F3 (or shift-accel-G) 4 times. you'll end up on the
1st instance on the 1st line.
5. now, try to continue to cycle backwards --hit shift-F3 (or shift-accel-G) again.

results: selection doesn't move.

expected: selection should continue to cycle backwards.

Comment 1

[Aaron Leventhal]

The only bug I'm seeing with Shift+F3/Accel+Shift+G is that it won't wrap back
around to the bottom of the document.

Comment 2

[Itamar]

Aaron you are right - changing summary...

Comment 3

[Aaron Leventhal]

Attachment: When wrapping through doc second time, make sure to start at end of doc

Comment 4

[Simon Fraser [no longer active]]

Are the ranges always going to be collapsed at this point?

Comment 5

[Aaron Leventhal]

> Are the ranges always going to be collapsed at this point?
Yes, GetSearchContainers() takes care of that.

Comment 6

[Akkana Peck]

Comment on attachment 108856 [details] [diff] [review]
When wrapping through doc second time, make sure to start at end of doc

r=akkana

Comment 7

[Aaron Leventhal]

fixed

Comment 8

[sairuh (rarely reading bugmail)]

vrfy'd fixed using 2002.12.16.08 comm trunk bits on linux rh8 and win2k, and
2002.12.16.07 mozilla mach-o bits on mac 10.2.2.

Comment 9

[Jesse Ruderman]

*** Bug 187058 has been marked as a duplicate of this bug. ***