<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/101848/">http://git.reviewboard.kde.org/r/101848/</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 rekonq.</div>
<div>By Vyacheslav Blinov.</div>
<p style="color: grey;"><i>Updated July 11, 2011, 10:13 a.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;">Thanks for your review, I really appreciate that you point me on my mistakes and give me possibilities to learn.
This is what was improved since last review:
* removed trailing spaces
* used "m_" prefix to show member variables
* used class's initialization list
* fixed intendation
* removed unneeded QWidget that was used as view
* fixed "magic number", now preview generated using actual browser window's sizing (so preview pixmap is same size as actual browser's page shown in tab)
* moved mask operations to resizing function
Johannes Tröscher:
I didn't changed scope of consts since I belive that they are specific to this class and not to global scope, I prefer to keep global scope clean for things that are really global.
Andrea Diamantini:
Isn't plasma transparency didn't works without compositing? Of course it can be made to be transparent without compositing, but it will be resource consuming (comparing to compositing solution) and slow on not so modern pc's where compositing usually can't be enabled cause of legacy video adapter. Anyway with compositing disabled tab preview still shown, it's just not transparent.
Pierre Rossi:
I came to use current rekonq`s sizing cause it looks more naturally, and don't have problems with nvidia's twinview. You can blame me, but it looks good at any resolution and browser size, I've used. I know it can look not so well with very small browser window (e.g. 100x100), but imho no one browser can be usable with such a sizing.</pre>
</td>
</tr>
</table>
<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;">Improvements:
* title of the page is removed since it is visible on a tab.
* widget now has a smooth 3d look'n'feel with Oxygen thanks to raisen frame
* widget has rouned corners in all the styles
* url of the page is elided
* with compositing tab previews will be shown semitransparent with 25% alpha by default, witch can be setted to be optional later
* code of tab preview is now separated from tab bar so it is more explicit</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 with different styles: it still looks ok, without compositing transparency didn't works</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/CMakeLists.txt <span style="color: grey">(db9506a)</span></li>
<li>src/tabbar.h <span style="color: grey">(c1b0ba5)</span></li>
<li>src/tabbar.cpp <span style="color: grey">(dee49ea)</span></li>
<li>src/tabpreviewpopup.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/tabpreviewpopup.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101848/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/101848/s/193/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/04/screenshot_400x100.png" style="border: 1px black solid;" alt="sample of tab preview" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>