Closed Bug 518865 Opened 15 years ago Closed 15 years ago

Calendar Properties item missing from the SeaMonkey Edit menu.

Categories

(Calendar :: Lightning: SeaMonkey Integration, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

(Whiteboard: [comm-1.9.1 only])

Attachments

(1 file)

http://mxr.mozilla.org/comm-central/source/calendar/lightning/content/messenger-overlay-sidebar.xul#125

Thunderbird uses "menu_EditPopup" while SeaMonkey uses "menu_Edit_Popup"
Includes attachment 405777 [details] [diff] [review] from Bug 516882 Comment 9. See also Bug 516882 Comment 7. If this is approved then we can remove the hack we added to SeaMonkey to hide the new message menu item. Later we can add more SeaMonkey specific adjustments to this file.
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #406686 - Flags: review?(mschroeder)
(In reply to comment #0)
> http://mxr.mozilla.org/comm-central/source/calendar/lightning/content/messenger-overlay-sidebar.xul#125
> 
> Thunderbird uses "menu_EditPopup" while SeaMonkey uses "menu_Edit_Popup"

Wouldn't it be easier to just change SM to use "menu_EditPopup" rather than having a separate overlay?
> Wouldn't it be easier to just change SM to use "menu_EditPopup" rather than
> having a separate overlay?

Unfortunate RC2 is out, or should be out by now.
Might the SeaMonkey Council accept this change for 2.0.1?
I fear potentially breaking extensions between 2.0.x versions. This might have been easier if it had come up before freezing up that much and going into final RC.
(In reply to comment #5)
> I fear potentially breaking extensions between 2.0.x versions. This might have
> been easier if it had come up before freezing up that much and going into final
> RC.
Potentially it means extensions have to do less to make themselves compatible with both TB and SM as they can use the same overlay for both when it comes to the edit menu.
Ian, I fully understand that, but breaking any extensions in security/stability updates is a very bad idea.
(In reply to comment #7)
> Ian, I fully understand that, but breaking any extensions in security/stability
> updates is a very bad idea.
Do we know if any, other than Lightning, currently overlay the Edit menu?
I'd be expecting any change to an id to be in 2.1 rather than 2.0.x anyway, my quoting you previously probably implied 2.0.x though, sorry. I was replying to the more general reason why we'd want to change the id.
I'm perfectly happy with doing that id change in 2.1, it's 2.0.x what I was sensible to ;-)

Unfortunately, I don't think we have any reasonable possibility right now to find out who else might be using that ID :(
1. Actually this ID is shared with Navigator via an overlay so there are three places where we have to change it.
2. This will actually make it easier for Firefox extensions to be ported to SeaMonkey as Thunderbird was just following the TB lead.
3. The rest of /suite/ actually uses menu_EditPopup.

I'd vote for this making 2.1. In the mean time we still want this fix in Lighting for SeaMonkey 2.0x
I wonder if would be a worthwhile endeavour to standardize a set of common (menuitem) ids across Mozilla-based applications to ease the work of extension authors...
Comment on attachment 406686 [details] [diff] [review]
Patch v1.0 Add Edit->Calendar Properties menu item to SeaMonkey [comm-1.9.1 Checkin: Comment 16]

r=mschroeder. Looks good (from a Calendar Project pov), but maybe a SeaMonkey reviewer should also have a look at this patch. One nit: We have suite-overlay-messenger.xul and communicator-overlay-preferences.xul in lightning/content with this patch. You should decide which prefix SeaMonkey related files should use, suite or communicator.
Attachment #406686 - Flags: review?(mschroeder) → review+
I'd prefer to go for suite everywhere, communicator is a Netscape-ism only, IMHO.
Comment on attachment 406686 [details] [diff] [review]
Patch v1.0 Add Edit->Calendar Properties menu item to SeaMonkey [comm-1.9.1 Checkin: Comment 16]

> r=mschroeder. Looks good (from a Calendar Project pov), but maybe a SeaMonkey
> reviewer should also have a look at this patch.

Thanks! Asking for additional review from IanN
Attachment #406686 - Flags: review?(iann_bugzilla)
Comment on attachment 406686 [details] [diff] [review]
Patch v1.0 Add Edit->Calendar Properties menu item to SeaMonkey [comm-1.9.1 Checkin: Comment 16]

r=me but only for comm-1.9.1, on comm-central we can, with suitable notification to extension developers, change the id for the edit menu on SeaMonkey.
Attachment #406686 - Flags: review?(iann_bugzilla) → review+
Keywords: checkin-needed
Comment on attachment 406686 [details] [diff] [review]
Patch v1.0 Add Edit->Calendar Properties menu item to SeaMonkey [comm-1.9.1 Checkin: Comment 16]

http://hg.mozilla.org/releases/comm-1.9.1/rev/34a5ec85f7fc
Attachment #406686 - Attachment description: Patch v1.0 Add Edit->Calendar Properties menu item to SeaMonkey → Patch v1.0 Add Edit->Calendar Properties menu item to SeaMonkey [comm-1.9.1 Checkin: Comment 16]
Keywords: checkin-needed
Will you use this bug or another for switching SM to use the same ID for the Edit menu as TB?
> Will you use this bug or another for switching SM to use the same ID for the
> Edit menu as TB?

I'll file another bug under the SeaMonkey product since the flags there are more suitable.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Please file also a bug for renaming communicator-overlay-preferences.xul as discussed in comment#12 and comment#13.
OS: Windows XP → All
Hardware: x86 → All
Whiteboard: [comm-1.9.1 only]
Target Milestone: --- → 1.0
Blocks: 526171
Blocks: 526172
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: