Closed Bug 765600 Opened 12 years ago Closed 12 years ago

Node cannot be inserted at the specified point in attendees dialog

Categories

(Calendar :: Dialogs, defect)

Lightning 1.5
defect
Not set
blocker

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Paenglab)

References

Details

Attachments

(1 file, 1 obsolete file)

I've had 2 user reports of this error when opening the atendees dialog. This is said to happen in 1.6b1 and 1.5 (and maybe earlier).

uncaught exception: [Exception... "Node cannot be inserted at the specified point in the hierarchy" code: "3" nsresult: "0x80530003 (HierarchyRequestError)" location: "chrome://calendar/content/calendar-event-dialog-attendees.js Line: 85"]

Also, while we are at it:

Warning: Error in parsing value for 'min-height', Declaration dropped. chrome://calendar/skin/calendar-event-dialog.css.
(In reply to Philipp Kewisch [:Fallen] from comment #0)
> Also, while we are at it:
> 
> Warning: Error in parsing value for 'min-height', Declaration dropped.
> chrome://calendar/skin/calendar-event-dialog.css.

This was fixed in Bug 758529 for Lightning 1.7. Maybe this can be pushed to 1.6.
Might be related to or be the cause of bug 765501 and bug 765530.
Thunderbird 12 + Lightning 1.4 doesn't show the error message. 
Therefore either caused by a change in Thunderbird 13 or Lightning 1.5.
I got another email and was told the user has downgraded Lightning from 1.6b1 / 1.5.1 to 1.5, where the issue doesn't show up. I guess its that one patch we slipped in to 1.5.1...
Blocks: 765530
At least 4 more reports via email. It is indeed the bug we slipped in, bug 762861. Due to the @import rule being added, the style rule in calendar-event-dialog-attendees.css line 85 can't be inserted due to being at the wrong place.

Two options here:

* Backout the bug we slipped in
* Attempt to fix it, possibly forcing insertation at the end of the stylesheet.
Severity: normal → blocker
Bug 762861 is only a cosmetic issue. I'd suggest to backout the patch on all branches to restore functionality and reopen Bug 762861 for a proper and tested fix.
Attached patch patch (obsolete) β€” β€” Splinter Review
Sorry to make such troubles. This patch reverts the change and adds the css  through the manifest. I tested it under 1.5.1 and it works without the nasty bug.
Attachment #634074 - Flags: review?(philipp)
Comment on attachment 634074 [details] [diff] [review]
patch

How much tested is that patch? I can't test it since I'm now also having compile issues on windows.

I'm going to defer to Stefan to decide if we should rather backout or push this fix for 1.5.2.
Attachment #634074 - Flags: review?(philipp) → review?(ssitter)
I tested it under XP and Win7. Invited a person and checked the icons in customize window on different toolbars.
Comment on attachment 634074 [details] [diff] [review]
patch

For a quick 1.5.2 bug fix release I'd prefer to just backout the causing patch.

All other calendar-windows overlay in the modified jar.mn use os=WINNT. 
Why is it not required for the new added line?
Attached patch Patch with os=WINNT β€” β€” Splinter Review
(In reply to Stefan Sitter from comment #10)
> Comment on attachment 634074 [details] [diff] [review]
> patch
> 
> All other calendar-windows overlay in the modified jar.mn use os=WINNT. 
> Why is it not required for the new added line?

Because the lines are in a #ifdef XP_WIN they only apply to Windows so  os=WINNT isn't really needed. But for the future with one XPI for all platforms it's better now set and makes then in the future no problems.
Attachment #634074 - Attachment is obsolete: true
Attachment #634074 - Flags: review?(ssitter)
Attachment #634569 - Flags: review?(ssitter)
Attachment #634569 - Flags: review?(ssitter)
Attachment #634569 - Flags: review+
Attachment #634569 - Flags: approval-calendar-release?
Attachment #634569 - Flags: approval-calendar-beta?
Attachment #634569 - Flags: approval-calendar-aurora?
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Comment on attachment 634569 [details] [diff] [review]
Patch with os=WINNT

Hmm so I guess you requesting approval for release means you think we can take the patch regardless of what you said in comment 10?
Attachment #634569 - Flags: approval-calendar-release?
Attachment #634569 - Flags: approval-calendar-release-
Attachment #634569 - Flags: approval-calendar-beta?
Attachment #634569 - Flags: approval-calendar-beta+
Attachment #634569 - Flags: approval-calendar-aurora?
Attachment #634569 - Flags: approval-calendar-aurora+
Comment on attachment 634569 [details] [diff] [review]
Patch with os=WINNT

Hmm so I guess you requesting approval for release means you think we can take the patch regardless of what you said in comment 10?
Attachment #634569 - Flags: approval-calendar-release- → approval-calendar-release+
Target Milestone: --- → 1.5
You need to log in before you can comment on or make changes to this bug.