Closed Bug 517805 Opened 10 years ago Closed 10 years ago

Move printDialog to calendar/base/content

Categories

(Calendar :: Printing, defect, trivial)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: martinschroeder, Assigned: martinschroeder)

References

Details

Attachments

(1 file)

Attached patch Patch v1Splinter Review
Changes in this patch:

* Cleanup of printDialog.xul/js: braces, space, indentation, use 'let'

* Remove unused vars gPrintSettings and contractids (printDialog.js)

* Replace usage of string bundle with calGetString(...)

* Remove unused stylesheets, overlay and CSS class (printDialog.xul)

* Move printDialog.xul/js to calendar-print-dialog.xul/js in calendar/base/content/dialogs/ and change jar.mn & calUtils.js accordingly.
Attachment #401766 - Flags: review?(philipp)
Comment on attachment 401766 [details] [diff] [review]
Patch v1

>+    let catenum = catman.enumerateCategory("cal-print-formatters");
> 
>     // Walk the list, adding items to the layout menupopup
>+    let layoutList = document.getElementById("layout-field");
>     while (catenum.hasMoreElements()) {
>+        let entry = catenum.getNext();
You may want to use iteratorUtils here, then you can just do for each (fixIterator(catman.enumerateCategory("cal-print-formatters")) {} or similar. Not sure if we've included this into Sunbird yet though.


Anyway, r=philipp
Attachment #401766 - Flags: review?(philipp) → review+
Duplicate of this bug: 520389
http://hg.mozilla.org/comm-central/rev/1cee73f0baea
http://hg.mozilla.org/releases/comm-1.9.1/rev/ff573f5bced9
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.