[Gal] Fixes for #331400

"simon.zheng" <[email protected]>
Newsgroups gmane.comp.gnome.evolution.patches
Message-ID <1141129063.1207.88.camel@fulltime>
Hi Varadhan,

Bug #331400 - Accented characters are displayed incorrectly in Tasks
print preview.

As discussion at IRC, adding some comments in patch.
Resending the patch for your review.

Thanks,
-Simon

_______________________________________________
Evolution-patches mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/evolution-patches
evolution-331400-cell-text-print-height.diff (text/x-patch, 1.4 KB)
Index: table/ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution/widgets/table/ChangeLog,v
retrieving revision 1.20
diff -u -p -r1.20 ChangeLog
--- table/ChangeLog	24 Feb 2006 07:56:58 -0000	1.20
+++ table/ChangeLog	28 Feb 2006 12:09:21 -0000
@@ -1,3 +1,12 @@
+2006-02-28  Simon Zheng  <[email protected]>
+
+	** Fixes bug #331400
+
+	* e-cell-text.c: (ect_print_height): Table item height is decided 
+	by the max cell height in item. And cell-text is just one of cells.
+	As done by ect_print(), to leave some margin for text, 2 for footer,
+	2 for header, actual print height should be 16 + 4. 
+
 2006-02-24  Srinivasa Ragavan  <[email protected]>
 
 	** Fixes bug #332408
Index: table/e-cell-text.c
===================================================================
RCS file: /cvs/gnome/evolution/widgets/table/e-cell-text.c,v
retrieving revision 1.144
diff -u -p -r1.144 e-cell-text.c
--- table/e-cell-text.c	30 Jan 2006 12:28:20 -0000	1.144
+++ table/e-cell-text.c	28 Feb 2006 12:09:21 -0000
@@ -1370,7 +1370,12 @@ ect_print_height (ECellView *ecell_view,
 		  int model_col, int view_col, int row,
 		  double width)
 {
-	return 16;
+	/* 
+	 * Font size is 16 by default. To leave some margin for cell 
+	 * text area, 2 for footer, 2 for header, actual print height 
+	 * should be 16 + 4.
+`	 */
+	return 16 + 4;
 }
 
 static int
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.