Bug 313091 (Kded-appmenu) - Kded-appmenu - Enabling makes "Window menu" and "on all desktops" buttons vanish
Summary: Kded-appmenu - Enabling makes "Window menu" and "on all desktops" buttons vanish
Status: RESOLVED FIXED
Alias: Kded-appmenu
Product: kwin
Classification: Plasma
Component: appmenu (show other bugs)
Version: git master
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Cédric Bellegarde
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-11 17:49 UTC by SpeccyMan
Modified: 2013-01-12 09:13 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description SpeccyMan 2013-01-11 17:49:58 UTC
When activating the feature (configure desktop>Application Appearance>Style>Fine Tuning> MenuBar) if "Title bar Button" is chosen this makes the "Window menu" and "on all desktops" buttons disapear. 

Even if all the other options are chosen the "Window menu" and "on all desktops" buttons  never reapear.

Reproducible: Always

Steps to Reproduce:
1.Go to "configure desktop>Application Appearance>Style>Fine Tuning> MenuBar" and choose "Title bar Button"
2.Apply changes

Actual Results:  
"Window menu" and "on all desktops" buttons from the window frame disapear

Expected Results:  
the new button should be added to the right side of the "Window menu" and "on all desktops" buttons
Comment 1 Cédric Bellegarde 2013-01-11 18:17:40 UTC
The bug is in kde-workspace/kcontrol/style
Comment 2 Cédric Bellegarde 2013-01-11 18:22:59 UTC
Can't reproduce here, switching from "In application" to "Title bar Button" add appmenu button on the left as wanted by code...

What is the output of:
$ grep ButtonsOn .kde*/share/config/kwinrc

before enabling appmenu ?
Comment 3 SpeccyMan 2013-01-11 18:46:02 UTC
(In reply to comment #2)
> Can't reproduce here, switching from "In application" to "Title bar Button"
> add appmenu button on the left as wanted by code...
> 
> What is the output of:
> $ grep ButtonsOn .kde*/share/config/kwinrc
> 
> before enabling appmenu ?

sorry, i'm kind of new in this "report bugs" thing :)

i've tried what you said on konsole but i've got nothing as result
Comment 4 Cédric Bellegarde 2013-01-11 18:48:05 UTC
Try this two commands:
$ grep ButtonsOn ~/.kde/share/config/kwinrc
$ grep ButtonsOn ~/.kde4/share/config/kwinrc

You should get something like this:
[gnumdk@cassiope ~]$ grep ButtonsOn .kde*/share/config/kwinrc 
ButtonsOnLeft=F
ButtonsOnRight=I_X
Comment 5 Cédric Bellegarde 2013-01-11 18:50:03 UTC
Ok, i know where is the bug ;) Will fix it soon...
Comment 6 SpeccyMan 2013-01-11 18:56:38 UTC
(In reply to comment #5)
> Ok, i know where is the bug ;) Will fix it soon...

good to know you found it because i'm still not able to get any result. LOL. 
I've even opened the file in question trying to find some [ButtonsOn] or something but had no luck in finding such name.

Glad i helped anyway. (This OpenSuse ideia about a bootable live system for tests is cool)
Comment 7 Thomas Lübking 2013-01-11 21:02:36 UTC
(In reply to comment #4)
> Try this two commands:
> $ grep ButtonsOn ~/.kde/share/config/kwinrc
> $ grep ButtonsOn ~/.kde4/share/config/kwinrc

ftr:
kreadconfig --file kwinrc --group Style --key ButtonsOnLeft
kreadconfig --file kwinrc --group Style --key ButtonsOnRight

while i keep telling ppl. to grep ~/.kde{4}/* that's not correct, because the kdedir could be anywhere and the settings in multiple layers (/usr/share, i think Ubuntu uses that quite some)
Comment 8 Cédric Bellegarde 2013-01-12 09:13:05 UTC
Git commit c4af201a4fd716b4cb98b64f484bceecb6c9af41 by Cedric Bellegarde.
Committed on 12/01/2013 at 10:09.
Pushed by cedric into branch 'master'.

M  +1    -1    kcontrol/style/kcmstyle.cpp

http://commits.kde.org/kde-workspace/c4af201a4fd716b4cb98b64f484bceecb6c9af41
Comment 9 Cédric Bellegarde 2013-01-12 09:13:46 UTC
Git commit c1c51c8b467b28aa17f31cf20627a7770172792d by Cedric Bellegarde.
Committed on 12/01/2013 at 10:10.
Pushed by cedric into branch 'KDE/4.10'.

M  +1    -1    kcontrol/style/kcmstyle.cpp

http://commits.kde.org/kde-workspace/c1c51c8b467b28aa17f31cf20627a7770172792d