Closed Bug 387756 Opened 17 years ago Closed 17 years ago

text search feature no longer remembers Wrap setting

Categories

(SeaMonkey :: General, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: nelson, Assigned: neil)

References

()

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

The text search feature dialog, which you get by typing ctrl-F in a web page
or in the main email window or in a mail composer window, changed in the 
switch from 1.5a to 2.0a1.  Two of the changes are feature regressions.
They are:

1) Can no longer search backwards.  The old search dialog had 3 checkboxes, 
one of which was search backwards.  This is absent from 2.0a1.

2) The "wrap" checkbox is always checked when the search dialog comes up.
Previously it was not (initially) checked by default, and (IIRC) it 
remembered the wrap setting from one search to the next if you changed it.
Flags: blocking-seamonkey2.0a1?
Gee.  I must need sleep.  The up/down radio buttons clearly serve the 
purpose of backwards searches.  The only issue here is not remembering
my previous wrap setting.  It DOES remember my previous "Match Case" 
setting, just not the Wrap setting.  
Summary: text search feature regressions in 2.0a1 → text search feature no longer remembers Wrap setting
This also breaks window.find("", false, false, false, true) for instance.
Attached patch Back out broken checkin (obsolete) — Splinter Review
Assignee: general → neil
Status: NEW → ASSIGNED
Attachment #271993 - Flags: review?(gavin.sharp)
This makes wrap the default but it remembers until restart if you turn it off.
Attachment #271993 - Attachment is obsolete: true
Attachment #272148 - Flags: review?(gavin.sharp)
Attachment #271993 - Flags: review?(gavin.sharp)
Attachment #272148 - Flags: review?(gavin.sharp) → review+
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Flags: blocking-seamonkey2.0a1?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: