<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22444">View Revision</a></tr></table><br /><div><div><p>In general +1 on the concept, but needs revision before it can go in.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D22444#inline-126537">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dolphinui.rc:117</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">        <Action priority="0" name="view_zoom_out"/>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <Action priority="0" name="view_zoom_reset"/>
</div><div style="padding: 0 8px; margin: 0 4px; ">        <Action priority="0" name="edit_cut"/>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Put it in between Zoom in and Zoom out, as it is in Okular and Konsole.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D22444#inline-126536">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dolphinviewactionhandler.cpp:197</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">QAction</span><span style="color: #aa2211">*</span> <span class="n">zoomResetAction</span> <span style="color: #aa2211">=</span> <span class="n">m_actionCollection</span><span style="color: #aa2211">-></span><span class="n">addAction</span><span class="p">(</span> <span class="n">QStringLiteral</span><span class="p">(</span><span style="color: #766510">"view_zoom_reset"</span><span class="p">)</span> <span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">zoomResetAction</span><span style="color: #aa2211">-></span><span class="n">setText</span><span class="p">(</span> <span class="n">i18nc</span><span class="p">(</span><span style="color: #766510">"@action:inmenu View"</span><span class="p">,</span> <span style="color: #766510">"Zoom Reset"</span><span class="p">)</span> <span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">zoomResetAction</span><span style="color: #aa2211">-></span><span class="n">setWhatsThis</span><span class="p">(</span><span class="n">i18nc</span><span class="p">(</span><span style="color: #766510">"@info:whatsthis zoom reset"</span><span class="p">,</span> <span style="color: #766510">"This resets the icon size to default."</span><span class="p">));</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Name should be "Reset zoom level" or "Zoom to default size"</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D22444#inline-126533">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dolphinviewactionhandler.cpp:466</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">const</span> <span style="color: #aa4000">int</span> <span class="n">resetLevel</span> <span style="color: #aa2211">=</span> <span class="p">(</span><span class="n">ZoomLevelInfo</span><span style="color: #aa2211">::</span><span class="n">minimumLevel</span><span class="p">()</span> <span style="color: #aa2211">+</span> <span class="n">ZoomLevelInfo</span><span style="color: #aa2211">::</span><span class="n">maximumLevel</span><span class="p">())</span><span style="color: #aa2211">/</span><span style="color: #601200">2</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">m_currentView</span><span style="color: #aa2211">-></span><span class="n">setZoomLevel</span><span class="p">(</span><span class="n">resetLevel</span><span class="p">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">That seems like a fairly arbitrary value to reset it to. You should read the default size and zoom to that instead. After all, that's what the WhatsThis text you wrote says it will do. :)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R318 Dolphin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22444">https://phabricator.kde.org/D22444</a></div></div><br /><div><strong>To: </strong>shubham, ngraham, elvisangelaccio<br /><strong>Cc: </strong>kfm-devel, kde-doc-english, aprcela, fprice, gennad, fbampaloukas, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros, skadinna, emmanuelp, mikesomov<br /></div>