Status of multi-monitor support

[Guillaume Pothier]

On Wed, Oct 15, 2008 at 2:18 PM, Aike J Sommer <dev at aikesommer.name> wrote:
>> > I think the problem is kwin forcing plasma onto that size... I commited a
>> > patch a while back to allow one root-window per screen, but this seems to
>> > not work on resizing!
>>
>> ugh ... what is kwin doing resizing anything in the first place?
>
>     if( isDesktop())
>         {
> ->      if (geometry() == workspace()->clientArea( ScreenArea, this ))
>            {
>            return;
>            }
> <-
>         QRect area = workspace()->clientArea( FullArea, this );
>         if( geometry() != area )
>             setGeometry( area );
>
>
> Is the part of geometry.cpp and there is similar in manage.cpp. The lines
> between -> and <- is what i added, but depending on wether kwin or plasma
> will get the resize first this will fail!
> Why kwin needs to resize desktop-type windows, i dont know!
>

As an experiment, I applied this patch to kdebase/workspace/kwin/geometry.cpp:
@@ -993,12 +989,12 @@
     {
     if( isDesktop())
         {
-        if (geometry() == workspace()->clientArea( ScreenArea, this
))
-            {
-            return;
-            }
+//         if (geometry() == workspace()->clientArea( ScreenArea,
this ))
+//             {
+//             return;
+//             }

-        QRect area = workspace()->clientArea( FullArea, this );
+        QRect area = workspace()->clientArea( ScreenArea, this );
         if( geometry() != area )
             setGeometry( area );
         return;


It resolved the DesktopView resizing to 3600x1200! Now, I need some
info on whether it is a correct fix or not, and if more is needed.

Cheers,
g