Port |Bug 526998 - Implement F2 keyboard shortcut for renaming focused attachments when composing (on Windows and Unix)|

RESOLVED FIXED in seamonkey2.9

Status

SeaMonkey
MailNews: Composition
--
enhancement
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.9
x86_64
Windows 7

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 588751 [details] [diff] [review]
patch [Checkin: Comment 5]

SSIA. Differences to what they did for TB:

* TB uses preprocessing, I made use of our platformMailnewsOverlay to exclude Mac
* they rewrote the whole controllers, I sticked to what we always do
* I didn't add Rename Attachment to the menus or change any l10n
* I didn't remove the onkeypress line from the attachmentlist (should I?)
Attachment #588751 - Flags: superreview?(neil)
Attachment #588751 - Flags: review?(mnyromyr)

Comment 1

5 years ago
(In reply to Jens Hatlak from comment #0)
> * TB uses preprocessing, I made use of our platformMailnewsOverlay to exclude Mac
Of course :-)

> * they rewrote the whole controllers, I sticked to what we always do
If we were to go with anything like their system, I would actually have a JS object that converts from the command-object-object to the controller object, rather than duplicating the supports/do/isCommandEnabled logic.

> * I didn't add Rename Attachment to the menus or change any l10n
> * I didn't remove the onkeypress line from the attachmentlist (should I?)
Doesn't seem relevant to the keyboard shortcut, so followup bugs, perhaps.
The controller refactoring isn't really relevant either, but it was needed anyway to fix pressing delete with the From field focused ;-)

Updated

5 years ago
Attachment #588751 - Flags: superreview?(neil) → superreview+

Comment 2

5 years ago
> Bug 718310 - Port |Bug 526998
Bug 526998 got backed out.
New patch in attachment 588761 [details] [diff] [review]
Interdiff in attachment 588763 [details] [diff] [review]
(Assignee)

Comment 3

5 years ago
(In reply to Philip Chee from comment #2)
> > Bug 718310 - Port |Bug 526998
> Bug 526998 got backed out.

Yeah I know. We don't have test that was failing (and needed to be adapted) and I didn't introduce new strings, so all is well. :-)

Comment 4

5 years ago
Comment on attachment 588751 [details] [diff] [review]
patch [Checkin: Comment 5]

>+var attachmentBucketController =

Since controllers are global objects, (new) controllers' names should have the g prefix, i.e. gAttachmentBucketController.

r=me with that.
Attachment #588751 - Flags: review?(mnyromyr) → review+
(Assignee)

Comment 5

5 years ago
Comment on attachment 588751 [details] [diff] [review]
patch [Checkin: Comment 5]

http://hg.mozilla.org/comm-central/rev/e51b6e8ee03b
Attachment #588751 - Attachment description: patch → patch [Checkin: Comment 5]
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.9
You need to log in before you can comment on or make changes to this bug.