<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/114271/">http://git.reviewboard.kde.org/r/114271/</a>
</td>
</tr>
</table>
<br />
<p>On December 3rd, 2013, 5:35 p.m. CET, <b>Yichao Yu</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">May I ask if how did you get the rounded corner?
I have got this[1] kind of white frame for menu with rounded corder and haven't found a good way to get rid of it (without interfering with other things) yet.
[1] http://wstaw.org/m/2013/12/03/plasma-desktopuj5572.png
</pre>
</blockquote>
<p>On December 4th, 2013, 7:20 a.m. CET, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">It still has the white frame - might not be visible in the screenshot. The problem seems to me like an RGB window is created, though I haven't verified yet.</pre>
</blockquote>
<p>On December 4th, 2013, 7:53 p.m. CET, <b>Yichao Yu</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I think the problem is that the window has an opaque clear color (which also means that it is a RGB window) but I'm not sure what is the best way to fix it.
</pre>
</blockquote>
<p>On December 5th, 2013, 6:57 a.m. CET, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">yes that's clearly the problem and I'm also not sure yet how to fix it. For QWidget based menus it's AFAIK done by the style. But with the QWindow the style isn't asked to operate on it. I want to try whether we can do something like having a style hint to create ARGB menus.</pre>
</blockquote>
<p>On December 5th, 2013, 10:54 a.m. CET, <b>Yichao Yu</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">One thing I am trying, which is also necessary to get translucent background in Qt5 at all, is to set the correct clear colors on the QQuickWindow (which is a method of QQuickWindow and therefore need direct link to QtQuick) in a event callback. (More specifically, menus and combobox lists are QQuickMenuPopupWindow's, setting the clear color to #0000 does give me a real rounded corner and this also allows me to add shadow and blur properties to them)
Two problems for this:
1. I'm not sure if this is acceptable in KDE at all..... (too hacky). However, sth like this has to be done in order to get the rounded corner or (correct me if I'm wrong) is it possible with the default 24bit's GL window?
2. QtQuickControl seems to be VERY inconsistent on how it fills background. e.g. I don't think PE_Widget is used at all for the main window and QtQuickControl simply uses the clear colors on it, also the region in menu under items is filled with an opaque color after PE_PanelMenu is called (filling it with #0000 in PE_PanelMenu only makes a difference on the edge) so I can neither get any gradient or transparency on menus.
My hope is that some of you who can directly talk to Qt ppl can patch that up to allow polishing QQuickItem/Window and/or make it respect background rendering of the styles. :-)
</pre>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">> I don't think PE_Widget is used at all for the main window and QtQuickControl simply uses the clear colors on it
yes, the ApplicationWindow control doesn't have any style integration yet. It's on the TODO list...</pre>
<br />
<p>- Martin</p>
<br />
<p>On December 3rd, 2013, 2:57 p.m. CET, Martin Gräßlin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma, David Edmundson and Hugo Pereira Da Costa.</div>
<div>By Martin Gräßlin.</div>
<p style="color: grey;"><i>Updated Dec. 3, 2013, 2:57 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-workspace
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">drawFrameMenuPrimitive is entered with a null QWidget if a QQuickMenu is rendered. By handling this case we get the menu to render with a frame. Unfortunately we cannot render the widget background as that really needs a non-null QWidget and we also do not yet get shadows.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I'm not sure whether this can affect other areas if QWidget is null.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kstyles/oxygen/oxygenstyle.cpp <span style="color: grey">(50c217f)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/114271/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/12/03/32336a9f-5bd7-4f47-974f-3420882f7cc3__qml-menu-fixed.png">QtQuick menu with fix</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/12/03/45e57c8b-324d-4d9a-8fc8-52211f2e8f8d__qml-menu-broken.png">QtQuick menu without fix</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>