Closed Bug 607583 Opened 10 years ago Closed 10 years ago

Move updateOptionItems() call from xul to js

Categories

(SeaMonkey :: MailNews: Composition, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b2

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(2 files)

Sync' with Thunderbird, in order to ease bug 537219, etc.

Port "backend" code part of
http://bonsai.mozilla.org/cvsquery.cgi?module=ThunderbirdTinderbox&sortby=Date&hours=2&date=explicit&mindate=2003-08-03+23%3A36&maxdate=2003-08-03+23%3A36
"Add a new quote button for quoting the selected message like Netscape 4.x"

NB: We might want to actually add a button too, but I'll leave that to someone else in another bug.
Please double-check: I'm just assuming this is good (enough) for SeaMonkey too.
Attachment #486315 - Flags: review?(iann_bugzilla)
Attachment #486315 - Flags: review?(iann_bugzilla) → review+
Comment on attachment 486315 [details] [diff] [review]
(Av1) Just copy Thunderbird code
[Checked in: Comment 2]

http://hg.mozilla.org/comm-central/rev/4d6d44d653f4
Attachment #486315 - Attachment description: (Av1) Just copy Thunderbird code → (Av1) Just copy Thunderbird code [Checked in: Comment 2]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
If you start by composing a new message, then open the 3pane window and select a message, the quote message menuitem remains disabled.

If you reply to a message, then close the 3pane window, the quote message menuitem remains enabled.

In both cases you can work around it by clicking in and out of the message body.
Depends on: 609926
(In reply to comment #3)

Right! (I kind of expected some issue like these :-/) I filed bug 609926 :-<

I see two options ftb:
*backout my port patch, reopen this bug then wait for bug 609926 to be fixed first.
*keep this regression, proceed with bug 537219, then synchronize again when bug 609926 is fixed.

What do you prefer?
In this sort of case there are several considerations to take into account. If the patch has only affected the nightlies, then this would suggest that it would be easier to back it out now and fix bug 537219 later. If this regression has been released, then it may be easier to fix bug 537219 now, especially if you don't know what the fix for bug 609926 might be.
"blocking-seamonkey2.1=?":
Just to remember to back this out for SM 2.0b2 if bug 609926 hasn't been fixed (and copied) by then.
Status: RESOLVED → REOPENED
blocking-seamonkey2.1: --- → ?
Resolution: FIXED → ---
Summary: Merge updateOptionItems() into updateComposeItems() → Move updateOptionItems() call from xul to js
Comment on attachment 490700 [details] [diff] [review]
(Bv1) Revert MsgComposeCommands.js part of patch Av1, Port bug 609926 fix instead
[Checked in: Comment 9]

>     // Options Menu
>     goUpdateCommand("cmd_spelling");
>-    goUpdateCommand("cmd_quoteMessage");
In the TB patch you add a call to updateOptionItems() here, why not for SM?

r=me with that explained/addressed.
Attachment #490700 - Flags: review?(iann_bugzilla) → review+
Comment on attachment 490700 [details] [diff] [review]
(Bv1) Revert MsgComposeCommands.js part of patch Av1, Port bug 609926 fix instead
[Checked in: Comment 9]

http://hg.mozilla.org/comm-central/rev/ec3138bfe33a


(In reply to comment #8)
> In the TB patch you add a call to updateOptionItems() here, why not for SM?

SM doesn't need that workaround (anymore), as SM doesn't have a 'Quote' button (yet).
Attachment #490700 - Attachment description: (Bv1) Revert MsgComposeCommands.js part of patch Av1, Port bug 609926 fix instead → (Bv1) Revert MsgComposeCommands.js part of patch Av1, Port bug 609926 fix instead [Checked in: Comment 9]
Status: REOPENED → RESOLVED
blocking-seamonkey2.1: ? → ---
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.