<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/119314/">https://git.reviewboard.kde.org/r/119314/</a>
     </td>
    </tr>
   </table>
   <br />



<table bgcolor="#e0e0e0" width="100%" cellpadding="12" style="border: 1px gray solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>
   <h1 style="margin: 0; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>
  </td>
 </tr>
</table>
<br />


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for Calligra.</div>
<div>By Sascha Suelzer.</div>


<p style="color: grey;"><i>Updated March 30, 2015, 2:29 p.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;">Submitted with commit 54e6fe9254fa924dd3628c91f93a45c0606b3fea by Sascha Suelzer to branch master.</pre>
  </td>
 </tr>
</table>







<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is an <em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">incomplete</em>, but working change, originally part of a bigger refactoring to make the Favorite Preset Manager a bit nicer to use by allowing filtering for the All Presets View, and optionally the user defined Tag groups.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">However, due to a lack of signals there was some trouble.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When a resource was selected it enabled the Add To Favorites button, but when the tag group was changed, invalidating the resource, the Add To Favorites button was still enabled, leading to a crash when pressed. </p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Anyway, the part that works implements a set-able read only mode for the Tagging Manager which, when Tag modification is disallowed, will hide the TagToolButton to create, rename or delete Tag groups, disable the context menu and disallow change in the members of a Tag group, but allows to filter the Tag group.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">For example, a Tag group called erasers, with only erasers in it, would allow filtering with search terms, but there would not be a save button and pressing enter or clearing the search box would revert to showing only the original erasers in the Tag group.</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I tested it by modifying the constructor of KoResourceItemChooser.cpp, adding a d->tagManager->allowTagModification(false); at the bottom.</p></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>libs/widgets/KoResourceTaggingManager.h <span style="color: grey">(4f43667)</span></li>

 <li>libs/widgets/KoResourceTaggingManager.cpp <span style="color: grey">(3cffe5c)</span></li>

 <li>libs/widgets/KoTagChooserWidget.h <span style="color: grey">(55029d6)</span></li>

 <li>libs/widgets/KoTagChooserWidget.cpp <span style="color: grey">(0306805)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/119314/diff/" style="margin-left: 3em;">View Diff</a></p>






  </td>
 </tr>
</table>



  </div>
 </body>
</html>