Review Request: KControl/randr: Make changes to the absolute X/Y coordinates of a display call setConfigDirty

[Thomas Gahr]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102631/#review6735
-----------------------------------------------------------


Just tested it and it fixes the issues mentioned in the bug 273102 :)
Now... I think it might be a little unlikely for me to be a kde-dev and actually just pushing stuff to the git repo - will you, Christoph of Aaron do that?

- Thomas


On Sept. 16, 2011, 2:10 p.m., Thomas Gahr wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102631/
> -----------------------------------------------------------
> 
> (Updated Sept. 16, 2011, 2:10 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> KControl/randr: Make changes to the absolute X/Y coordinates of a display call setConfigDirty and thus enable the "apply" button.
> In some situations the absolute X/Y position is the only thing the user needs to change. At the moment you need to make some dummy change (change resolution back and forth or alike) to enable the "apply" button. This is not a good user experience.
> 
> Sorry for the unnecessary whitespace-changes, qt-creator auto-strips unnecessary space upon saving. If everything else is fine with this patch and anybody insists on it I will update the patch dropping these changes.
> 
> 
> Diffs
> -----
> 
>   kcontrol/randr/outputconfig.cpp 38c8386 
> 
> Diff: http://git.reviewboard.kde.org/r/102631/diff
> 
> 
> Testing
> -------
> 
> Backported the patch to kde-workspace sources 4.7.0 used in Fedora's KDE rpms (i.e. change valueChanged(int) to textChanged(const QString&), built a new rpm from the patched sources and tested it on my install. Works just as expected. I have yet to finish the setup of my kde-devel environment to test it on current master. Will keep you updated if it breaks anything.
> 
> [update] Works fine with current master
> 
> 
> Thanks,
> 
> Thomas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20110922/fe2c0f3e/attachment.html>