Closed Bug 347790 Opened 16 years ago Closed 16 years ago

[printing] title is always printed as HTMLTitle

Categories

(Calendar :: Printing, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: damian.publicemail, Assigned: jminta)

Details

(Keywords: APIchange)

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; pl; rv:1.8.0.6) Gecko/20060728 Firefox/1.5.0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060807 Calendar/0.3a2+

Whatever you type in title inbox at print dialog you alwas see HTMLTitle

Reproducible: Always

Steps to Reproduce:
1. press Ctrl+P
2. type complete title in print dialog
3. Press OK
4. next dialog, press "print" button
   -> instead your title you get HTMLTitle

Actual Results:  
title is predefined can not be changed

Expected Results:  
see screenshot
Version: unspecified → Trunk
Attached image screenshot
This would probably block 0.3 if we can get it confirmed. ->qa
Keywords: qawanted
Confirmed during all my printing-fu.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking0.3+
OS: Windows XP → All
Hardware: PC → All
Keywords: qawanted
Patch adds a title argument that we can pass to our print-formatters.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #232970 - Flags: second-review?(dmose)
Attachment #232970 - Flags: first-review?(mattwillis)
Comment on attachment 232970 [details] [diff] [review]
pass the title around

This looks good. I actually will use all of this except the calPrintEngine.js piece in my other bug, so landing this would be good.

r=lilmatt
Attachment #232970 - Flags: first-review?(mattwillis) → first-review+
Comment on attachment 232970 [details] [diff] [review]
pass the title around

-                      [array, size_is(aCount)] in calIItemBase aItems);
+                      [array, size_is(aCount)] in calIItemBase aItems,
+                      in AString aTitle);
At least at an @param comment for the one you're adding. x2

r=dmose with that
Attachment #232970 - Flags: second-review?(dmose) → second-review+
Patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Keywords: APIchange
nice, even polish letters are printed properly

verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.