<div dir="ltr">On Mon, Aug 5, 2013 at 11:02 AM, Kevin Krammer <span dir="ltr"><<a href="mailto:krammer@kde.org" target="_blank">krammer@kde.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thursday, 2013-08-01, Aurélien Gâteau wrote:<br>
<br>
> Should work, but extending the QPlatformIntegration class for such a<br>
> feature feels a bit overkill to me. Is there a simpler alternative?<br>
<br>
</div>If at all then QPlatformIntegration is probably the wrong place, more like<br>
QPlatformTheme (which handles things more similar like native menu, native<br>
dialogs).<br>
<br>
Interestingly there must be another type of runtime loading somewhere because<br>
the QKdeTheme can obviously not depend on any KDE code at build time.<br>
<br>
I wonder if syntax highlighting is not something that should be up to the<br>
style?<br>
<br>
Cheers,<br>
Kevin<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Kevin Krammer, KDE developer, xdg-utils developer<br>
KDE user support, developer mentoring<br>
</font></span><br>_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
<br></blockquote></div><br></div><div class="gmail_extra">Our QPlatformTheme does depend on KF5, and I don't see how we're going to be able to work without that dependency, we'll just have to release it together with Qt. Let's work with this assumption...</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">On the other hand, I agree that doing this kind of things is a bit at the limit of what QPT does... I don't know how much of a hack is to have this in KStyle...</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Aleix</div></div>