Closed Bug 533580 Opened 15 years ago Closed 15 years ago

strip busted tbird support

Categories

(Cloud Services :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mconnor, Assigned: mconnor)

References

Details

Attachments

(1 file)

We should figure out what, if anything, should be synced with Thunderbird, but not for 1.0.
Flags: blocking-weave1.0+
http://hg.mozilla.org/labs/weave/rev/5519943c948d If there's interest in doing something interesting here at some point, someone should file a bug.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
The sync.xul change broke SeaMonkey integration. :-( Clicking on the Weave status bar entry in SeaMonkey (using left or right mouse click) now triggers this in the Error Console with Weave 1.0b3: Error: menuPopup is null Source: chrome://weave/content/sync.js Line: 381 which is if (menuPopup.parentNode != menuButton) where menuPopup is var menuPopup = document.getElementById('sync-menu-popup'); cf. sync.xul for the rest. Granted, the comment said it's only for TB... Mike, can you please revert that change (adapting the comment to read SeaMonkey)? I have no Weave source tree around so it would require me some time to prepare a clean patch.
This fixes the SeaMonkey Tools menu item for me.
Attachment #417271 - Flags: review?(mconnor)
Comment on attachment 417271 [details] [diff] [review] patch Sv1.0 Fix SeaMonkey breakage. Heh, oops.
Attachment #417271 - Flags: review?(mconnor) → review+
Keywords: checkin-needed
Blocks: 534789
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: