ralsina / marave

Automatically exported from code.google.com/p/marave
GNU General Public License v2.0
0 stars 2 forks source link

Preferences dialog doesn't appear in OpenSUSE 11.2 #71

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Originally reported by  Fahad.Alsaidi in Issue 67:

Clicking the preferences button doesn't do anythin, nothing strange printed in 
terminal.

Original issue reported on code.google.com by roberto.alsina on 23 Feb 2010 at 8:36

GoogleCodeExporter commented 9 years ago
Can't reproduce it.

Original comment by roberto.alsina on 23 Feb 2010 at 8:39

GoogleCodeExporter commented 9 years ago
I have the same problem.

Original comment by filipe.c...@gmail.com on 24 Feb 2010 at 1:52

GoogleCodeExporter commented 9 years ago
I need a hand here, I just can't reproduce it :-(
Any chance of 1 hour with remote, unprivileged access to a box having this 
problem?

Original comment by roberto.alsina on 25 Feb 2010 at 2:47

GoogleCodeExporter commented 9 years ago
I've bisected the problem to this update: r358

Author: roberto.alsina <roberto.alsina@57fa06e0-0c1b-11df-9e2b-972ea447d3bf>
Date:   Fri Feb 19 15:04:49 2010 +0000

    Starting big code cleanup. Killed the animatedopacity class, now Marave requires 
Qt 4.6

I have a feeling it's a Qt4.5 vs 4.6 problem but i'm not sure what it is yet.

Original comment by filipe.c...@gmail.com on 25 Feb 2010 at 1:59

GoogleCodeExporter commented 9 years ago
Could you try r437? I have a similar problem reported with Qt 4.5 in Issue 74

Original comment by roberto.alsina on 27 Feb 2010 at 1:58

GoogleCodeExporter commented 9 years ago
Still the same problem.

Original comment by filipe.c...@gmail.com on 27 Feb 2010 at 11:56

GoogleCodeExporter commented 9 years ago
Issue 74 has been merged into this issue.

Original comment by roberto.alsina on 27 Feb 2010 at 1:58

GoogleCodeExporter commented 9 years ago
I tried r437 and I'm getting the same error.

Original comment by andresgattinoni on 27 Feb 2010 at 2:40

GoogleCodeExporter commented 9 years ago
The following patch solves the problem for me. It was a missing call to the 
later() 
function.

diff --git a/marave/main.py b/marave/main.py
index 7c927af..8dd574c 100644
--- a/marave/main.py
+++ b/marave/main.py
@@ -208,6 +208,8 @@ def animheight(thing, target, thendo=None):
             thing.hanim.finished.connect(thendo)
     else:
         thing.resize(thing.width(),target)
+        if thendo:
+           thendo()

 class FunkyButton(QtGui.QPushButton):
     def __init__(self, icon, text, scene, name=None):

Original comment by filipe.c...@gmail.com on 27 Feb 2010 at 8:57

GoogleCodeExporter commented 9 years ago
Applied the path from comment 9 as r438, please check

Original comment by roberto.alsina on 27 Feb 2010 at 9:04

GoogleCodeExporter commented 9 years ago
Since filipe created the patch and andrés has a bug with focus where the prefs 
are visible, I 
assume this works now ;-)

Original comment by roberto.alsina on 1 Mar 2010 at 10:13

GoogleCodeExporter commented 9 years ago
I checked r438 version and the bug is fixed. Thanks

Original comment by Fahad.Al...@gmail.com on 1 Mar 2010 at 4:17