<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/116965/">https://git.reviewboard.kde.org/r/116965/</a>
</td>
</tr>
</table>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/116965/diff/2/?file=263310#file263310line252" style="color: black; font-weight: bold; text-decoration: underline;">kstars/tools/pvplotwidget.cpp</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">void PVPlotWidget::mouseDoubleClickEvent( QMouseEvent *e ) {</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">252</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">this</span><span class="o">-></span><span class="n">magFactor</span><span class="p">(</span><span class="n">e</span><span class="o">-></span><span class="n">modifiers</span><span class="p">());</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Please be consistent in code formatting: this->magFactor( e->modifiers() );</pre>
</div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/116965/diff/2/?file=263310#file263310line260" style="color: black; font-weight: bold; text-decoration: underline;">kstars/tools/pvplotwidget.cpp</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">void PVPlotWidget::mouseDoubleClickEvent( QMouseEvent *e ) {</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">252</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">setLimits</span><span class="p">(</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">x</span><span class="p">()</span> <span class="o">+</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">right</span><span class="p">()</span> <span class="o">-</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">y</span><span class="p">()</span> <span class="o">+</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">bottom</span><span class="p">()</span> <span class="o">-</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span> <span class="p">);</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">260</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">setLimits</span><span class="p">(</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">x</span><span class="p">()</span> <span class="o">+</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">right</span><span class="p">()</span> <span class="o">-</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">y</span><span class="p">()</span> <span class="o">+</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">bottom</span><span class="p">()</span> <span class="o">-</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span> <span class="p">);</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Again, code formatting:
factor * 0.01 * size
etc.</pre>
</div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/116965/diff/2/?file=263310#file263310line268" style="color: black; font-weight: bold; text-decoration: underline;">kstars/tools/pvplotwidget.cpp</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">void PVPlotWidget::mouseDoubleClickEvent( QMouseEvent *e ) {</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">260</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">setLimits</span><span class="p">(</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">x</span><span class="p">()</span> <span class="o">-</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">right</span><span class="p">()</span> <span class="o">+</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">y</span><span class="p">()</span> <span class="o">-</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">bottom</span><span class="p">()</span> <span class="o">+</span> <span class="mf">0.0<span class="hl">2</span></span><span class="o">*</span><span class="n">size</span> <span class="p">);</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">268</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">setLimits</span><span class="p">(</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">x</span><span class="p">()</span> <span class="o">-</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">right</span><span class="p">()</span> <span class="o">+</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">y</span><span class="p">()</span> <span class="o">-</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span><span class="p">,</span> <span class="n">dataRect</span><span class="p">().</span><span class="n">bottom</span><span class="p">()</span> <span class="o">+</span> <span class="n"><span class="hl">factor</span></span><span class="o"><span class="hl">*</span></span><span class="mf">0.0<span class="hl">1</span></span><span class="o">*</span><span class="n">size</span> <span class="p">);</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Same here.</pre>
</div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/116965/diff/2/?file=263310#file263310line275" style="color: black; font-weight: bold; text-decoration: underline;">kstars/tools/pvplotwidget.cpp</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">275</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="kt">int</span> <span class="n">ranFactor</span> <span class="o">=</span> <span class="p">(</span><span class="n">qrand</span><span class="p">()</span> <span class="o">%</span> <span class="mi">21</span> <span class="o">+</span> <span class="mi">10</span><span class="p">);</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Also here...</pre>
</div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/116965/diff/2/?file=263310#file263310line276" style="color: black; font-weight: bold; text-decoration: underline;">kstars/tools/pvplotwidget.cpp</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">276</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">factor</span> <span class="o">=</span> <span class="p">(</span> <span class="n">modifier</span> <span class="o">&</span> <span class="n">Qt</span><span class="o">::</span><span class="n">ControlModifier</span><span class="p">)</span> <span class="o">?</span> <span class="mi">1</span> <span class="o">:</span> <span class="n">ranFactor</span><span class="p">;</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">... and here.</pre>
</div>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Apart from that, it would be nice if those two zoom in/out buttons were working in coarse mode.</pre>
<p>- Rafal Kulaga</p>
<br />
<p>On April 7th, 2014, 1:06 p.m. UTC, Vijay Dhameliya 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 KStars and Rafal Kulaga.</div>
<div>By Vijay Dhameliya.</div>
<p style="color: grey;"><i>Updated April 7, 2014, 1:06 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kstars
</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 zoom in/out factor is set constant which is very small so zoom in/out works very slowly, i.e. user have to keep rolling mouse wheel for long to get zoom enough to observe single planet.
This features allows user to do quick zoom in/out by pressing ctrl + wheel/+-keys. </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;">Testing has done for following use cases:
1) Zoom in/out by pressing +/- continuously till it can not zoom in/out furthermore
2) Zoom in/out by rolling rolling wheel up/down continuously till it can not zoom in/out furthermore
3) Zoom in/out by pressing Ctrl + +/- continuously till it can not zoom in/out furthermore
4) Zoom in/out by rolling Ctrl + rolling wheel up/down continuously till it can not zoom in/out furthermore
5) Zoom in/out by pressing +/- step by step till it can not zoom in/out furthermore
6) Zoom in/out by rolling rolling wheel up/down step by step till it can not zoom in/out furthermore
7) Zoom in/out by pressing Ctrl + +/- step by step till it can not zoom in/out furthermore
8) Zoom in/out by rolling Ctrl + rolling wheel up/down step by step till it can not zoom in/out furthermore
</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>kstars/tools/pvplotwidget.h <span style="color: grey">(ef90ace)</span></li>
<li>kstars/tools/pvplotwidget.cpp <span style="color: grey">(b8a8b30)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/116965/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>