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

Re: Status of multi-monitor support

2008-10-15 Thread Guillaume Pothier
2008/10/15 Aaron J. Seigo <[EMAIL PROTECTED]>: > other than shitty drivers? ;) > > hm.. if the containment is the right size and so is the view ... dunno; > usually that's a driver issue. is it just physically cut off or is it just too > small? if you are using kwin desktop effects, does it work be

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

Re: Plasma::Svg and performance

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Marco Martin wrote: > On Wednesday 15 October 2008, Aaron J. Seigo wrote: > > On Wednesday 15 October 2008, Marco Martin wrote: > > > with a kconfigroup? would make sense? > > > > it's the easy answer, so i'd say: "let's go for it". > > > > we can benchmark it once it

Re: Status of multi-monitor support

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Guillaume Pothier wrote: > tell it to take a snapshot of the window under the cursor). The scene > rect is also set correctly, and the containment also. So if the view's > size, the containment's size and the scene rect are correct, is there > anything else that could

Re: Status of multi-monitor support

2008-10-15 Thread Guillaume Pothier
Now I have an intermittent problem with the panel... Panel is at the bottom of screen #1, which by default is 1680x1050. If I change the resolution to 1280x1024, the panel is resized ok. But when I switch back to 1680x1050, only the leftmost 1280 pixels are displayed, for the rest I can see the wal

Re: Status of multi-monitor support

2008-10-15 Thread Guillaume Pothier
On Wed, Oct 15, 2008 at 3:11 PM, <[EMAIL PROTECTED]> wrote: > Looks like arace condition ... Kwin updates first so plasma is not adjusted > yet and so the "is it the size of a screen" test fails. > > I guess the question is if kwin should be trying to manage the root window at > all. Either it s

Re: Plasma::Svg and performance

2008-10-15 Thread Marco Martin
On Wednesday 15 October 2008, Aaron J. Seigo wrote: > On Wednesday 15 October 2008, Marco Martin wrote: > > with a kconfigroup? would make sense? > > it's the easy answer, so i'd say: "let's go for it". > > we can benchmark it once it is in place and working. changing the cache > implementation wil

Re: plasma 4.2 feature freeze meeting

2008-10-15 Thread Richard Moore
I can do 15:00 UTC Saturday. Rich. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Status of multi-monitor support

2008-10-15 Thread aseigo
Looks like arace condition ... Kwin updates first so plasma is not adjusted yet and so the "is it the size of a screen" test fails. I guess the question is if kwin should be trying to manage the root window at all. Either it shouldn't touch anything or we will need a new window hint for rootwi

Re: Status of multi-monitor support

2008-10-15 Thread Guillaume Pothier
On Wed, Oct 15, 2008 at 2:18 PM, Aike J Sommer <[EMAIL PROTECTED]> 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 resizin

Re: Plasma::Svg and performance

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Jamboarder wrote: > > From: Aaron J. Seigo <[EMAIL PROTECTED]> > > > >> setImagePath (if the path is different that the current one) > > > > this only affects Plasma::SharedSvgRenderer usage, and is local to any > > given Plasma::Svg object. > > Might the pixmap cache

Re: Plasma::Svg and performance

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Riccardo Iaconelli wrote: > On Wednesday 15 October 2008 17:58:51 Jamboarder wrote: > > Should a clear/refresh be triggered if the path hasn't changed but the > > actual file has? > > Yes, I think so. Is a file watcher expensive? not really .. > Can't we simply use o

Re: Status of multi-monitor support

2008-10-15 Thread Aike J Sommer
> > 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 (ge

Re: Plasma::Svg and performance

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Marco Martin wrote: > with a kconfigroup? would make sense? it's the easy answer, so i'd say: "let's go for it". we can benchmark it once it is in place and working. changing the cache implementation will be an easy task after we have it there in the first place. i

Re: Plasma::Svg and performance

2008-10-15 Thread Jamboarder
> From: Aaron J. Seigo <[EMAIL PROTECTED]> >> setImagePath (if the path is different that the current one) > this only affects Plasma::SharedSvgRenderer usage, and is local to any given > Plasma::Svg object. Might the pixmap cache end up holding data for svg paths that are no longer being used b

Re: plasma 4.2 feature freeze meeting

2008-10-15 Thread Riccardo Iaconelli
On Wednesday 15 October 2008 17:47:37 Marco Martin wrote: > On Tuesday 14 October 2008, Aaron J. Seigo wrote: > > hi all.. > > > > i'd like to host a meeting on irc this weekend to go over feature freeze > > goals for plasma in KDE 4.2. > > > > does 15:00 UTC work for people? > > hope i can be ther

Re: Plasma::Svg and performance

2008-10-15 Thread Marco Martin
On Wednesday 15 October 2008, Aaron J. Seigo wrote: > > > (maybe imited to a fixed number of > > elements say 100 to not make it grow forever) > > it will need to be a fixed size, yes, even without non-existent elements. > 100 is probably too small though. this isn't a huge amount of data, really.

Re: Status of multi-monitor support

2008-10-15 Thread Guillaume Pothier
2008/10/15 Aaron J. Seigo <[EMAIL PROTECTED]>: > so something is misidentifying screen 0 when setting the geometry of the view. > look in DesktopView::adjustSize; it's either getting the wrong screen() value > or the wrong geometry for that screen. As far as I can tell, adjustSize gets screen ids

Re: Status of multi-monitor support

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Aike J Sommer wrote: > Am Mittwoch 15 Oktober 2008 18:15:27 schrieb Guillaume Pothier: > > Right, I have more information now, but I need some help. The problem > > that manifests in the screenshot I sent earlier is that one > > DesktopView occupies the whole display (

Re: Plasma::Svg and performance

2008-10-15 Thread Riccardo Iaconelli
On Wednesday 15 October 2008 17:58:51 Jamboarder wrote: > Should a clear/refresh be triggered if the path hasn't changed but the > actual file has? Yes, I think so. Is a file watcher expensive? Can't we simply use one on the theme's SVGs here? Bye, -Riccardo -- GPG key: 3D0F6376 When encryptin

Re: Status of multi-monitor support

2008-10-15 Thread Guillaume Pothier
On Wed, Oct 15, 2008 at 1:27 PM, Aike J Sommer <[EMAIL PROTECTED]> 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! Where should I have a look? And what actually is

Re: Status of multi-monitor support

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Guillaume Pothier wrote: > Right, I have more information now, but I need some help. The problem > that manifests in the screenshot I sent earlier is that one > DesktopView occupies the whole display (ie both screens) instead of > just one screen. Something causes it t

Re: Plasma::Svg and performance

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Jamboarder wrote: > > From: Aaron J. Seigo <[EMAIL PROTECTED]> > > one outstanding issue will be clearing the disk and parameter caches when > > the theme is updated on disk but not changed in the config. > > > > before i go any further with this approach: any thoughts

Re: Status of multi-monitor support

2008-10-15 Thread Aike J Sommer
Am Mittwoch 15 Oktober 2008 18:15:27 schrieb Guillaume Pothier: > Right, I have more information now, but I need some help. The problem > that manifests in the screenshot I sent earlier is that one > DesktopView occupies the whole display (ie both screens) instead of > just one screen. Something ca

Re: Plasma::Svg and performance

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Marco Martin wrote: > ok, this patch is the idea, it seems to work, still has two problems dunno > how relevant: > -when checking for element existence it computes the rect, a bit of a > waste, but this way don't have to keep two separate caches for rect and for > exis

Re: Plasma::Svg and performance

2008-10-15 Thread Aaron J. Seigo
On Wednesday 15 October 2008, Alexis Ménard wrote: > I will try it...Some other parts like icons in the taskbar or in the pager > kill performance. (KIconLoarder takes forever) are you sure it's KIconLoader in that case? and do you have any numbers on the performance impact of them? -- Aaron J.

Re: Status of multi-monitor support

2008-10-15 Thread Guillaume Pothier
Right, I have more information now, but I need some help. The problem that manifests in the screenshot I sent earlier is that one DesktopView occupies the whole display (ie both screens) instead of just one screen. Something causes it to resize to 3600x1200, which is my whole desktop's geometry. Ho

Re: Plasma::Svg and performance

2008-10-15 Thread Jamboarder
> From: Aaron J. Seigo <[EMAIL PROTECTED]> > one outstanding issue will be clearing the disk and parameter caches when the > theme is updated on disk but not changed in the config. > > before i go any further with this approach: any thoughts, ideas, inspirations? > It might be useful in general

Re: plasma 4.2 feature freeze meeting

2008-10-15 Thread Marco Martin
On Tuesday 14 October 2008, Aaron J. Seigo wrote: > hi all.. > > i'd like to host a meeting on irc this weekend to go over feature freeze > goals for plasma in KDE 4.2. > > does 15:00 UTC work for people? hope i can be there yeah should be ok ___ Plasma-

Re: Plasma::Svg and performance

2008-10-15 Thread Marco Martin
On Wednesday 15 October 2008, Aaron J. Seigo wrote: > On Tuesday 14 October 2008, Aaron J. Seigo wrote: > > before i go any further with this approach: any thoughts, ideas, > > inspirations? > > i'd also be interested in people applying the patch locally and ensuring it > doesn't break anything for

Re: Plasma::Svg and performance

2008-10-15 Thread Marco Martin
On Wednesday 15 October 2008, Marco Martin wrote: > On Wednesday 15 October 2008, Aaron J. Seigo wrote: > > hi .. > > > > doing some profiling with valgrind/massif, i noticed that a large amount > > of start up processing and memory usage is spent in parsing SVGs. > > surprise! ;) > > > > i introdu

Re: Plasma::Svg and performance

2008-10-15 Thread Marco Martin
On Wednesday 15 October 2008, Alexis Ménard wrote: > I will try it...Some other parts like icons in the taskbar or in the pager > kill performance. (KIconLoarder takes forever) doesn't taskbar and pager use the pixmap coming from the windows as icons? > 2008/10/15 Aaron J. Seigo <[EMAIL PROTECTED]

Re: Plasma::Svg and performance

2008-10-15 Thread Marco Martin
On Wednesday 15 October 2008, Aaron J. Seigo wrote: > hi .. > > doing some profiling with valgrind/massif, i noticed that a large amount of > start up processing and memory usage is spent in parsing SVGs. surprise! ;) > > i introduced a KPixmapCache for on-disk caching to help prevent the need to >

Re: Plasma::Svg and performance

2008-10-15 Thread Alexis Ménard
I will try it...Some other parts like icons in the taskbar or in the pager kill performance. (KIconLoarder takes forever) 2008/10/15 Aaron J. Seigo <[EMAIL PROTECTED]> > On Tuesday 14 October 2008, Aaron J. Seigo wrote: > > before i go any further with this approach: any thoughts, ideas, > > insp