<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/102621/">http://git.reviewboard.kde.org/r/102621/</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.</div>
<div>By Jaime Torres Amate.</div>


<p style="color: grey;"><i>Updated Sept. 21, 2011, 7:57 p.m.</i></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">One more try.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description  (updated)</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 third verion of this patch does:
* Shows the city, UTC displacement, and time in one line and the date (I hope English people say "on the" where Spanish people say "del día"), in another line, always complete.
* removes a <table> that makes the rendering harder
* adds a <br> to include an space between the dates and the event. 

Look at the screnshot.

In any case, I think one part of this patch MUST be commited.
* removes a <table> that makes the rendering harder
* adds a <br> to include an space between the dates and the event. 
That is, replace 
if (!subText.isEmpty()) {
     subText.prepend("<table>");                subText is never empty as is created with QString("<table>")
     subText.append("</table>");                
   }
with
subText.append("</table><br>");

Of course, kwarning() << data; is not there anymore.</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;">Checked with zero, one and several timezones with short and large city names in two machines.</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=260394">260394</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>libs/plasmaclock/clockapplet.cpp <span style="color: grey">(b1275af)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/102621/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/102621/s/262/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/15/plasma.260394.2_400x100.png" style="border: 1px black solid;" alt="version 2" /></a>

 <a href="http://git.reviewboard.kde.org/r/102621/s/267/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/21/instantanea3_400x100.png" style="border: 1px black solid;" alt="showing also the UTC displacement" /></a>

</div>


  </td>
 </tr>
</table>




  </div>
 </body>
</html>