<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/104900/">http://git.reviewboard.kde.org/r/104900/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 10th, 2012, 8:41 a.m., <b>David Faure</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;">Preferred? I'm confused. Aren't the checked keywords the only ones that work, and the other ones are disabled?
IMHO this is enabled/disabled, not preferred (which sounds like the other ones are working, just harder to reach)</pre>
</blockquote>
<p>On May 10th, 2012, 3:38 p.m., <b>Dawit Alemayehu</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;">Nope. That is exactly the confusion this patch is fixing. The checkbox was originally added, by whomever created the Konqueror search bar, to allow users to select or mark their preferred shortcuts that would appear in the search bar's drop down box. Otherwise, there were too many shortcuts to show there. Additionally, the shortcuts that are marked preferred are now used in application context menus to provide users the options to do quick searches.
Anyhow, when the same type of confusion as yours arose the first time and people opened a bug report about it, I thought I could resolve the issue by adding a checkbox, "Use selected shortcuts only" (now renamed to "Use preferred shortcuts only"), to allow behavior one expected from a checkbox appearing in front of an item on a list. Unfortunately and in hind sight very obviously that did not help at all as is evident from the bug reports listed above. Hence the change to resolve the issue once and for all. I even changed the "Whats This" help into "Tool Tips" to provide quick feedback to the user as to the purpose of those checkboxes.</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 see. Good change, then.</pre>
<br />
<p>- David</p>
<br />
<p>On May 9th, 2012, 8:35 p.m., Dawit Alemayehu wrote:</p>
<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 KDE Runtime.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated May 9, 2012, 8:35 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;">The following patch moves the "Preferred/Favorite" web shortcut selection checkbox into its own column to avoid confusion. The new column is marked as "Preferred" and also shows a tool tip message about is functionality. See the screenshot below.</pre>
</td>
</tr>
</table>
<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=168223">168223</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=218164">218164</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kurifilter-plugins/ikws/ikwsopts.cpp <span style="color: grey">(f1cc481)</span></li>
<li>kurifilter-plugins/ikws/ikwsopts_p.h <span style="color: grey">(9cfc12c)</span></li>
<li>kurifilter-plugins/ikws/ikwsopts_ui.ui <span style="color: grey">(440c201)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104900/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/104900/s/563/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/05/09/webshortcut_preferred_shortcut_checkbox_400x100.png" style="border: 1px black solid;" alt="Preferred selection column" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>