Composer should have only one menu item for Find and Replace... Right now there are two menu items under the Edit menu that do exactly the same thing. This is very confusing.

Yuck! I agree. Seems like we should do this for 6.5 (there's one more chance to make UI string changes, according to Robin.) Setting to 0.9.2 The current "Find" should be something like "Find / Replace..." or "Find & Replace..." Any preferences?

'Find and Replace'

Either "Find and Replace..." or "Find & Replace..."

So we need to wait for the "checkin window" before final UI freeze to check this in, correct (i.e., can't do it now)?

Thanks, Ryan r=cmanske

these fixes should certainly go in, but don't we need to fix mail compose too? I think we need to change these files: ComposerCommands.js --> remove nsReplaceCommand & registration of cmd_replace MsgComposeCommands.js --> remove 3 lines which reference cmd_replace messengercomposer.xul --> remove menu item

sr=sfraser. cc kerz to check that other users of editor in the commercial tree are OK.

looks great r=anthonyd

a= asa@mozilla.org for checkin to 0.9.1

Ryan wanted me to check this in for him, but apparently the mailnews part of the patch is un-sr='ed. cc sspitzer for review. I won't check this in (yet?).

assuming it's ok with jglick, sr=sspitzer jglick, if it is ok can you update the spec? http://www.mozilla.org/mailnews/specs/compose/Comp_Menus.html

a= asa@mozilla.org for checkin to the trunk. (on behalf of drivers)

checked in.

Updated spec on its way...

verified.