Add open button functionality to the search dialog.
adding nsbeta1 keyword and accepting the bug.
Status: NEW → ASSIGNED
If you mean "open to folder where message is located", then that is already covered in bug 56419.
QA Contact: esther → laurel
Nevermind, I forgot the spec showed Open button just for opening message to standalone window.
marking nsbeta1+ and moving to mozilla0.8
Priority: -- → P2
Target Milestone: --- → mozilla0.8
Fix in hand. Works for both single and multiple messages open. Attaching patch.
Adding patch, review keywords. Adding seth, david to cc list for reviews/super reviews.
Keywords: patch, review
Making this dependent on bug 66958. I tested it with the patch taka provided in that bug.
Depends on: 66958
a lot of that code looks copied from mailWindowOverlay.js can you find a way to do this, without copying code? what happens if no messages are selected and I click on the open button. can you make it so we disable / enable the open button based on the selection?
Wouldn't it be confusing to have an "Open" button (which opens the selected messages) next to a "Close" button (which closes the search window)? I suggest putting "Open" and other items that have to do with selected messages at the left edge of the window, or making those commands only show up on the context menu (bug 66970) for the tree items.
fix in hand. attaching patch. please review/super review asap as i can get this in today if it is fine.
fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
OK using feb 8 commercial trunk builds with win98, linux rh6.0 and mac OS 9.0
Status: RESOLVED → VERIFIED
Oh yeah, forgot to add: Works for both single and multiple selection (separate windows opened for each message in a multiple selection Open). Also logged a bug to move the Open (and File, Delete buttons when they are implemented) to the right of the results pane per spec. (spec not available on mozilla yet)
Component: MailNews: Search → MailNews: Message Display
QA Contact: laurel → search
You need to log in before you can comment on or make changes to this bug.