Closed Bug 309505 Opened 19 years ago Closed 19 years ago

Switching views in Lightning results in gray screen

Categories

(Calendar :: Lightning Only, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jminta, Assigned: shaver)

References

(Depends on 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Whenever a user is already viewing a particular calendar screen and then
switches views, the result is simply a gray screen. Switching from a normal
thunderbird email view works fine.
Attached patch uncollapse the view (obsolete) β€” β€” Splinter Review
The problem is that LtnObserveDisplayDeckChange gets called during the
view-drawing.  However, event.target in this case is the actual view, not the
view-box.  This means it fails to match |if (id == "calendar-view-box")| and so
the view-box gets collapsed.  One solution would be to try and tweak the
addEventListener line, but this solution was just too simple to pass up.
Attachment #196932 - Flags: first-review?(dmose)
Comment on attachment 196932 [details] [diff] [review]
uncollapse the view

I agree that it makes sense to workaround the Gecko bug so that Lightning at
least has a chance of working in versions of TBird that don't have this fix. 
However this fix makes for really non-obvious control flow.  Please fix up the
testing in the listener instead, with comments about the bug being worked
around.
Attachment #196932 - Flags: first-review?(dmose) → first-review-
Attached patch modify event listener β€” β€” Splinter Review
version 2 checks to make sure that we have the deck we're actually looking for.
Attachment #196932 - Attachment is obsolete: true
Attachment #197010 - Flags: first-review?(dmose)
Comment on attachment 197010 [details] [diff] [review]
modify event listener

r=dmose; good job tracking this one down!
Attachment #197010 - Flags: first-review?(dmose) → first-review+
patch checked in.  As soon as I find some free time, I'll work on writing up the
gecko bug.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Depends on: 309805
QA Contact: shaver → lightning
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: