This prevents harcoding the depth, and uses the one from the gdk
window.  For this, the preview has to be done after the window is
realized, so the code is wrapped in the expose-event callback.

Also add a white background to the preview, otherwise artifacts appear
for previews smaller than the preview box.

Signed-off-by: Manuel Quiñones <ma...@laptop.org>
---
 src/jarabe/journal/expandedentry.py | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/jarabe/journal/expandedentry.py 
b/src/jarabe/journal/expandedentry.py
index e0c603f..d0c7166 100644
--- a/src/jarabe/journal/expandedentry.py
+++ b/src/jarabe/journal/expandedentry.py
@@ -194,11 +194,18 @@ class ExpandedEntry(gtk.EventBox):
         return date
 
     def _create_preview(self):
-        width = style.zoom(320)
-        height = style.zoom(240)
         box = gtk.EventBox()
         box.modify_bg(gtk.STATE_NORMAL, style.COLOR_WHITE.get_gdk_color())
 
+        box.connect('expose-event', self.__expose_event_cb)
+        box.connect_after('button-release-event',
+                          self._preview_box_button_release_event_cb)
+        return box
+
+    def __expose_event_cb(self, box, event):
+        width = style.zoom(320)
+        height = style.zoom(240)
+
         if len(self._metadata.get('preview', '')) > 4:
             if self._metadata['preview'][1:4] == 'PNG':
                 preview_data = self._metadata['preview']
@@ -214,8 +221,11 @@ class ExpandedEntry(gtk.EventBox):
                 surface = cairo.ImageSurface.create_from_png(png_file)
                 png_width = surface.get_width()
                 png_height = surface.get_height()
-                pixmap = gtk.gdk.Pixmap(None, png_width, png_height, 24)
+                gdk_window = self.get_toplevel().window
+                pixmap = gtk.gdk.Pixmap(gdk_window, png_width, png_height, -1)
                 cr = pixmap.cairo_create()
+                cr.set_source_rgb(1, 1, 1)
+                cr.paint()
                 cr.set_source_surface(surface, 0, 0)
                 cr.scale(width / png_width, height / png_height)
                 cr.paint()
@@ -230,15 +240,13 @@ class ExpandedEntry(gtk.EventBox):
 
         if has_preview:
             box.add(im)
+            im.show()
         else:
             label = gtk.Label()
             label.set_text(_('No preview'))
             label.set_size_request(width, height)
             box.add(label)
-
-        box.connect_after('button-release-event',
-                          self._preview_box_button_release_event_cb)
-        return box
+            label.show()
 
     def _create_technical(self):
         vbox = gtk.VBox()
-- 
1.7.11.2

_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to