Created attachment 56598 [details] Solves the problem with unsaved disabled outputs Version: unspecified (using KDE 4.6.0) OS: Linux Outputs which were firstly enabled and then were disabled by user can't be stored to krandrrc properly due to cyclic dependency in RandROutput::slotDisable() method: it tries to void rect and refreshrate using RandROutput::proposeRect() and RandROutput::proposeRefreshRate() methods which in turn try to enable the output. So in RandROutput::save() it's treated as enabled and the fact that it's actually disabled isn't saved. I removed unnecessary proposeRect() and proposeRefreshRate() calls from slotDisable() and also added to RandROutput::applyProposed() special case to save always if the output is disabled (as we don't have something like proposeActive var and I don't think it's necessary). Reproducible: Always Steps to Reproduce: 1. Enable one of the outputs. Apply the results. See that it was saved to ~/.kde/share/config/krandrrc 2. Disable it. Apply the results. See that settings on krandrrc weren't changed for the output.
Git commit 0040d4b70e2c4bc4c4fe4fb33cf0018d430a2205 by Aaron Seigo. Committed on 04/02/11 at 08:16. Pushed by aseigo into branch 'KDE/4.6'. save disabled screens, don't enable by accident based on a patch by Alexey Chernov BUG:264702 M +18 -9 kcontrol/randr/randroutput.cpp http://commits.kde.org/kde-workspace/0040d4b70e2c4bc4c4fe4fb33cf0018d430a2205
Git commit 39dd0dd2946cc864f884e32ad3dbb43055f56f12 by Aaron Seigo. Committed on 04/02/11 at 08:16. Pushed by aseigo into branch 'master'. save disabled screens, don't enable by accident based on a patch by Alexey Chernov BUG:264702 M +18 -9 kcontrol/randr/randroutput.cpp http://commits.kde.org/kde-workspace/39dd0dd2946cc864f884e32ad3dbb43055f56f12
Aaron, thank you very much for committing my patches!