<table><tr><td style="">ngraham added a comment.
</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/D7668" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D7668#142562" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7668#142562</a>, <a href="https://phabricator.kde.org/p/elvisangelaccio/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@elvisangelaccio</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Note that this only works if there are no custom places yet (probably not a big deal).</p></div>
</blockquote>

<p>Yes, the idea here is to change the default setting for new installs, and this code is only run the first time, before the user has added any custom places.</p>

<p>Thanks for pointing out the #ifdef HAVE_BALOO issue. I'll admit wasn't actually able to test this due to the Frameworks 5.38.0 bump yesterday which caused a few build issues, and I couldn't get all the dependencies to build. I'll make the requested change, and test when I'm able to sort out the build issues, or when Neon issues package updates for version 5.38.0.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7668" rel="noreferrer">https://phabricator.kde.org/D7668</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin, dfaure, davidedmundson, ltoscano, broulik, elvisangelaccio<br /><strong>Cc: </strong>elvisangelaccio, cfeck, navarromorales, nicolasfella, firef, andrebarros, emmanuelp<br /></div>