<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 07/11/2013 12:33 PM, Sebastian
Kügler wrote:<br>
</div>
<blockquote cite="mid:171701492.OGUO9rQXVX@monet" type="cite">
<pre wrap="">hi Hugo,
On Thursday, July 11, 2013 10:52:02 Hugo Pereira Da Costa wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Hi Everyone,
lately I've been working (so far in a local branch) on porting oxygen to
KF5/Qt5
Good news is: this is pretty much done
(<a class="moz-txt-link-freetext" href="http://wstaw.org/m/2013/07/11/plasma-desktopJw2526.png">http://wstaw.org/m/2013/07/11/plasma-desktopJw2526.png</a>)
except for a few glitches (see below).
</pre>
</blockquote>
<pre wrap="">Awesome. :)
</pre>
<blockquote type="cite">
<pre wrap="">Right now, there is only one place where oxygen still depends on
libkdeui, and it is when using kcolorbutton for configuring the shadow
button. All the rest is KF5 only.
Questions:
- is frameworks-crash the right kde-workspace branch to push it ?
</pre>
</blockquote>
<pre wrap="">frameworks-scratch, yes. :)</pre>
</blockquote>
ok. All pushed. In the end I kept the commits separated, despite the
back and forth, since all of them (I think) compile. <br>
Note: for some reason I cannot configure kde-workspace unless I
comment the line "find_package(KDeclarative QUIET CONFIG)
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<style type="text/css">
p, li { white-space: pre-wrap; </style>"<br>
in topLevel CMakeLists, which apparently conflicts with
<meta name="qrichtext" content="1">
"find_package(kdeclarative REQUIRED NO_MODULE)" some lines above.<br>
Since I am not sure whether it is related to my setup or a true flaw
in the makefiles, I have not pushed that change. <br>
<br>
Comments welcome.<br>
<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>
</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;">I'm open to bug reports or
comments on the code. Will go on debugging in the meanwhile.<br>
</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>
Hugo<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>
<blockquote cite="mid:171701492.OGUO9rQXVX@monet" type="cite">
<pre wrap="">
</pre>
<blockquote type="cite">
<pre wrap="">- how should the commits be organized ? is one fat 'initial' push ok ?
My own working on this has been: "disable this feature untill I get the
rest working, then reenable and fix it" so that I do not think this kind
of history is interesting ...
</pre>
</blockquote>
<pre wrap="">I don't have a strong preference for that. Whatever makes you feel better.
(I've been doing small commits to that branch.)
</pre>
<blockquote type="cite">
<pre wrap="">- is there some reviewing needed ? To be honest, I'd be grateful if
someone could give a quick look at my CMakeLists.txt, cause I might not
have gotten everything right.
</pre>
</blockquote>
<pre wrap="">If we can find someone to review, that would certainly be nice. Otherwise,
starting to use it is probably also a good idea to catch bugs.
</pre>
<blockquote type="cite">
<pre wrap="">The issues/features:
- I have only ported kde-workspace/libs/oxygen and
kde-workspace/kstyle/oxygen
- the code that pass menu shadows (in the form of pixmap handles as X
properties) to kwin has been disabled because: I do not know how to
paint on an X11 pixmap with Qt5, and anyway this will likely be
rewritten in kwin
- there is a slight glitch with the grab window from empty areas
feature, in the sense that after a grab, the next mouse press event is
ignored (still needs investigating).
- I'm uncertain about some of the options which have become obsoleted
from KGlobalSettings (and are said to be set via the style), and notably
where to read the settings from. In any case, that's no big deal and
I'll investigate further.
</pre>
</blockquote>
<pre wrap="">These need someone else to reply.
Cheers and thanks,
</pre>
</blockquote>
<br>
</body>
</html>