Created attachment 367254 [details] this patch delay the signals connections to after loading and setting up the complete ui Description of problem: if u have compiz enabled via desktop-effects, enable wobbly windows, close the dialog, reopen it : wobbly are disabled Version-Release number of selected component (if applicable): desktop-effects-0.8.4-2.fc12.i686 How reproducible: Always Steps to Reproduce: 1.Open Desktop Effects, check compiz, wobbly, and cube 2.close the dilog 3.reopen it : wobbly is disabled Actual results: if we have wobbly and cube, opening desktop-effects disables wobbly Expected results: if we have wobbly and cube, opening desktop-effects won't disable wobbly Additional info: I didn't understand the problem exactly, nevertheless, the attached patch solves it
Looking at it, the problem is that on_option_toggled() handles both the cube and wobbly settings. When we are setting up the widgets, we first change the value of 'cube' and then the value of 'wobbly', so if we set 'cube' to true, then on_option_toggled() will be called before the wobbly toggle button is set, so we'll think the user has disabled wobbly. Your fix seems like a reasonable approach to fixing the issue. (The other approach I could imagine would be to split on_option_toggled() into an on_cube_toggled() and an on_wobbly_toggled(). But your approach is as good.) Do you think you could redo it as a proper git patch ? If you aren't familiar with git: use 'git commit' to commit it locally to a checkout of the desktop-effects git repostory. Make sure you've set your name and email as described in, say, http://help.github.com/git-email-settings/. The commit message should consist of a single short line that summarizes what the commit does, a blank line, then a detailed description of what was changed and (more importantly) why. Then format the commit as a patch using 'git format-patch HEAD -1' and attach the resulting file. Not sure if we'll be able to get this in right now during the beta freeze. Do you happen to know if this was a regression from F11? It's not apparent to me from reading the code - my guess would that probably it did regress but I'm not completely sure of that.
This bug appears to have been reported against 'rawhide' during the Fedora 12 development cycle. Changing version to '12'. More information and reason for this action is here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping
Do not know why this bug was closed, reopening.
desktop-effects-0.8.5-1.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/desktop-effects-0.8.5-1.fc13
desktop-effects-0.8.5-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/desktop-effects-0.8.5-1.fc12
desktop-effects-0.8.5-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update desktop-effects'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/desktop-effects-0.8.5-2.fc13
desktop-effects-0.8.5-1.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update desktop-effects'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/desktop-effects-0.8.5-1.fc12
desktop-effects-0.8.5-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
desktop-effects-0.8.5-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.