<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="https://git.reviewboard.kde.org/r/116789/">https://git.reviewboard.kde.org/r/116789/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On March 15th, 2014, 11:09 a.m. EET, <b>Mehrdad Momeny</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;">It's not good to keep the image in it's size! this will ruin our GUI, as you can see in this screenshot: http://imagebin.org/299447
I prefer the image to be smaller and even maybe resizable! I mean, if user choose to have smaller window, it doesn't push avatar or anything else out of view.
Also, it seems that your change corrupted current layout of User tweets! as you can see the third tweet in that screenshot.</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;">I am with Mehrdad with his opinion, I've tried your patch and it fetches the image but it ruins the UI in a bad way.
IMHO perhaps you could load a small image like what happens with Youtube links and then wrap the <img> with an <a> that refers to the image URL.
d->mImage = QString("<td width=\"%1\" height=\"%2\"><a href=\"%3\"><img src=\"img://postImage\" /> </a></td>").arg(QString(d->mCurrentPost->mediaSizeWidth), QString(d->mCurrentPost->mediaSizeHeight), d->imageUrl );</pre>
<br />
<p>- Ahmed</p>
<br />
<p>On March 13th, 2014, 10:35 p.m. EET, Daniel Kreuter wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Choqok and Mehrdad Momeny.</div>
<div>By Daniel Kreuter.</div>
<p style="color: grey;"><i>Updated March 13, 2014, 10:35 p.m.</i></p>
<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=299433">299433</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
choqok
</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;">Currently Choqok doesn't show images which may be part of Tweets. This bugfix adds support for images and adds them to the postwidget underneath the tweet.</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;">Tested and images are shown in the posts. If a tweet doesn't have an image, the post looks as before.</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>helperlibs/twitterapihelper/twitterapimicroblog.cpp <span style="color: grey">(f0c8296)</span></li>
<li>libchoqok/choqoktypes.h <span style="color: grey">(c8cc59c)</span></li>
<li>libchoqok/ui/postwidget.h <span style="color: grey">(da460c9)</span></li>
<li>libchoqok/ui/postwidget.cpp <span style="color: grey">(eb005ae)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/116789/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>