broulik added a comment.

  It's not much code on our side because Qt has the code nowadays. :)


> graesslin wrote in FindQt5PlatformSupport.cmake:1
> given that it has my copyright I assume you copied from kwin - that would be 
> a reason to move it to ecm. We have two users.

+1 to that.

I had one issue regarding the include path, though, but I can't reproduce it 
anymore and I forgot what I actually did but the path I need is


but I sometimes got


but I'm sure we can figur that out.

> graesslin wrote in kdeplatformtheme.cpp:59
> can we really make that static? What about runtime changes?

That's what Qt does and I don't think Qt is equipped to re-create a platform 
menu at runtime; at least that's what I was told when I raised the very same 
concern in the upstream Qt codereview that added this.

We could perhaps do it for newly created menus but when the service becomes 
unavailable at runtime there's no way for us to re-create existing menus.

  rPLASMAINTEGRATION Integration for Qt applications in Plasma



To: broulik, #plasma
Cc: graesslin, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, 

Reply via email to