[pitivi] Add a scrollwindow to the effectconfiguration when needed

[Edward Hervey <edwardrv src gnome org>]

commit 8015309bece6d1891b39555814d8091c6171c238
Author: Thibault Saunier <tsaunier gnome org>
Date:   Wed Jul 21 10:05:34 2010 -0400

    Add a scrollwindow to the effectconfiguration when needed

 pitivi/ui/effectsconfiguration.py |   21 +++++++++++++++++----
 1 files changed, 17 insertions(+), 4 deletions(-)
---
diff --git a/pitivi/ui/effectsconfiguration.py b/pitivi/ui/effectsconfiguration.py
index 3e3fd45..1ec16e6 100644
--- a/pitivi/ui/effectsconfiguration.py
+++ b/pitivi/ui/effectsconfiguration.py
@@ -43,17 +43,30 @@ class EffectsPropertiesHandling:
         """
         if effect not in self.cache_dict:
             #Here we should handle special effects configuration UI
-            effect_configuration_ui = gtk.ScrolledWindow()
             effect_set_ui = GstElementSettingsWidget()
             effect_set_ui.setElement(effect, ignore=PROPS_TO_IGNORE,
                                                default_btn=True, use_element_props=True)
-            effect_configuration_ui.add_with_viewport(effect_set_ui)
+            nb_rows = effect_set_ui.get_children()[0].get_property('n-rows')
+            if nb_rows > 4:
+                effect_configuration_ui = gtk.ScrolledWindow()
+                effect_configuration_ui.add_with_viewport(effect_set_ui)
+                self.cache_dict[effect] = effect_configuration_ui
+            else:
+                self.cache_dict[effect] = effect_set_ui
             self._connectAllWidgetCbs(effect_set_ui, effect)
-            self.cache_dict[effect] = effect_configuration_ui
-        effect_set_ui = self.cache_dict[effect].get_children()[0].get_children()[0]
+
+        effect_set_ui = self._getEffectSetUI(effect)
+
         self._current_effect_setting_ui = effect_set_ui
         return self.cache_dict[effect]
 
+    def _getEffectSetUI(self, effect):
+        if type(self.cache_dict[effect]) is gtk.ScrolledWindow:
+            effect_set_ui = self.cache_dict[effect].get_children()[0].get_children()[0]
+        else:
+            effect_set_ui = self.cache_dict[effect]
+        return effect_set_ui
+
     def _flushSeekVideo(self):
         self.pipeline.pause()
         if self.pipeline is not None: