msgReadSMIMEOverlay.xul: 1 s/observes/command/

VERIFIED FIXED in Thunderbird 11.0

Status

MailNews Core
Security: S/MIME
--
trivial
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: sgautherie, Assigned: aceman)

Tracking

(Blocks: 1 bug)

Trunk
Thunderbird 11.0
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
See bug 613435 as an example.
(Assignee)

Comment 1

6 years ago
Created attachment 572308 [details] [diff] [review]
change "observes" to "command"
[Checked in: Comment 3]

Do you mean like this?
The other 2 line changes are trailing spaces removals.
Assignee: nobody → acelists
Status: NEW → ASSIGNED
Attachment #572308 - Flags: review?(sgautherie.bz)
Comment on attachment 572308 [details] [diff] [review]
change "observes" to "command"
[Checked in: Comment 3]

Neil's best to review this, though Serge can provide feedback if he wants.
Attachment #572308 - Flags: review?(sgautherie.bz)
Attachment #572308 - Flags: review?(neil)
Attachment #572308 - Flags: feedback?(sgautherie.bz)

Updated

6 years ago
Attachment #572308 - Flags: review?(neil) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
(Reporter)

Updated

6 years ago
Attachment #572308 - Flags: feedback?(sgautherie.bz)
Checked in: http://hg.mozilla.org/comm-central/rev/543f0ca1555e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 10.0
(Reporter)

Comment 4

6 years ago
TB file still needs this too.
Status: RESOLVED → REOPENED
Flags: in-testsuite-
Resolution: FIXED → ---
(Assignee)

Comment 5

6 years ago
TB file? What is that?
(Reporter)

Comment 6

6 years ago
(In reply to :aceman from comment #5)
> TB file? What is that?

See 'URL' field of this bug.
For the time being, /mailnews/ file is used by SeaMonkey, /mail/ file is used by Thunderbird.
Status: REOPENED → ASSIGNED
(Assignee)

Comment 7

6 years ago
Created attachment 572907 [details] [diff] [review]
the same change, for Thunderbird

Thanks, for the info, I am just learning the basics of the TB source tree.
I thought /mailnews is for "mailnews core" (shared parts), /mail is for TB, /suite is for seamonkey. Is that not true?

Patch for TB attached.
(Assignee)

Updated

6 years ago
Attachment #572907 - Flags: review?(neil)
(Assignee)

Updated

6 years ago
Attachment #572907 - Flags: review?(neil) → review?(dbienvenu)
(Reporter)

Comment 8

6 years ago
(In reply to :aceman from comment #7)

> I thought /mailnews is for "mailnews core" (shared parts), /mail is for TB,
> /suite is for seamonkey. Is that not true?

You understood correctly.
But there are some exceptions in /mailnews, mostly when files were forked but not (yet) moved/unforked, like this one.

> Patch for TB attached.

Can you update the css line too, to keep both file in sync'?
(Assignee)

Comment 9

6 years ago
Yes, I can look into it.
Comment on attachment 572907 [details] [diff] [review]
the same change, for Thunderbird

If Neil's happy so am I. Thanks.
Attachment #572907 - Flags: review?(dbienvenu) → review+
(Assignee)

Comment 11

6 years ago
Should I make the css fix in a separate patch?
Ah, just update the patch with it in.
(Assignee)

Comment 13

6 years ago
Created attachment 573574 [details] [diff] [review]
the same change, for Thunderbird + css line fix (trailing space)
[Checked in: Comment 14]

done
Attachment #572907 - Attachment is obsolete: true
Attachment #573574 - Flags: review?(mbanner)
Attachment #573574 - Flags: review?(mbanner) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
Whiteboard: [good first bug] → [checkin the Thunderbird patch]
Checked in: http://hg.mozilla.org/comm-central/rev/aeb7f4dbcb2e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [checkin the Thunderbird patch]
(Reporter)

Updated

6 years ago
Attachment #572308 - Attachment description: change "observes" to "command" → change "observes" to "command" [Checked in: Comment 3]
(Reporter)

Updated

6 years ago
Attachment #573574 - Attachment description: the same change, for Thunderbird + css line fix (trailing space) → the same change, for Thunderbird + css line fix (trailing space) [Checked in: Comment 14]
(Reporter)

Updated

6 years ago
Target Milestone: Thunderbird 10.0 → Thunderbird 11.0
(Assignee)

Updated

6 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.