<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://svn.reviewboard.kde.org/r/6513/">http://svn.reviewboard.kde.org/r/6513/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 20th, 2011, 3:51 p.m., <b>Lukáš Tinkl</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looks OK to me</pre>
</blockquote>
<p>On April 8th, 2011, 11:18 p.m., <b>Lamarque Souza</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Can this patch be commited to trunk?</pre>
</blockquote>
<p>On July 11th, 2011, 12:23 a.m., <b>Lamarque Souza</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ok, I will commit this patch then.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Patch submitted. I do not know why but it has not been marked as so. Someone please close this review request.</pre>
<br />
<p>- Lamarque</p>
<br />
<p>On February 21st, 2011, 5:29 p.m., alter_schwede wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdelibs and usability.</div>
<div>By alter_schwede.</div>
<p style="color: grey;"><i>Updated Feb. 21, 2011, 5:29 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I already posted this patch here:
https://bugs.kde.org/show_bug.cgi?id=265955
This is a patch to the kded_randrmonitor service that solves the below
mentioned problem.
If only one output is connected and the user tries to switch the display it
makes sure the connected output gets activated and deactivates all disconnected
output afterwards.
Reproducible: Always
Steps to Reproduce:
1. Connect an external display to your laptop and use your latop's hotkey (e.g.
Fn+F6) to active it and deactivate your laptop's display.
2. Disconnect the external display
3. Try to switch back to internal display with your keyboard
Actual Results:
Nothing happens
----
I improved my patch.
Now it also automatically deactivates all disconnected outputs on resume from
suspend and switches to a connected monitor if needed. This should fix e.g. Bug
#257642.
</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=257642">257642</a>,
<a href="https://bugs.kde.org/show_bug.cgi?id=265955">265955</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/KDE/4.6/kdebase/workspace/kcontrol/randr/module/randrmonitor.h <span style="color: grey">(1221086)</span></li>
<li>/branches/KDE/4.6/kdebase/workspace/kcontrol/randr/module/randrmonitor.cpp <span style="color: grey">(1221086)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/6513/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>