Bug 213353

Summary:	Desktop --> Multiple Desktop should have similar configuration capabilities as Pager Plasmoid
Product:	[Plasma] kwin	Reporter:	Ben Kevan <Ben.Kevan>
Component:	general	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	dpalacio, fest.in
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:	4.8.0
Sentry Crash Report:

Description Ben Kevan 2009-11-06 01:09:32 UTC

Version:           1.0 (using KDE 4.3.3)
OS:                Linux
Installed from:    openSUSE RPMs

I actually consider this a regression (bug) and not a feature request as there is lost functionality. 

I recently switched form GNOME to KDE (previously I was a KDE 3.5.10 user).. 

I went to configure "Multiple Desktops" with the KWin Compositing and was all happy.. Except I wanted 1 row of desktops with 4 desktops. 

I went into systemsettings --> Desktop --> Multiple Desktops, and the configuration was not there.. 

I later found out I had to setup the pager plasmoid (which I previously removed since I use the Kwin effects to change, and do not rely on the pager usage). However, I had to add the pager plasmoid to configure my multiple desktops to my configuration noted above. 

I then removed the panel plasmoid, and again my configurations were lost. 

It seems barbaric that to configure multiple desktops to my liking (and keep these settings) I am required to have the pager plasmoid running somewhere. These configurations should be set within systemsettings --> Desktop --> Multiple Desktops and should be configurable by both systemsettings and the pager plasmoid not by ONLY one or the other. 

In previous versions of KDE3 this was available within both.

Comment 1 Martin Flöser 2009-11-06 08:16:30 UTC

For KDE 4.4. it is planned to have the number of rows settings in the multiple desktop kcm. It is actually added but requires some work in Plasma. Unfortunately the relevant specification requires that the layout is set by the pager and not by the window manager. So even if the option is moved to multiple desktop kcm you will need the pager plasmoid.

Comment 2 David Palacio 2009-11-07 19:03:35 UTC

Are you talking about the "Desktop grid" effect? You can configure the layout to use: 1. the pager 2. "automatic" 3. personalized.

Comment 3 Ben Kevan 2009-11-07 21:02:53 UTC

David, 

That's the point, you're "required" to have the pager plasmoid, which shouldn't be a requirement for kcm style features (ie they were in kcm along with pager in KDE 3). I understand that work between kcm / plasmoids is currently underway. 

Anyway.. I'm talking about any "workspace" layout, and not just the desktop grid. The same configuration is true with "Desktop Cube" along with the "Desktop Grid".

Comment 4 Fest 2010-07-23 23:16:45 UTC

There is option to configure virtual desktop in system settings (KDE 4.4.92). But it's still buggy. 
For example, if you delete pager widget, and configure only in system settings: choose custom layout (1 row), desktop grid show desktops in 1 row. But if you try to slide between desktops(mouse wheel),you slide in rows !

Comment 5 Martin Flöser 2011-07-31 15:01:43 UTC

Git commit 4bf7761307a2453a63d918c60e13c26980727830 by Martin Gräßlin.
Committed on 24/07/2011 at 14:59.
Pushed by graesslin into branch 'master'.

Move control of desktop layout from pager to KWin

The desktop layout (number of rows) should be
controlled by the window manager as it is also
responsible for the number of desktops and the
names of them.

The setting for the rows is moved from the pager
UI to the virtual desktops KCM. The desktop layout
is set when KWin starts and updated by the KCM.

With this change there is no process claiming the
manager selection for pager any more. This means
the KDE Plasma Workspaces are no longer compliant
to the complete section of _NET_DESKTOP_LAYOUT in
the EWMH.

REVIEW: 102073
BUG: 277965
FEATURE: 105779
FEATURE: 213353
FIXED-IN: 4.8.0

M  +10   -0    kwin/workspace.cpp
M  +2    -2    kwin/kcmkwin/kwindesktop/main.ui
M  +4    -57   plasma/desktop/applets/pager/pagerConfig.ui
M  +30   -8    kwin/kcmkwin/kwindesktop/main.cpp
M  +0    -3    plasma/desktop/applets/pager/pager.h
M  +10   -47   plasma/desktop/applets/pager/pager.cpp

http://commits.kde.org/kde-workspace/4bf7761307a2453a63d918c60e13c26980727830

Comment 6 Martin Flöser 2012-04-13 19:24:35 UTC

Bulk change: move all KWin kcm bugs to product kwin