From ec803b08152fe76fdf648ab41b280e7acb08a17c Mon Sep 17 00:00:00 2001
From: Brandon Lewis <brandon@collabora.co.uk>
Date: Fri, 2 Jul 2010 12:26:42 +0200
Subject: [PATCH] make sure to set properties on new track objects

---
 ges/ges-timeline-text-overlay.c | 13 ++++++++++---
 ges/ges-timeline-title-source.c | 12 +++++++++---
 2 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/ges/ges-timeline-text-overlay.c b/ges/ges-timeline-text-overlay.c
index 3475210ce8..7de42b8f7e 100644
--- a/ges/ges-timeline-text-overlay.c
+++ b/ges/ges-timeline-text-overlay.c
@@ -175,7 +175,7 @@ ges_tl_text_overlay_class_init (GESTimelineTextOverlayClass * klass)
           "Pango font description of font to be used for rendering. "
           "See documentation of pango_font_description_from_string "
           "for syntax.", DEFAULT_PROP_FONT_DESC,
-          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+          G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS));
 
   /**
    * GESTimelineTextOverlay:valignment
@@ -184,8 +184,9 @@ ges_tl_text_overlay_class_init (GESTimelineTextOverlayClass * klass)
    */
   g_object_class_install_property (G_OBJECT_CLASS (klass), PROP_VALIGNMENT,
       g_param_spec_enum ("valignment", "vertical alignment",
-          "Vertical alignment of the text", GES_TIMELINE_TITLE_SRC_VALIGN_TYPE,
-          DEFAULT_PROP_VALIGNMENT, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+          "Vertical alignment of the text", GES_TEXT_VALIGN_TYPE,
+          DEFAULT_PROP_VALIGNMENT, G_PARAM_READWRITE | G_PARAM_CONSTRUCT |
+          G_PARAM_STATIC_STRINGS));
   /**
    * GESTimelineTextOverlay:halignment
    *
@@ -341,6 +342,12 @@ ges_tl_text_overlay_create_track_object (GESTimelineObject * obj,
     res = (GESTrackObject *) ges_track_text_overlay_new ();
     GST_DEBUG ("Setting text property");
     ges_track_text_overlay_set_text ((GESTrackTextOverlay *) res, tfs->text);
+    ges_track_text_overlay_set_font_desc ((GESTrackTextOverlay *) res,
+        tfs->font_desc);
+    ges_track_text_overlay_set_halignment ((GESTrackTextOverlay *) res,
+        tfs->halign);
+    ges_track_text_overlay_set_valignment ((GESTrackTextOverlay *) res,
+        tfs->valign);
   }
 
   else {
diff --git a/ges/ges-timeline-title-source.c b/ges/ges-timeline-title-source.c
index 32d123f644..2857a91a93 100644
--- a/ges/ges-timeline-title-source.c
+++ b/ges/ges-timeline-title-source.c
@@ -166,7 +166,7 @@ ges_tl_title_src_class_init (GESTimelineTitleSourceClass * klass)
 
   g_object_class_install_property (object_class, PROP_TEXT,
       g_param_spec_string ("text", "Text", "The text to display",
-          DEFAULT_PROP_TEXT, G_PARAM_READWRITE | G_PARAM_CONSTRUCT));
+          DEFAULT_TEXT, G_PARAM_READWRITE | G_PARAM_CONSTRUCT));
 
   /**
    * GESTimelineTitleSource:font-dec
@@ -178,8 +178,8 @@ ges_tl_title_src_class_init (GESTimelineTitleSourceClass * klass)
       g_param_spec_string ("font-desc", "font description",
           "Pango font description of font to be used for rendering. "
           "See documentation of pango_font_description_from_string "
-          "for syntax.", DEFAULT_PROP_FONT_DESC,
-          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+          "for syntax.", DEFAULT_FONT_DESC,
+          G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS));
 
   /**
    * GESTimelineTitleSource:valignment
@@ -341,6 +341,12 @@ ges_tl_title_src_create_track_object (GESTimelineObject * obj, GESTrack * track)
     res = (GESTrackObject *) ges_track_title_source_new ();
     GST_DEBUG ("Setting text property");
     ges_track_title_source_set_text ((GESTrackTitleSource *) res, tfs->text);
+    ges_track_title_source_set_font_desc ((GESTrackTitleSource *) res,
+        tfs->font_desc);
+    ges_track_title_source_set_halignment ((GESTrackTitleSource *) res,
+        tfs->halign);
+    ges_track_title_source_set_valignment ((GESTrackTitleSource *) res,
+        tfs->valign);
   }
 
   else if (track->type == GES_TRACK_TYPE_AUDIO) {