<br><br><div class="gmail_quote">2009/2/1 Guillaume Pothier <span dir="ltr"><<a href="mailto:gpothier@gmail.com">gpothier@gmail.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Alessandro, sorry I didn't reply earlier,</blockquote><div><br> Hi Guillaume, np :)<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I was offline this week.<br>
I think your fix is not in the ideal place. I think that what happens<br>
is that you are using a driver that is not xrandr1.2-aware (please<br>
tell me if I'm wrong),</blockquote><div><br> how can I check that?<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> and therefore Kephal falls back to<br>
QDesktopWidget for screen management; unfortunately QDesktopWidget has<br>
this problem that it still reports the old screen information while<br>
the notifications are processed. But the fix for this should be in<br>
Kephal, not in plasma (otherwise the problem would reappear in other<br>
components such as kwin).</blockquote><div><br> I deeply checked and Kephal actually returns correct screen info (2 screens,<br> the first (0, 0, 1280, 1024), the second(1280, 0, 1680, 1050)). Moreover adjustSize() uses Plasma::View::screen()<br>
in order to resize the view, and before delaying adjustSize, that screen was just wrong.<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
I actually had a patch to fix this but I was waiting for somebody to<br>
test it before committing. Can you check this bug:<br>
<a href="http://bugs.kde.org/show_bug.cgi?id=181325" target="_blank">http://bugs.kde.org/show_bug.cgi?id=181325</a> and test the patch,<br>
reverting your own patch, and see if it works for you?</blockquote><div><br>
It does not do me the trick.. I still have second screen
displacement.. I feel the issues are not correlated, anyway i'd wait for<br>
someone else, more expert than me, to talk :) <br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
Cheers,<br>
g<br>
</blockquote><div><br>Cheers :)<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
2009/1/30 Alessandro Diaferia <<a href="mailto:alediaferia@gmail.com">alediaferia@gmail.com</a>>:<br>
<div><div></div><div class="Wj3C7c">> As you can see from the sources Plasma::View uses QTimer::singleShot(0,<br>
> this, SLOT(privateInit())) in the ctor but DesktopView calls adjustSize() in<br>
> the ctor with no delay..<br>
><br>
> This causes the fact that adjustSize() uses invalid information about the<br>
> current screen to setup its size and makes my second screen disappear (or<br>
> actually, with a wrong size/position).<br>
> I managed to fix this giving a delay also to adjustSize() so that when it is<br>
> called the correct information are already set (and screen() returns the<br>
> correct number in this case).<br>
><br>
> Hope this is ok to commit, since i'd like to have my screen back officially<br>
> :P<br>
><br>
> Since i'm having troubles with the reviewboard here comes the patch:<br>
><br>
> Index: shells/desktop/desktopview.cpp<br>
> ===================================================================<br>
><br>
> --- shells/desktop/desktopview.cpp (revision 918687)<br>
> +++ shells/desktop/desktopview.cpp (working copy)<br>
> @@ -104,7 +104,7 @@ DesktopView::DesktopView(Plasma::Contain<br>
> QBrush b(tile);<br>
> setBackgroundBrush(tile);<br>
><br>
><br>
> - adjustSize();<br>
> + QTimer::singleShot(0, this, SLOT(adjustSize()));<br>
><br>
> Kephal::Screens *screens = Kephal::Screens::self();<br>
> connect(screens, SIGNAL(screenResized(Kephal::Screen *, QSize, QSize)),<br>
><br>
><br>
> Hugs!!<br>
><br>
> --<br>
> Alessandro Diaferia<br>
><br>
</div></div><div><div></div><div class="Wj3C7c">> _______________________________________________<br>
> Plasma-devel mailing list<br>
> <a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
> <a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
><br>
><br>
_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Alessandro Diaferia<br>