Closed Bug 663400 Opened 14 years ago Closed 14 years ago

Changing view causes an error "this.tabbox is null"

Categories

(Calendar :: Calendar Frontend, defect)

All
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bv1578, Assigned: bv1578)

Details

Attachments

(1 file)

I was quite sure this bug had already been reported but I can't find it. Steps to reproduce: 1. in calendar view click on a different tab in order to switch view (day, week, multiweek, month), sometimes more than once; -> in the error console appears the error: Error: this.tabbox is null Source File: chrome://global/content/bindings/tabbox.xml Line: 476 Maybe related to bug 596759?
I get the error causing ALL calendar events not being displayed in week view (other views are correctly rendered). All worked fine before upgrading. (I use Thunderbird 5.0 + lightning 1.04b)
Finally, after reloading, the error message remains, but display is now correct.
Attached patch fix - v1Splinter Review
Patch for bug 596759 highlighted the issue. This patch fixes, but I'm not sure whether it's formally corrected to leave the box with class="navigation-bottombox" inside the tabbox.
Assignee: nobody → bv1578
Status: NEW → ASSIGNED
Attachment #569526 - Flags: review?(philipp)
Comment on attachment 569526 [details] [diff] [review] fix - v1 What we are doing with those tabs isn't formally correct anyways ;-) The <tabbox> should have a <tabpanels> child, so don't worry about the bottombox. I've locally added a few style fixes for mac, there are some mac-specific tabbox rules that need overwriting. I've looked into windows/linux themes and they don't have such rules.
Attachment #569526 - Flags: review?(philipp) → review+
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0b8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: