<table><tr><td style="">kossebau 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/D28608">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/D28608#642820" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D28608#642820</a>, <a href="https://phabricator.kde.org/p/dfaure/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@dfaure</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I think a too-generically named subdir creates a risk that people put other kinds of things in there, that's my main idea behind this name.</p></div>
</blockquote>

<p>Comparing "lib/qt5/plugins/kf5/kontakt" & "lib/qt5/plugins/kf5/kontaktplugins", I would not think that it help making sure that people see this as plugins "for" kontact, not plugins "from" kontakt. Given also that the "for" aspect is already a thing for any other subdirs in the plugins/ folder.<br />
(Personally I would even use "lib/qt5/plugins/kontakt5" instead, as this is not about plugins for anything from "kf5". And the "qt5" in the path is only because of technology needs, due to using Qt plugin loading logic, where any non-qt5 path would need more setup in normal system installations. Otherwise one would even expect "lib/plugins/kontakt5").</p>

<p>I see the point of following a naming pattern already seen elsewhere in the code. Though like the service type already modifies it, splitting the name and inserting a / -> "Kontact/Plugin", I would argue to do the same here, as in reusing the "plugins" from earlier in the path, and using the "kontact" only at the lowest ordering level.</p>

<p>My 2 cents as outsider :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R206 KMail</div></div></div><br /><div><strong>BRANCH</strong><div><div>kontact_kmailplugin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28608">https://phabricator.kde.org/D28608</a></div></div><br /><div><strong>To: </strong>dfaure, mlaurent, winterz<br /><strong>Cc: </strong>kossebau, kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>