613791 - Find Again and Find Previous ignore "wrap around"

Status: RESOLVED DUPLICATE of bug 621944

(Thunderbird :: Message Reader UI, defect)

Description

[Frank Burleigh]

User-Agent:       Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8pre) Gecko/20101120 Firefox/4.0b8pre
Build Identifier: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8pre) Gecko/20101116 Thunderbird/3.3a1

It's easy to create a scenario where Edit, Find Again and Find Previous don't find obviously visible text.  That appears to be because the functions respect the current settings on the Find dialog (search beckwards, wrap around).

Reproducible: Always

Steps to Reproduce:
1. Make a bunch of text.  Put "my man!" in the middle of the text.

2. Open Find dialog, turn *on* wrap around and turn *on* search backwards.  Close the find dialog without finding.

3. Put the cursor before "my man!" Now use Find Previous.  It fails.  To me, that's wrong.  Number one, I'm unconvinced that the behavior should be paying attention to an invisible setting that might persist across sessions.  And number two, if you're paying attention to these settings, wrap around must be broken.

It's easy to play with different settings of the checkboxes in the find dialog.



This whole area seems a little messy.  If you're gonna have Find Previous, then I think Find Again would be Find Next -- else "Find Previous" might mean find the thing I wanted to find the time I found before this time.

Comment 1

[Ludovic Hirlimann [:Usul]]

Is there anything in Tools -> error console when you do this ?

Comment 2

[Frank Burleigh]

None that I see.

But I did see this along the way:

1. Close TB
2. Open TB, turn on error console, clear.
3. Select a message.  Cmd+r to reply.

For me, this produces:

Error: uncaught exception: [Exception... "Operation is not supported"  code: "9" nsresult: "0x80530009 (NS_ERROR_DOM_NOT_SUPPORTED_ERR)"  location: "chrome://messenger/content/messengercompose/MsgComposeCommands.js Line: 1361"]

But only on the first attempt.

Comment 3

[Wayne Mery (:wsmwk)]

Reporter, do you still see this problem when using version 10 or newer?

if you are unable to reproduce, please close by setting status to resolved, and resolution to WORKSFORME or another appropriate setting.

If you are able to reproduce, please add new details, and a testcase if one does not already exist in the bug report.

Comment 4

[Frank Burleigh]

The behavior is unchanged.  

I think I can simplify and focus this bug without having to restate the "steps to reproduce."

* With a blob of text in a new message window, place your cursor after text you want to find, then enable "wrap around."  I'd expect a search function to highlight the search text.  Earlybird (12a2) does not.

* With a blob of text in a new message window, place your cursor before the text you want to find, enable "wrap around" and "search backward."  I'd expect a search function to search to the beginning of text, then to wrap to the end of text, then search.  Earlybird 12a2 does not.

Comment 5

[Wayne Mery (:wsmwk)]

I can't reproduce with Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Thunderbird/15.0a1 20120508.  Perhaps I'm doing something wrong