<table><tr><td style="">malteveerman 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/D11862">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/D11862#239984" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11862#239984</a>, <a href="https://phabricator.kde.org/p/markey/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@markey</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>See, the problem with removing features like this is, there will be a huge outcry from users. That's guaranteed, even without knowing for sure how much the scripting feature is really used in the wild.</p>

<p>I was discussing about this with Heiko. Maybe QtScript could be replaced with QJSEngine, similar to how the following patch for another project attempts to do it?</p>

<p><a href="https://phabricator.kde.org/D7033" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D7033</a></p></div>
</blockquote>

<p>I see your point about removing features. But I've taken a look at the most popular scripts on opendesktop.org and they all rely on Qt bindings. Unless there is some way to restore bindings support this feature is broken and people will complain anyway.<br />
About replacing QtScript with QJSEngine: I think it's quite possible to do so, but I have my doubts if the development and maintenance work is worth it given our limited manpower.<br />
Anyway, I'll open a new revision just with the patch that implements the lyrics functionality in C++.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R181 Amarok</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11862">https://phabricator.kde.org/D11862</a></div></div><br /><div><strong>To: </strong>malteveerman, Amarok<br /><strong>Cc: </strong>markey, Amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber, heikobecker<br /></div>