Move printDialog to calendar/base/content

RESOLVED FIXED in 1.0b1

Status

Calendar
Printing
--
trivial
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: mschroeder, Assigned: mschroeder)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
Created attachment 401766 [details] [diff] [review]
Patch v1

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+
(Assignee)

Updated

8 years ago
Duplicate of this bug: 520389
(Assignee)

Comment 3

8 years ago
http://hg.mozilla.org/comm-central/rev/1cee73f0baea
http://hg.mozilla.org/releases/comm-1.9.1/rev/ff573f5bced9
Status: ASSIGNED → RESOLVED
Last Resolved: 8 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.