Closed Bug 500488 Opened 16 years ago Closed 16 years ago

Can't open Calendar and Tasks tabs; adopt Lightning to openTab API changes in Bug 467768

Categories

(Calendar :: Lightning Only, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ssitter, Assigned: Fallen)

References

Details

(Whiteboard: [needed beta][no l10n impact])

Attachments

(1 file)

According to the announcement in <http://groups.google.com/group/mozilla.dev.apps.thunderbird/msg/5396616f6252b4d6> the openTab API will be changed with Bug 467768. It should be checked if Lightning requires update: http://mxr.mozilla.org/comm-central/search?string=opentab&find=/calendar/
Flags: blocking-calendar1.0?
Flags: blocking-calendar1.0? → blocking-calendar1.0+
Whiteboard: [not needed beta][no l10n impact]
Seems this is definitely needed.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Whiteboard: [not needed beta][no l10n impact] → [needed beta][no l10n impact]
Attached patch Fix - v1 (deleted) — Splinter Review
This should take care
Attachment #385707 - Flags: review?(mschroeder)
Just another change in Thunderbird that requires Lightning to drop support for 3.0b2 or to implement a solution that works in both 3.0b2 and 3.0b3pre.
Depends on: 467768
Comment on attachment 385707 [details] [diff] [review] Fix - v1 Works like a charm! r=mschroeder Just one nit: > <command id="calendar_go_to_today_command" >- oncommand="document.getElementById('tabmail').openTab('calendar', document.getElementById('calendar-tab-button').getAttribute('tooltiptext')); goToDate(now())"/> >+ oncommand="document.getElementById('tabmail').openTab('calendar', { title: document.getElementById('calendar-tab-button').getAttribute('tooltiptext') }); goToDate(now())"/> > </commandset> Please, fix the indentation.
Attachment #385707 - Flags: review?(mschroeder) → review+
(In reply to comment #4) > Just another change in Thunderbird that requires Lightning to drop support for > 3.0b2 or to implement a solution that works in both 3.0b2 and 3.0b3pre. Just to be clear, the API change still allows lightning to work in 3.0b2 as well as 3.0b3pre as long as the consumers of the lightning openTab code are modified simultaneously. (The use of the object merely becomes an odd convention in 3.0b2 rather than required.) This would potentially cause breakage for extensions that utilize the tabmail API to make lightning do things since it is harder for them to make changes in lock-step with lightning.
Summary: Adopt Lightning to openTab API changes in Bug 467768 → Can't open Calendar and Tasks tabs; adopt Lightning to openTab API changes in Bug 467768
Yes, I think we are fine, this shouldn't break anything. Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/370680198dec> -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
Strange mercurial things happened, I needed to repush the nit fixes http://hg.mozilla.org/comm-central/rev/9f2a0925e9cf
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: