<div dir="ltr"><div>In general this looks OK, there's some useful features and I can see myself using this. I'd very much like it to become part of Plasma.</div><div><br></div><div>I gave it a review and made some notes below.</div><div><br></div><div>kded.cpp</div><div>The touchpad backend leaks?</div><div><br></div><div>There are blocking calls in the constructor using isServiceRegistered</div><div>combined with the dataengine querying this kded module in a blocking way in init we have a strong potential to deadlock the two applications</div><div>For KDED modules we have to be a lot more strict to never block as others will be querying us.</div><div><br></div><div>I don't understand why we're watching for plasmashell/kglobalaccel anyway. Could you explain the rationale here.</div><div><br></div><div>applet:</div><div>The applet is completely not ported.</div><div><br></div><div>Applet translations need to go into a differnet .po file with a specific name</div><div>Copy a Messages.sh from one of the other plasmoids</div><div><br></div><div>KCM:</div><div>reverse scrolling doesn't seem to work</div><div>"disabled taps and scrolling only" -- The HIG says to avoid negated checkbox descriptions.</div><div><br></div><div>The translation_domain doesn't seem to be set, so kded/kcm won't load anything</div><div><br></div><div>touchpad backend.h:</div><div>This class shouldn't be instantiated directly, don't make the constructor public.</div><div><br></div><div>The X backend:</div><div>This looked scary so I didn't review it at all.</div>​</div>