Bug 372685

Summary: Oxygen theme style configuration icon out of place
Product: [Plasma] kwin Reporter: Marax Faraii <maraxfaraii>
Component: decorationsAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: kainz.a, simonandric5
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Image showing the configuration icon

Description Marax Faraii 2016-11-20 05:15:44 UTC
Created attachment 102325 [details]
Image showing the configuration icon

Apologies if this is the wrong place of filing this so please move it to appropriate location.

Now, to the bug. Best way to describe it is in the title as well as attached screenshot.

In the window decoration menu there are configuration icons above all the various styles, except for oxygen. 
If the window is maximized the icon is visible but when it's not the icon doesn't show at all.

Plasma version: 5.8.3
Kde Frameworks: 5.28.0
QT version: 5.6.2
Comment 1 Christoph Feck 2016-11-21 03:14:55 UTC
The configure icon is placed relative to the total decoration size (including shadows).
Comment 2 Martin Flöser 2016-11-21 07:59:27 UTC
@Andreas: can you have a look. That seems to be related to your QML change. If you don't know what it caused, please bug either Marco or David about it ;-)
Comment 3 Marax Faraii 2016-11-21 08:48:58 UTC
Don't know if this will be of any use but on my other laptop, running similar setup the icon also is out of place. However, not as much. Just slightly below widgetstyle.

The screen size is: 15in and 1366x768 

The one in the image is:
17in 1920x1080

Perhaps the misplacing of the icon is with scaling?
Comment 4 David Edmundson 2016-11-28 09:25:25 UTC
Git commit d0e0f6c88cdd032d3bdb9dc7b0c85902fb3d859a by David Edmundson.
Committed on 28/11/2016 at 09:24.
Pushed by davidedmundson into branch 'Plasma/5.8'.

Align configure button consistently across delegates

Summary:
Currently the configure button is aligned to the win decoration in the
background. This is bound to 40 - the inactive decorations shadow.

This means the configure button ends up in different places across each
item,
looking a bit weird and potentially not even on top of the right
delegate.

This patch aligns the configure button relative to the overall delegate
so that it's in the same place on every item

Test Plan: Looked at KCM

Reviewers: #plasma

Subscribers: plasma-devel, kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D3526

M  +3    -2    kcmkwin/kwindecoration/qml/Previews.qml

https://commits.kde.org/kwin/d0e0f6c88cdd032d3bdb9dc7b0c85902fb3d859a