Bug 303090 - Remove Tiling Support From KWin
Summary: Remove Tiling Support From KWin
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: Quick Tiling (show other bugs)
Version: git master
Platform: unspecified Linux
: NOR task
Target Milestone: 4.10
Assignee: Martin Flöser
URL: https://git.reviewboard.kde.org/r/105...
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-06 05:55 UTC by Martin Flöser
Modified: 2012-07-14 09:18 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.10
Sentry Crash Report:
mgraesslin: ReviewRequest+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Flöser 2012-07-06 05:55:39 UTC
The window tiling support should be removed from KWin core. This has been discussed on the mailinglist [1] and the main reasons for removal are currently:

* code is unmaintained and previous developer does not want to work on it any more (asked at Akademy)
* does not support multi-screen
* conflicts with other parts of KWin (activities, tabbing)
* nobody of core dev group uses tiling

There is work going on to get tiling done in a KWin script, which sounds like a good replacement which could make it for 4.10.

[1] http://lists.kde.org/?l=kwin&m=133149673110558&w=2
Comment 1 Martin Flöser 2012-07-14 09:18:36 UTC
Git commit 34027455aaa2ee738c45987ca2d8cb7d65491bf0 by Martin Gräßlin.
Committed on 13/07/2012 at 09:53.
Pushed by graesslin into branch 'master'.

Remove Tiling support from KWin

As discussed on the mailinglist [1] the tiling support is
removed from KWin. The main reasons for this step are:
* it is unmaintained
* it is a mode not used by any of the core KWin team
* original developer said at Akademy 2012 that he is not
  interested in picking up the work again
* tiling has quite some bugs, e.g. multi screen not supported
* is conflicting with other concepts in KWin, e.g. activities

There is ongoing work to get tiling supported through a KWin
script, which is a preferred way as it does not influence the
existing C++ code base.

[1] http://lists.kde.org/?l=kwin&m=133149673110558&w=2
FIXED-IN: 4.10
REVIEW: 105546

M  +0    -22   kwin/CMakeLists.txt
M  +0    -1    kwin/config-kwin.h.cmake
M  +15   -57   kwin/geometry.cpp
M  +0    -93   kwin/kcmkwin/kwinoptions/windows.cpp
M  +0    -14   kwin/kcmkwin/kwinoptions/windows.h
M  +0    -16   kwin/kcmkwin/kwinrules/ruleswidget.cpp
M  +0    -2    kwin/kcmkwin/kwinrules/ruleswidget.h
M  +193  -263  kwin/kcmkwin/kwinrules/ruleswidgetbase.ui
M  +0    -343  kwin/kwin.notifyrc
M  +1    -0    kwin/libkdecorations/kdecoration.h
M  +1    -2    kwin/libkdecorations/kdecoration_p.cpp
M  +0    -3    kwin/notifications.cpp
M  +0    -1    kwin/notifications.h
M  +0    -34   kwin/options.cpp
M  +0    -52   kwin/options.h
M  +0    -4    kwin/org.kde.KWin.xml
M  +0    -7    kwin/rules.cpp
M  +0    -4    kwin/rules.h
M  +0    -17   kwin/sm.cpp
D  +0    -141  kwin/tiling/tile.cpp
D  +0    -125  kwin/tiling/tile.h
D  +0    -523  kwin/tiling/tiling.cpp
D  +0    -112  kwin/tiling/tiling.h
D  +0    -243  kwin/tiling/tilinglayout.cpp
D  +0    -148  kwin/tiling/tilinglayout.h
D  +0    -123  kwin/tilinglayoutfactory.cpp
D  +0    -79   kwin/tilinglayoutfactory.h
D  +0    -154  kwin/tilinglayouts/columns/columns.cpp
D  +0    -49   kwin/tilinglayouts/columns/columns.h
D  +0    -60   kwin/tilinglayouts/floating/floating.cpp
D  +0    -59   kwin/tilinglayouts/floating/floating.h
D  +0    -79   kwin/tilinglayouts/spiral/spiral.cpp
D  +0    -49   kwin/tilinglayouts/spiral/spiral.h
M  +0    -55   kwin/useractions.cpp
M  +0    -97   kwin/workspace.cpp
M  +0    -19   kwin/workspace.h

http://commits.kde.org/kde-workspace/34027455aaa2ee738c45987ca2d8cb7d65491bf0