<table><tr><td style="">romangg 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/D29024">View Revision</a></tr></table><br /><div><div><p>These are not specific issues but some general complains about the overall concept chosen here without providing an alternative solution.</p>

<p>I chose this plugin system because it allows robust extension to the current system. Plugins are well contained and reuse already existing infrastructure. Besides moving the code around there are practically no logic changes to the KWayland backend.</p>

<p>Note that I need to have this plugin system or something similar in for 5.19 or I will be forced to fork libkscreen permanently for KWinFT. I would like to omit this and instead continue my work on libkscreen as a KDE project like I have worked on it in the last two years.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R110 KScreen Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29024">https://phabricator.kde.org/D29024</a></div></div><br /><div><strong>To: </strong>romangg, Plasma<br /><strong>Cc: </strong>davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>