<html><head></head><body><div id="fenix-user-content">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><span style="font-family:'NokiaPureTextLight'">Wait, I got a comment but can't post it from the phone, will be back in about an hour </span></p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><span style="font-family:'NokiaPureTextLight'"> </span></p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><span style="font-family:'NokiaPureTextLight'">--</span></p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><span style="font-family:'NokiaPureTextLight'"> </span></p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><span style="font-family:'NokiaPureTextLight'">Sent from my Nokia N9</span></p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><span style="font-family:'NokiaPureTextLight'"> </span></p></div><br><div id="fenix-reply-header"><p>On 29.10.2014 12:57 Kai Uwe Broulik wrote:<br></p></div><div id="fenix-quoted-body"><div>
  <div style="font-family:Verdana,Arial,Helvetica,Sans-Serif">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border:1px #c9c399 solid;border-radius:6px">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/120887/" target="_blank">https://git.reviewboard.kde.org/r/120887/</a>
     </td>
    </tr>
   </tbody></table>
   <br>



 <p>Ship it!</p>



 <pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word"><p style="padding:0;margin:0;line-height:inherit;white-space:inherit">Ship It!</p></pre>
 <br>









<p>- Kai Uwe Broulik</p>


<br>
<p>On Oktober 29th, 2014, 11:49 vorm. UTC, Marco Martin wrote:</p>









<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border:1px #888a85 solid;border-radius:6px">
 <tbody><tr>
  <td>

<div>Review request for KDE Frameworks.</div>
<div>By Marco Martin.</div>


<p style="color:grey"><i>Updated Okt. 29, 2014, 11:49 vorm.</i></p>









<div style="margin-top:1.5em">
 <b style="color:#575012;font-size:10pt">Repository: </b>
knotifications
</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">
 <tbody><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"><p style="padding:0;margin:0;line-height:inherit;white-space:inherit">There are some cases in which applications need a custom icon for a particular notification, and used to work around sending the pixmap of the icon instead (of which you don't know if will end up the right size)</p>
<p style="padding:0;margin:0;line-height:inherit;white-space:inherit">this adds an icon property to knotification, if none set, the one set in the notifyrc is used as before</p></pre>
  </td>
 </tr>
</tbody></table>



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

 <li>src/knotification.h <span style="color:grey">(456e84b)</span></li>

 <li>src/knotification.cpp <span style="color:grey">(0ea13cb)</span></li>

 <li>src/notifybypopup.cpp <span style="color:grey">(c7add40)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/120887/diff/" style="margin-left:3em" target="_blank">View Diff</a></p>






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








  </div>
 </div>


<br>_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
<br></div><br></body></html>