Closed Bug 349048 Opened 16 years ago Closed 16 years ago

Lightning does not work if the old calendar extension is also installed


(Calendar :: Lightning Only, defect)

Not set


(Not tracked)



(Reporter: jminta, Assigned: jminta)




(1 obsolete file)

If you install Lightning in a Thunderbird profile that already has the calendar extension installed, Lightning will not work correctly.  Symptoms include: large red 'undefined entity' bar at the bottom of the thunderbird window and many javascript errors.

Lightning should notify users of the conflict and disable the extension for them.
Attached patch check and disable (obsolete) — Splinter Review
This patch checks for the presence of a conflict and if it finds one, disables the old extension.  I'm going to try to work on migrating that data later today.
Assignee: nobody → jminta
Attachment #234260 - Flags: second-review?(dmose)
Attachment #234260 - Flags: first-review?(mattwillis)
Comment on attachment 234260 [details] [diff] [review]
check and disable

>Index: calendar/lightning/content/ltnCompatibility.xul
>+   - die as a result of the conflict (including messanger-overlay-sidebar.js).
Typo: messenger

I'm not sure I dig the filename, as we're not abbreviating things like that currently.
Could it instead be named something about disabling the old extension?

>Index: calendar/locales/en-US/chrome/lightning/
>+disableExtMessage=You currently have the old calendar extension installed.  This extension and lightning are not compatible.  Disable the old extension?
Capitalize Lightning?

>+disableExtDone=The old calendar extension has been disabled. Thunderbird will now restart.
You probably want to use @MOZ_APP_DISPLAYNAME@ or something instead of hardcoding Thunderbird.

>+disableDoneTitle=Old Extension disabled
Pick a capitalization for Old Calendar Extension. I don't care which, but be consistent.

r1=lilmatt with those fixed.
Attachment #234260 - Flags: first-review?(mattwillis) → first-review+
Comment on attachment 234260 [details] [diff] [review]
check and disable

It seems to me that this patch asks the user to make a choice, but gives them insufficient information about why they would want to choose one way or the other.  In particular, I assume that the only reason that one might want to stick with the old extension is because no automatic data migration is provided from old to new.
Attachment #234260 - Flags: second-review?(dmose) → second-review-
After talking this over with dmose, we decided that we should land the migration stuff first, so that there's no need to present the user with a choice.  Just disabling the old extension would be fine then.
Depends on: 349586
After installing Lightning RC2 over Thundrebird with calendar extension, the layout of the Thunderbird became altered. In the maximized view, the bottom bar (the gray area containing the number of unread mail, the total number of messages etc.) became enlarged occupying the 10-15% of the screen.
I have a laptop w/ SXGA TFT screen using the 1024x768 resolution.  
(In reply to comment #5)

That is the reason why you must disable/uninstall the old Calendar extension as noted on [].
Attachment #234260 - Attachment is obsolete: true
Fixed by checkin for bug 349586.

Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.