If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Remove docs for "Read Mail" and "New Message..."

RESOLVED FIXED in Firefox 2 alpha1

Status

Firefox Graveyard
Help Documentation
RESOLVED FIXED
12 years ago
a year ago

People

(Reporter: Jeff Walden (gone starting June 8), Assigned: Gids Goldberg)

Tracking

({fixed1.8.1})

Trunk
Firefox 2 alpha1
fixed1.8.1

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

12 years ago
These items were removed in bug 306453.  I haven't looked to determine whether accel+M was removed as a keyboard shortcut for reading mail or not, so that'll need to be checked before a patch is posted.

Comment 1

12 years ago
The keyboard shortcut was removed as well.
(Assignee)

Comment 2

12 years ago
Created attachment 209194 [details] [diff] [review]
Removes references to 'read mail' and 'new message'
Attachment #209194 - Flags: review?

Comment 3

12 years ago
Comment on attachment 209194 [details] [diff] [review]
Removes references to 'read mail' and 'new message' 

Almost :)
You missed the toc entries:
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/browser/locales/en-US/chrome/help/firebird-toc.rdf&mark=330-331&rev=1.42#326
Attachment #209194 - Flags: review? → review-
(Assignee)

Comment 4

12 years ago
Created attachment 209220 [details] [diff] [review]
Removes references to 'read mail' and 'new message' 

Addresses Steffen's comment.
Attachment #209194 - Attachment is obsolete: true
Attachment #209220 - Flags: review?
(Assignee)

Updated

12 years ago
Attachment #209220 - Flags: review? → review?(steffen.wilberg)

Comment 5

12 years ago
Comment on attachment 209220 [details] [diff] [review]
Removes references to 'read mail' and 'new message' 

That's it, thanks.

Requesting approval1.8.1 since the patch in bug 306453 already has branch approval.
Attachment #209220 - Flags: review?(steffen.wilberg)
Attachment #209220 - Flags: review+
Attachment #209220 - Flags: approval1.8.1?

Updated

12 years ago
Assignee: nobody → gidsgoldberg

Comment 6

12 years ago
Checking in mozilla/browser/locales/en-US/chrome/help/firebird-toc.rdf;
/cvsroot/mozilla/browser/locales/en-US/chrome/help/firebird-toc.rdf,v  <--  firebird-toc.rdf
new revision: 1.43; previous revision: 1.42
done
Checking in mozilla/browser/locales/en-US/chrome/help/menu_reference.xhtml;
/cvsroot/mozilla/browser/locales/en-US/chrome/help/menu_reference.xhtml,v  <--  menu_reference.xhtml
new revision: 1.37; previous revision: 1.36
done
Checking in mozilla/browser/locales/en-US/chrome/help/shortcuts.xhtml;
/cvsroot/mozilla/browser/locales/en-US/chrome/help/shortcuts.xhtml,v  <--  shortcuts.xhtml
new revision: 1.42; previous revision: 1.41
done
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Updated

12 years ago
Attachment #209220 - Flags: approval1.8.1? → branch-1.8.1?(benjamin)

Comment 7

12 years ago
Comment on attachment 209220 [details] [diff] [review]
Removes references to 'read mail' and 'new message' 

This will end up auto-committed by mirroring I suspect, but a=me
Attachment #209220 - Flags: branch-1.8.1?(benjamin) → branch-1.8.1+

Comment 8

12 years ago
1.8 branch:

Checking in mozilla/browser/locales/en-US/chrome/help/firebird-toc.rdf;
/cvsroot/mozilla/browser/locales/en-US/chrome/help/firebird-toc.rdf,v  <--  firebird-toc.rdf
new revision: 1.34.2.8; previous revision: 1.34.2.7
done
Checking in mozilla/browser/locales/en-US/chrome/help/menu_reference.xhtml;
/cvsroot/mozilla/browser/locales/en-US/chrome/help/menu_reference.xhtml,v  <--  menu_reference.xhtml
new revision: 1.31.2.6; previous revision: 1.31.2.5
done
Checking in mozilla/browser/locales/en-US/chrome/help/shortcuts.xhtml;
/cvsroot/mozilla/browser/locales/en-US/chrome/help/shortcuts.xhtml,v  <--  shortcuts.xhtml
new revision: 1.34.4.8; previous revision: 1.34.4.7
Keywords: fixed1.8.1
Target Milestone: Firefox 2 → Firefox 2 alpha1
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.