Closed Bug 549802 Opened 11 years ago Closed 11 years ago
Reply button doesn't launch mail composition window
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.3a3pre) Gecko/20100302 SeaMonkey/2.1a1pre Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.3a3pre) Gecko/20100302 SeaMonkey/2.1a1pre I am using Seamonkey's Mail/News to access a Gmail IMAP account. I have the message preview pane disabled, and I have Seamonkey set to open each new e-mail in a separate window. Whenever I click on the reply button in the window of a message I want to reply to, nothing happens. I can right-click on the message in the Thread Pane and click on "Reply to Sender Only", and I will get a window to compose my reply in. But if I click the Reply button, Mail/News just sits there. Reproducible: Always Steps to Reproduce: 1. Open Mail/News. 2. Open a message. 3. Click on the Reply button. Actual Results: Reply button registers a click by depressing, but no Reply window is launched. Expected Results: Mail/News should have launched a message composition window with the sender's e-mail address in the "To:" field.
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a2pre) Gecko/20100224 SeaMonkey/2.1a1pre Error Console display: Error: gFolderDisplay is not defined Source File: chrome://messenger/content/mailWindowOverlay.js Line: 1131
Status: UNCONFIRMED → NEW
Component: MailNews: Message Display → MailNews: Composition
Ever confirmed: true
OS: Linux → All
QA Contact: message-display → mailnews-composition
Hardware: x86_64 → All
Version: unspecified → Trunk
This fixes it for me. Luckily the back-end functions called by gFolderDisplay already take care of stand alone mode windows, i.e. the actually displayed message is checked rather than the one currently selected in the thread pane: gFolderDisplay.selectedMessage -> gDBView.hdrForFirstSelectedMessage -> GetKeyForFirstSelectedMessage: // If we don't have a tree selection we must be in stand alone mode... <http://mxr.mozilla.org/comm-central/source/mailnews/base/src/nsMsgDBView.cpp#7022>
Comment on attachment 430403 [details] [diff] [review] add folderDisplay JS Remove it from messenger.xul, no?
(In reply to comment #5) > (From update of attachment 430403 [details] [diff] [review]) > Remove it from messenger.xul, no? I don't know which implications that might have (I didn't introduce it in messenger.xul in the first place) so I'll let Karsten decide. Cf. comment 3: TB has it in 3 files, we (currently) have it only one. For example I don't know if we'd have to add the line to our messageWindow.xul if we removed it from messenger.xul.
(In reply to comment #6) > (In reply to comment #5) > > (From update of attachment 430403 [details] [diff] [review] [details]) > > Remove it from messenger.xul, no? > TB has it in 3 files, One too many, in fact. > we (currently) have it only one. For example I don't know if we'd have to > add the line to our messageWindow.xul if we removed it from messenger.xul. But if you're adding it to mailWindowOverlay.xul then it will get used whenever mailWindowOverlay does, i.e. both messenger and messageWindow. Or you could ignore the overlay and just add it to messageWindow.xul directly.
Attachment #430403 - Flags: review?(mnyromyr) → review-
Bug 546040 depends on this and since I requested branch approval there I do it here, too, using approval‑seamonkey2.0.4 to be in sync with bug 546040. Any change (e.g. move to approval‑seamonkey2.0.5) to the flags there should be done here, too.
Comment on attachment 432568 [details] [diff] [review] move folderDisplay JS [Checkin: comment 11] Also a204=me here and in 546040 if Neil agrees.
Attachment #432568 - Flags: review?(mnyromyr) → review+
Attachment #432568 - Flags: superreview?(neil) → superreview+
Comment on attachment 432568 [details] [diff] [review] move folderDisplay JS [Checkin: comment 11] http://hg.mozilla.org/comm-central/rev/2750fdd9a5f6
Attachment #432568 - Attachment description: move folderDisplay JS → move folderDisplay JS [Checkin: comment 11]
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
Attachment #432568 - Flags: approval-seamonkey2.0.4? → approval-seamonkey2.0.4+
You need to log in before you can comment on or make changes to this bug.