[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 241332] [regression] Panels messed up when the second screen is
From:       "Anselmo L.S.Melo (anselmolsm)" <anselmolsm () gmail ! com>
Date:       2010-07-31 4:38:54
Message-ID: 20100731043854.6BDA25773A () immanuel ! kde ! org
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=241332





--- Comment #23 from Anselmo L. S. Melo (anselmolsm) <anselmolsm gmail com>  2010-07-31 06:38:51 ---
Ok, I spent some time reading this code and making tests / checking debug
output / trying to fix it, with no success until now/

I'll divide the results in 2 cases:

1) The transition between 2 monitors -> 1 monitor happens while plasma-desktop
is not running (i.e., planels placed by setupDesktop)

One thing I noticed is that the in this case the comparison pv->screen() ==
screen->id() in canRelocatePanel is always false, what makes this method
returns true. So, once we have a panel candidate to be relocated, it is going
to be relocated, independent of the other conditions.

Without that verification, panel overlaps does not happen anymore.
Panels in different positions than the ones in the primary screen are
relocated.

---

2) The second screen is turned off while plasma-desktop is running (i.e.,
screenRemoved in action):

pv->screen() == screen->id() is always true (at least it happened in every
test). The problem seems to be pv->geometry().intersects(newGeom) because its
result change during the process, probably due to an update in the geometry of
the panel being analysed.

----

In both cases, if the extra monitor reappear, nothing happens with the panels
(the ones originally from the extra screen are not reset to the original
position).

---

Well, this is what I found today. Consider that it was my first contact with
this part of plasma code, so I probably skipped something important in these
tests. 


A question that comes to my mind is: Why one would be interested in mixing
panels from different screens when running the "single screen"? Besides that,
when the 2 monitors setup is restored, panels need to be moved manually.
I don't know, it sounds weird and I got the feeling it is over engineering to
solve a very particular use case.

Well, that's all for today, I'm sleepy, I can't even review this text, sorry in
advance about any stupid thing written here.

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread]