Bug 1631902 Comment 11 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

- Show a summary of each event and task in the ICS file that's being imported in the ICS file import dialog.
- Allow each event or task to be imported individually (in addition to import of the whole file).

Something I just realized when testing this is that our ICS import code doesn't import tasks only events.  So we should either 1. make it import tasks, or 2. change the wording of this dialog so it doesn't claim that tasks can be imported.  I'd say 1 is preferable.  If we do 1, that could be a follow-up. If 2 then we should do it as part of this patch to avoid unnecessary string churn.

Richard, flagging you for feedback on CSS changes.  Namely, now that we are using the "calendar summary dialog" CSS for this as well, maybe some re-organization is in order?  (I'm sure the aesthetics can be improved as well, but we could do that as a follow-up, when there's time, what with everything needing to be done before the 78 release.)  I'll upload some screenshots.
- Show a summary of each event and task in the ICS file that's being imported in the ICS file import dialog.
- Allow each event or task to be imported individually (in addition to import of the whole file).

~~Something I just realized when testing this is that our ICS import code doesn't import tasks only events.  So we should either 1. make it import tasks, or 2. change the wording of this dialog so it doesn't claim that tasks can be imported.  I'd say 1 is preferable.  If we do 1, that could be a follow-up. If 2 then we should do it as part of this patch to avoid unnecessary string churn.~~  Edit: the tasks are imported, I just hadn't looked for them hard enough.

Richard, flagging you for feedback on CSS changes.  Namely, now that we are using the "calendar summary dialog" CSS for this as well, maybe some re-organization is in order?  (I'm sure the aesthetics can be improved as well, but we could do that as a follow-up, when there's time, what with everything needing to be done before the 78 release.)  I'll upload some screenshots.

Back to Bug 1631902 Comment 11