<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://git.reviewboard.kde.org/r/106862/">http://git.reviewboard.kde.org/r/106862/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.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 Plasma, KDE Usability and Dan Vrátil.</div>
<div>By Kai Uwe Broulik.</div>
<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;">This patch adds a context menu entry "Screen Resolution" next to the Configure $containment entry.
My rationale behind this is that I think many people (especially those from the Windows world, even more with Win7) expect to be able to manipulate the Screen Resolution and arrangement settings right from their Desktop. It is easily adjustable in case kcm_randr gets replaced by the new kscreen-based one.
Since this was a quick patch I am fine with it not being included, especially since I do not know what plans there are for the new display configuration in terms of easy quickaccess using a plasmoid (ie. common settings such as "Clone", "Projector Only", "Extend Right", etc)
I initially wanted to add the KCM directly to the backgrounddialog thing but that would have made it too complex, I guess. (I thought there was an easy function to just add an existing kcm to a paged dialog, but I couldn't remember).
The wording/appearance of the entry could definitly be better (the ellipsis should be dropped there, I think)</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Entry shows up (enabled by default), can be enabled and disabled, and clicking it triggeres the Randr KCM.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plasma/generic/containmentactions/contextmenu/menu.h <span style="color: grey">(7ef79d3)</span></li>
<li>plasma/generic/containmentactions/contextmenu/menu.cpp <span style="color: grey">(33f07e7)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106862/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/106862/s/771/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/10/15/screenresentry_400x100.jpg" style="border: 1px black solid;" alt="Context menu entry" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>