Re: Toolbox positioning issue (yes, with multiple monitors...)

2008-10-21 Thread Guillaume Pothier
On Tue, Oct 21, 2008 at 9:18 AM, Thomas Fjellstrom <[EMAIL PROTECTED]> wrote: > On Thursday 16 October 2008, Guillaume Pothier wrote: >> 2008/10/16 Aaron J. Seigo <[EMAIL PROTECTED]>: >> > just curious (since i haven't actually tested it myeslf yet and need to >> > step out here for a bit): did you

Re: Toolbox positioning issue (yes, with multiple monitors...)

2008-10-21 Thread Thomas Fjellstrom
On Thursday 16 October 2008, Guillaume Pothier wrote: > 2008/10/16 Aaron J. Seigo <[EMAIL PROTECTED]>: > > just curious (since i haven't actually tested it myeslf yet and need to > > step out here for a bit): did you also test placement of the toolbox when > > zoomed out with your patch? > > It wor

Re: Toolbox positioning issue (yes, with multiple monitors...)

2008-10-16 Thread Guillaume Pothier
2008/10/16 Aaron J. Seigo <[EMAIL PROTECTED]>: > > just curious (since i haven't actually tested it myeslf yet and need to step > out here for a bit): did you also test placement of the toolbox when zoomed > out with your patch? It works as expected: the code for scaled mode was already correct! g

Re: Toolbox positioning issue (yes, with multiple monitors...)

2008-10-16 Thread Aaron J. Seigo
On Thursday 16 October 2008, Guillaume Pothier wrote: > Ok, committed. That should fix that bug: > http://bugs.kde.org/show_bug.cgi?id=166770 (the BUG: in the commit > message resulted in a mail sent to me saying I don't have enough > privileges to mark the bug fixed). just curious (since i haven'

Re: Toolbox positioning issue (yes, with multiple monitors...)

2008-10-16 Thread Guillaume Pothier
Ok, committed. That should fix that bug: http://bugs.kde.org/show_bug.cgi?id=166770 (the BUG: in the commit message resulted in a mail sent to me saying I don't have enough privileges to mark the bug fixed). Cheers, g 2008/10/15 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Wednesday 15 October 2008,

Re: Toolbox positioning issue (yes, with multiple monitors...)

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Guillaume Pothier wrote: > Hi, > In a multimonitor setup the desktop containment's toolbox are not well > positioned. Sometimes they appear on only one monitor, sometimes both > appear at the same corner, or at some weird position. > > The issue is in ContainmentPriva

Toolbox positioning issue (yes, with multiple monitors...)

2008-10-15 Thread Guillaume Pothier
Hi, In a multimonitor setup the desktop containment's toolbox are not well positioned. Sometimes they appear on only one monitor, sometimes both appear at the same corner, or at some weird position. The issue is in ContainmentPrivate::positionToolBox(), the toolbox is positioned in the screens co