Remove/hide "Show Remote Content" menuitem(s) in MailNews

RESOLVED FIXED

Status

SeaMonkey
MailNews: Message Display
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: stefanh, Assigned: Chris Thomas (CTho) [formerly cst@andrew.cmu.edu cst@yecc.com])

Tracking

({fixed-seamonkey1.0, fixed1.8.0.1})

1.8 Branch
fixed-seamonkey1.0, fixed1.8.0.1
Bug Flags:
blocking-seamonkey1.0 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

7.21 KB, patch
Ian Neal
: review+
neil@parkwaycc.co.uk
: superreview+
Chris Thomas (CTho) [formerly cst@andrew.cmu.edu cst@yecc.com]
: approval-seamonkey1.0+
Details | Diff | Splinter Review
(Reporter)

Description

12 years ago
We should remove the "Show Remote Content" menuitem(s) on branch - it doesn't work. IanN is currently working on enabling the feature, so branch is enough.
(Reporter)

Comment 1

12 years ago
Created attachment 202573 [details] [diff] [review]
Hide menuitems

I'll just hide them (not tested yet).
(Reporter)

Updated

12 years ago
Attachment #202573 - Attachment is obsolete: true
(Reporter)

Comment 2

12 years ago
We should remove those menuitems for 1.0, since they don't work.
Flags: blocking-seamonkey1.0?
Flags: blocking-seamonkey1.0? → blocking-seamonkey1.0+
Assignee: mail → cst
Created attachment 208217 [details] [diff] [review]
remove them for real
Attachment #208217 - Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #208217 - Flags: review?(iann_bugzilla)
Status: NEW → ASSIGNED
Whiteboard: [cst: r? sr?]

Comment 4

12 years ago
Comment on attachment 208217 [details] [diff] [review]
remove them for real

Assuming that this only lands where bug 294307 doesn't :-)
Attachment #208217 - Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Blocks: 323357

Updated

12 years ago
Attachment #208217 - Flags: review?(iann_bugzilla) → review+
Whiteboard: [cst: r? sr?] → [cst: landme when 1.8.0 reopens]
Attachment #208217 - Flags: approval-seamonkey1.0?

Comment 5

12 years ago
Comment on attachment 208217 [details] [diff] [review]
remove them for real

a=me for SM1.0, one to go
Comment on attachment 208217 [details] [diff] [review]
remove them for real

second a=me :p
Attachment #208217 - Flags: approval-seamonkey1.0? → approval-seamonkey1.0+
Checked in to MOZILLA_1_8_0_BRANCH
Keywords: fixed1.8.0.1
Whiteboard: [cst: landme when 1.8.0 reopens] → fixed-seamonkey1.0
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Keywords: fixed-seamonkey1.0
Whiteboard: fixed-seamonkey1.0
You need to log in before you can comment on or make changes to this bug.