<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 01/04/2014 08:52 PM, David Faure
      wrote:<br>
    </div>
    <blockquote cite="mid:2261823.bpXgEodEF5@asterix" type="cite">
      <pre wrap="">On Saturday 04 January 2014 20:19:43 Hugo Pereira Da Costa wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">The first problem is that with kf5, the style configuration kcm expects
a libkstyle_oxygen_config.so *plugin* for this configuration, whereas it
was "kstyle_oxygen_config.so" in kde4 times.

now, as the error message indicates: plugins should not start with "lib"
and thus, oxygen still installs kstyle_oxygen_config.so only
so that the ill-named plugin is not found, and you get the error dialog.

To fix,
- either we fix the kcm by changing the looked for plugin name (by
removing the lib prefix)
</pre>
      </blockquote>
      <pre wrap="">
Yes. Why does the kcm look for a lib prefix now? Surely there's no good reason 
for that, the kde4 way was fine.

</pre>
      <blockquote type="cite">
        <pre wrap="">(I think it was added in the first place due to the lack of replacement
for kde4_add_plugin)
</pre>
      </blockquote>
      <pre wrap="">
But we can still remove the prefix, can't we? You do so for oxygen :)</pre>
    </blockquote>
    yes:<br>
    <meta name="qrichtext" content="1">
    <p style=" margin-top:0px; margin-bottom:0px; margin-left:0px;
      margin-right:0px; -qt-block-indent:0; text-indent:0px;
      -qt-user-state:0;"><!--StartFragment-->add_library(kstyle_oxygen_config
      MODULE ...)</p>
    <p style="-qt-paragraph-type:empty; margin-top:0px;
      margin-bottom:0px; margin-left:0px; margin-right:0px;
      -qt-block-indent:0; text-indent:0px;"><br>
      <!--EndFragment--></p>
    <meta http-equiv="Content-Type" content="text/html;
      charset=ISO-8859-1">
    <style type="text/css">
p, li { white-space: pre-wrap; }
</style><br>
    <blockquote cite="mid:2261823.bpXgEodEF5@asterix" type="cite">
      <pre wrap="">

</pre>
      <blockquote type="cite">
        <pre wrap="">For the second issue,
QStyle::SH_Widget_Animate is set to false not in oxygen style but in
KStyle by itself.
This is unrelated to the style configuration.

I am not too inclined to set it to true, since I am absolutely unclear
what this style hint is meant for. Can someone explain ?
</pre>
      </blockquote>
      <pre wrap="">
It should be true on fast local desktops, and false on slow/remote desktops, I 
suppose. IOW user configurable.</pre>
    </blockquote>
    ok. So this should go to kstyle (nothing oxygen explicit in there).<br>
    <br>
    In fact, kstyle returns:<br>
    <br>
    <meta name="qrichtext" content="1">
    <p style=" margin-top:0px; margin-bottom:0px; margin-left:0px;
      margin-right:0px; -qt-block-indent:0; text-indent:0px;
      -qt-user-state:0;"><!--StartFragment--> KConfigGroup
      g(KSharedConfig::openConfig(), "KDE-Global GUI Settings");</p>
    <p style=" margin-top:0px; margin-bottom:0px; margin-left:0px;
      margin-right:0px; -qt-block-indent:0; text-indent:0px;
      -qt-user-state:0;"> return g.readEntry("GraphicEffectsLevel", 0);</p>
    <p style="-qt-paragraph-type:empty; margin-top:0px;
      margin-bottom:0px; margin-left:0px; margin-right:0px;
      -qt-block-indent:0; text-indent:0px;"><br>
      mmm. No clue which KCM sets this :)<br>
      (but that answers Dominik's original question I guess)<br>
      <br>
      <!--EndFragment--></p>
    <meta http-equiv="Content-Type" content="text/html;
      charset=ISO-8859-1">
    <style type="text/css">
p, li { white-space: pre-wrap; }
</style><br>
    <blockquote cite="mid:2261823.bpXgEodEF5@asterix" type="cite">
      <pre wrap="">

</pre>
      <blockquote type="cite">
        <pre wrap="">Notably, I
don't think it should be set to true for _all_ widgets, since most
widgets are already animated internally by oxygen (and thus should not
be by the app), though it indeed should return "true" for
KMessageWidget, which is not animated internally.

So: should I enable the flag only for widgets that oxygen does not
animate itself ?
</pre>
      </blockquote>
      <pre wrap="">
No, I think it's rather unrelated. The animations from the widgets themselves 
are (as can be found by grepping qtbase)
- opening trees in QTreeView
- moving tabs in QTabBar
- dockareas and toolbars

Does oxygen animate any of that?
I'd be surprised, since these were already animated in Qt4.
What's new is letting users turn it off, via kstyle using kconfig.
</pre>
    </blockquote>
    Nope does not.<br>
    Thanks for the explanation.<br>
    <blockquote cite="mid:2261823.bpXgEodEF5@asterix" type="cite">
      <pre wrap="">
</pre>
    </blockquote>
    <br>
  </body>
</html>