Closed Bug 444822 Opened 12 years ago Closed 12 years ago

Add documentation to UI code and some UI code fixes

Categories

(Calendar :: Calendar Views, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

References

Details

Attachments

(1 file)

This patch moves around some UI code to a more appropriate place and adds some documentation. This was originally part of the alarms patch.
Attachment #329129 - Flags: review?(Berend.Cornelius)
Blocks: 353492
Comment on attachment 329129 [details] [diff] [review]
UI code fixes - v1

>-    var calendarToUse = aCalendarToUse || aItem.calendar;
>-    var calendars = getCalendarManager().getCalendars({});
>-    var indexToSelect = 0;
>-    var index = -1;
>+var calendarToUse = aCalendarToUse || aItem.calendar;
>+var calendars = getCalendarManager().getCalendars({});
>+var indexToSelect = 0;
>+var index = -1;

Why did you remove the indentation here? I think it's not conforming to our style guide and IMO should also not be taken over there as we do declaration and definition in one step.

"CreateAlarm" is never used in the project as I noticed. I don't mind providing it anyway but is there an important member that we should deliver (like the "icalString" for your "createDuration()" function)?
Patch is a good cleanup. r=berend
Attachment #329129 - Flags: review?(Berend.Cornelius) → review+
(In reply to comment #1)
> Why did you remove the indentation here? I think it's not conforming to our
> style guide and IMO should also not be taken over there as we do declaration
> and definition in one step.
Good catch, wasn't done on purpose.

> "CreateAlarm" is never used in the project as I noticed. I don't mind providing
> it anyway but is there an important member that we should deliver (like the
> "icalString" for your "createDuration()" function)?
> Patch is a good cleanup. r=berend
No parameters needed for now.


Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9
Checked via mxr.mozilla.com -> VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.