Bug 195745 - Having the list of virtual desktops sorted is not intuitive for switching between desktops
Summary: Having the list of virtual desktops sorted is not intuitive for switching bet...
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: general (other bugs)
Version First Reported In: unspecified
Platform: unspecified Linux
: LO normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
: 216418 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-06-09 09:43 UTC by Christian Trippe
Modified: 2009-11-27 20:05 UTC (History)
1 user (show)

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


Attachments
screenshot showing the relevant kwin shortcuts (186.24 KB, image/png)
2009-06-09 19:31 UTC, Christian Trippe
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Trippe 2009-06-09 09:43:03 UTC
Version:           Unbekannt (using 4.2.88 (KDE 4.2.88 (KDE 4.3 >= 20090527)) "release 135", KDE:KDE4:Factory:Desktop / openSUSE_11.1)
Compiler:          gcc
OS:                Linux (i686) release 2.6.27.21-0.1-default

I have configured Strg+Tab to switch between desktops. It is in my opinion very confusing that the list of virtual desktops ,which is displayed when pressing this longer, is in numerical order and not in the order of desktops recently used. It is very confusing to see the marker jumping (randomly)around, when going through the list.

I do not use any desktop effects.
Comment 1 Christian Trippe 2009-06-09 12:24:45 UTC
Just for clarification: I reported this as bug and not as wish, because this is a regression, as it works in KDE 4.1. Sorry for not adding this in the description.
Comment 2 Andreas Pakulat 2009-06-09 14:01:58 UTC
I don't see how this would be a wish, as far as I understood the reporter the problem is that the visual ordering of the list is different from the order in which Strg+Tab walks through the list. Which is clearly a bug, even more so if it worked properly in KDE 4.1.
Comment 3 Martin Flöser 2009-06-09 14:48:39 UTC
sorry I didn't see that it is a regression - I thought it should change the behaviour and that would be a feature request.

There are btw two different functionalites: walk through desktops and walk through desktop list. The one is jumpy, the other follows the ordering. IIRC that was already the behaviour in 4.2 but I will test.
Comment 4 Christian Trippe 2009-06-09 19:19:32 UTC
(In reply to comment #3)
> sorry I didn't see that it is a regression - I thought it should change the
> behaviour and that would be a feature request.

This was my fault, the description was definitely lacking the fact that this is a regression.

> 
> There are btw two different functionalites: walk through desktops and walk
> through desktop list. The one is jumpy, the other follows the ordering. IIRC
> that was already the behaviour in 4.2 but I will test.

I am using the function "(Zwischen Arbeitsflächen umschalten" (switch between desktops).

Btw, there are two functions in the shortcut setting , which allow to "walk through desktop list". See screenshot
1) Nächster Eintrag in Arbeitsflächenliste (Next entry in desktop list)
2) Zur nächsten Arbeitsfläche umschalten (Change to next desktop)

However only the first one displays a list, the second one just walks through the desktops. Do you want another bug report?
Comment 5 Christian Trippe 2009-06-09 19:31:54 UTC
Created attachment 34390 [details]
screenshot showing the relevant kwin shortcuts
Comment 6 Martin Flöser 2009-06-09 20:06:30 UTC
(In reply to comment #4)
> Btw, there are two functions in the shortcut setting , which allow to "walk
> through desktop list". See screenshot
> 1) Nächster Eintrag in Arbeitsflächenliste (Next entry in desktop list)
> 2) Zur nächsten Arbeitsfläche umschalten (Change to next desktop)
> 
> However only the first one displays a list, the second one just walks through
> the desktops. Do you want another bug report?
There is a third option "Zwischen Arbeitsflächen umschalten". Number one and three should show the box.

Btw I am no able to reproduce the issue and can confirm the regression although I don't understand it.
Comment 7 Martin Flöser 2009-09-06 23:35:30 UTC
will be definately fixed for 4.4 with the new tabbox implementation. I hope to figure out why it doesn't work in 4.3 and to provide a patch.
Comment 8 Martin Flöser 2009-09-13 13:36:52 UTC
SVN commit 1022861 by graesslin:

Here comes the new TabBox. It is a complete rewrite using a MVC approach. Here some highlights:
 * Models and Delegates for Clients and Desktops
 * Horizontal, vertical and tabular layout
 * Layout of one item can be configured by an XML definition
 * A desktop item can include a client list
 * An optional second list view showing only the selected item
 * A new KCM "kwintabbox"
 * An alternative TabBox with independent settings and keybindings
 * Optional Highlight Windows effect integration
 * List scrolls instead of removing items
 * Scroll wheel support
 * Cursor key support
 * Middle click on item closes window
BUG: 195745
BUG: 197187
BUG: 201103
FEATURE: 118184
FEATURE: 156723
FEATURE: 177441
FEATURE: 182897
FEATURE: 193882
GUI:

 M  +12 -2     CMakeLists.txt  
 M  +6 -0      client.cpp  
 M  +8 -0      client.h  
 M  +23 -15    effects/boxswitch/boxswitch.cpp  
 M  +3 -0      effects/boxswitch/boxswitch.h  
 M  +8 -2      effects/coverswitch/coverswitch.cpp  
 M  +2 -0      effects/coverswitch/coverswitch.h  
 M  +3 -2      effects/flipswitch/flipswitch.cpp  
 M  +1 -0      effects/flipswitch/flipswitch.h  
 M  +4 -0      effects/highlightwindow/highlightwindow.cpp  
 M  +4 -3      effects/presentwindows/presentwindows.cpp  
 M  +1 -0      effects/presentwindows/presentwindows.h  
 M  +1 -0      kcmkwin/CMakeLists.txt  
 M  +32 -26    kcmkwin/kwincompositing/main.cpp  
 A             kcmkwin/kwintabbox (directory)  
 A             kcmkwin/kwintabbox/CMakeLists.txt  
 A             kcmkwin/kwintabbox/kwintabbox.desktop  
 A             kcmkwin/kwintabbox/layoutconfig.cpp   [License: GPL (v2+)]
 A             kcmkwin/kwintabbox/layoutconfig.h   [License: GPL (v2+)]
 A             kcmkwin/kwintabbox/layoutconfig.ui  
 A             kcmkwin/kwintabbox/main.cpp   [License: GPL (v2+)]
 A             kcmkwin/kwintabbox/main.h   [License: GPL (v2+)]
 A             kcmkwin/kwintabbox/main.ui  
 A             kcmkwin/kwintabbox/previewhandlerimpl.cpp   [License: GPL (v2+)]
 A             kcmkwin/kwintabbox/previewhandlerimpl.h   [License: GPL (v2+)]
 M  +2 -0      kwinbindings.cpp  
 M  +7 -1      lib/kwinglobals.h  
 A             tabbox (directory)  
 M  +391 -898  tabbox.cpp  
 M  +64 -120   tabbox.h  
 A             tabbox/DefaultTabBoxLayouts.xml  
 A             tabbox/clientitemdelegate.cpp   [License: GPL (v2+)]
 A             tabbox/clientitemdelegate.h   [License: GPL (v2+)]
 A             tabbox/clientmodel.cpp   [License: GPL (v2+)]
 A             tabbox/clientmodel.h   [License: GPL (v2+)]
 A             tabbox/desktopitemdelegate.cpp   [License: GPL (v2+)]
 A             tabbox/desktopitemdelegate.h   [License: GPL (v2+)]
 A             tabbox/desktopmodel.cpp   [License: GPL (v2+)]
 A             tabbox/desktopmodel.h   [License: GPL (v2+)]
 A             tabbox/itemlayoutconfig.cpp   [License: GPL (v2+)]
 A             tabbox/itemlayoutconfig.h   [License: GPL (v2+)]
 A             tabbox/tabboxconfig.cpp   [License: GPL (v2+)]
 A             tabbox/tabboxconfig.h   [License: GPL (v2+)]
 A             tabbox/tabboxhandler.cpp   [License: GPL (v2+)]
 A             tabbox/tabboxhandler.h   [License: GPL (v2+)]
 A             tabbox/tabboxview.cpp   [License: GPL (v2+)]
 A             tabbox/tabboxview.h   [License: GPL (v2+)]
 M  +17 -2     useractions.cpp  
 M  +2 -2      workspace.cpp  
 M  +14 -4     workspace.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1022861
Comment 9 Martin Flöser 2009-11-27 20:05:48 UTC
*** Bug 216418 has been marked as a duplicate of this bug. ***