<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/100374/">http://git.reviewboard.kde.org/r/100374/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 21st, 2011, 12:19 a.m., <b>Andrea Diamantini</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;">That's not exactly what I suggested you. Remove the m_developersMenu variable from the class definition and reload the tools menu every time, so that if developer tools are activated you add the developers menu. If not you don't create it at all.</pre>
</blockquote>
<p>On January 21st, 2011, 3:25 p.m., <b>Felix Rohrbach</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;">I would have thought that this is too slow (you can feel the difference - compare the time it needs to show the first and the second time you open the menu with this patch), but of course I can change it. </pre>
</blockquote>
<p>On January 22nd, 2011, 9:37 a.m., <b>Andrea Diamantini</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;">You are right :(
In this case, my idea does not fit. What about reproposing the previous version (diff 3) and merging it? I guess we are not gaining what I supposed with this...</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 would prefer the last version (4), so that the menu is created the first time the user uses it. I think a delay is ok if it's just the first time (like with dialogs) and it might save us some startup time. Additionally, I don't like version 3 because changing the visibility with aboutToShow() is much cleaner then using a settingsChanged() slot, so if you want to create the menu at startup (the way it is now) I would change at least this.</pre>
<br />
<p>- Felix</p>
<br />
<p>On January 20th, 2011, 4:50 p.m., Felix Rohrbach wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 Felix Rohrbach.</div>
<p style="color: grey;"><i>Updated Jan. 20, 2011, 4:50 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;">This patch contains the changes to the context menu which we decided in our meeting:
1. Hide new tab entry if it is not necessary.
2. Print and Search action: I removed the print action. The search action is now in the "search with" submenu if some text is selected.
3. Hide the development menu entries and add an option to show them.
Clone: git://anongit.kde.org/clones/rekonq/felixr/rekonq
Branch: meetingMenuChanges
</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>src/mainwindow.h <span style="color: grey">(02cd723)</span></li>
<li>src/mainwindow.cpp <span style="color: grey">(59fff83)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100374/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>