Closed
Bug 286455
Opened 20 years ago
Closed 19 years ago
Can only see events from one calendar
Categories
(Calendar :: Sunbird Only, defect)
Calendar
Sunbird Only
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mvl, Assigned: mvl)
Details
Attachments
(1 file)
|
19.91 KB,
patch
|
vlad
:
first-review+
pavlov
:
second-review+
|
Details | Diff | Splinter Review |
The display should get a composite calendar to get the events from all calendars, not just the selected one.
| Assignee | ||
Comment 1•20 years ago
|
||
first step patch. This fixes some errors in the composite calendar, uses a global in calendar.js to get it and makes the views use it. (there are things left to be done, like coloring the events etc)
Attachment #177640 -
Flags: first-review?(vladimir)
Comment on attachment 177640 [details] [diff] [review] first step (checked in) Looks good to me, passing on to stuart for more detailed review of the resources/content stuff. r=vladimir
Attachment #177640 -
Flags: second-review?(pavlov)
Attachment #177640 -
Flags: first-review?(vladimir)
Attachment #177640 -
Flags: first-review+
Comment 3•20 years ago
|
||
Comment on attachment 177640 [details] [diff] [review] first step (checked in) Seems like good progress. How do you plan on storing it so that only certain calendars are part of the composite calendar? As it stands now, any calendar you've created will be part of it and will show up.
Attachment #177640 -
Flags: second-review?(pavlov) → second-review+
| Assignee | ||
Updated•20 years ago
|
Attachment #177640 -
Attachment description: first step → first step (checked in)
| Assignee | ||
Comment 4•20 years ago
|
||
(In reply to comment #3) > How do you plan on storing it so that only certain > calendars are part of the composite calendar? For now using calICalendarManager.setCalendarPref. It's not the best solution, since it is a front-end thing, and not something for the manager, but it's the best I can think of. If we ever allow multiple displays, we can use the display-id to the pref name (or something to that effect)
| Assignee | ||
Comment 5•19 years ago
|
||
This works now. marking fixed.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
QA Contact: gurganbl → sunbird
You need to log in
before you can comment on or make changes to this bug.
Description
•