This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104427/

On March 27th, 2012, 5:21 p.m., Martin Gräßlin wrote:

kwin/geometry.cpp (Diff revision 1)
QRect Workspace::clientArea(clientAreaOption opt, int screen, int desktop) const
292
        if (is_multihead)
298
        return QRect(0, 0, displayWidth(), displayHeight());
293
            return Kephal::ScreenUtils::screenGeometry(screen_number);
294
        else
295
            return Kephal::ScreenUtils::desktopGeometry();
after thinking about it I could not see any difference any more. In both cases the complete display is returned.

On March 27th, 2012, 7:39 p.m., Thomas Lübking wrote:

Pot. related to how many screens kephal sees in the multihead (maybe even xinerama instead xrandr?) case.
If it sees the "other" screen as well, desktopGeometry() would largely differ from screenGeomtry() but latter be equal to display(Width|Height)()

So by dropping kephal, this pitfall could have implicitly be avoided.
-> Did you test on a multihead config (i'm tonight not in the mood to break my setup ;-)
of course I did not test on multihead. But my old laptop has a xorg.conf prepared for multihead and I could run a recompile there.

- Martin


On March 27th, 2012, 5:19 p.m., Martin Gräßlin wrote:

Review request for kwin and Alex Fiestas.
By Martin Gräßlin.

Updated March 27, 2012, 5:19 p.m.

Description

Kephal has turned into not being more than a wrapper around QDesktopWidget and does not even provide syntax sugar.

Testing

currently running kwin without kephal on a multiscreen setup - so far everything's fine.

Diffs

  • kwin/CMakeLists.txt (463692d)
  • kwin/effects/screenshot/screenshot.h (f05d0ec)
  • kwin/events.cpp (30350aa)
  • kwin/geometry.cpp (5c176b7)
  • kwin/kcmkwin/kwinoptions/CMakeLists.txt (319ab47)
  • kwin/kcmkwin/kwinoptions/windows.cpp (30c94c0)
  • kwin/kcmkwin/kwintabbox/CMakeLists.txt (42c0a00)
  • kwin/libkdecorations/CMakeLists.txt (739698e)
  • kwin/libkdecorations/kcommondecoration.cpp (02a33e9)
  • kwin/libkwineffects/CMakeLists.txt (9cd2a6b)
  • kwin/options.cpp (717a82d)
  • kwin/scene.cpp (4e69121)
  • kwin/screenedge.cpp (0b57185)
  • kwin/tabbox/declarative.cpp (c8abf60)
  • kwin/workspace.cpp (5542a3d)

View Diff