Closed
Bug 309505
Opened 19 years ago
Closed 19 years ago
Switching views in Lightning results in gray screen
Categories
(Calendar :: Lightning Only, defect)
Calendar
Lightning Only
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jminta, Assigned: shaver)
References
(Depends on 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
|
1.29 KB,
patch
|
dmosedale
:
first-review+
|
Details | Diff | Splinter Review |
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.
| Reporter | ||
Comment 1•19 years ago
|
||
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 2•19 years ago
|
||
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-
| Reporter | ||
Comment 3•19 years ago
|
||
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 4•19 years ago
|
||
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+
| Reporter | ||
Comment 5•19 years ago
|
||
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
Updated•19 years ago
|
QA Contact: shaver → lightning
You need to log in
before you can comment on or make changes to this bug.
Description
•