<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/110701/">http://git.reviewboard.kde.org/r/110701/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/110701/diff/3/?file=164918#file164918line52" style="color: black; font-weight: bold; text-decoration: underline;">applets/notes/notes.cpp</a>
<span style="font-weight: normal;">
(Diff revision 3)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">52</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="k">const</span> <span class="n">QString</span> <span class="nf">defaultBackgroundColor</span><span class="p">()</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">the const here is not necessary</pre>
</div>
<br />
<p>- Aaron J.</p>
<br />
<p>On June 20th, 2013, 2:22 p.m. UTC, Wolfgang Bauer wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma and Anne-Marie Mahfouf.</div>
<div>By Wolfgang Bauer.</div>
<p style="color: grey;"><i>Updated June 20, 2013, 2:22 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;">The notes plasmoid by default takes the text color from the current plasma theme but has "yellow" hardcoded as default background color.
This can lead to unreadable notes by default with certain plasma themes, e.g. Produkt.
It's especially annoying if you're using such a theme because you can't globally change the default notes colors but have to do it for each note on its own (f.e. every time after pasting text to the desktop by pressing the middle mouse button).
This patch changes the default background color to black if the text color is brighter than a certain threshold.
Also the background color is re-read from the config on plasma theme change to make it change as well if necessary.
The brightness is determined with QColor::lightness().
I chose 100 as threshold, because yellow's lightness value is 128 and yellow on yellow wouldn't be good either... ;-)
I don't know if 100 is the perfect value, but it works as intended with all the themes I have installed.</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;">Created a note on the desktop (with default text and background color) and switched the plasma theme to each one I have installed. (Air, Air for Netbooks, Air openSUSE, Androbit, Aya, Produkt, Slim Glow, Tibanna, openSUSE)
Notes were readable with any theme, the background color changed to black where necessary, was yellow as before otherwise.
This patch is also part of openSUSE's plasma-addons package since a month ago.</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="http://bugs.kde.org/show_bug.cgi?id=320350">320350</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>applets/notes/notes.cpp <span style="color: grey">(5c2ed70)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110701/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/06/20/notes_Produkt_without_patch.png">Note in Produkt theme without this patch</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/06/20/notes_Produkt_with_patch.png">Note in Produkt theme with this patch</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>