Author: fejj
Date: 2007-06-25 13:17:04 -0400 (Mon, 25 Jun 2007)
New Revision: 80697

Modified:
   trunk/moon/src/text.cpp
Log:
added comments about why we don't use a single PangoLayout yet



Modified: trunk/moon/src/text.cpp
===================================================================
--- trunk/moon/src/text.cpp     2007-06-25 17:11:48 UTC (rev 80696)
+++ trunk/moon/src/text.cpp     2007-06-25 17:17:04 UTC (rev 80697)
@@ -542,6 +542,12 @@
        char *text;
        int w, h;
        
+       // FIXME: we could use a single PangoLayout for getting
+       // overall width/height if we created a proper
+       // PangoAttrList. However, we can't really do this until we
+       // find a way of changing the Foreground brush for runs if
+       // they override it.
+       
        text = text_block_get_text (this);
        
        if (layout == NULL) {
@@ -707,6 +713,12 @@
        if (block_width < 0.0)
                CalcActualWidthHeight (cr);
        
+       // FIXME: we could use a single PangoLayout for drawing if we
+       // created a proper PangoAttrList. However, we can't really do
+       // this until we find a way of changing the Foreground brush
+       // for runs if they override it - until then, we have to lay
+       // out the text/runs manually :(
+       
        font_mask = pango_font_description_get_set_fields (font);
        
        if (foreground == NULL) {

_______________________________________________
Mono-patches maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches

Reply via email to