Re: Review Request: Bug with Plasma::Delegate text painter

2008-11-21 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/274/#review268 --- Ship it! - Aaron On 2008-11-20 13:06:28, Jesse Zamora wrote: >

Re: JOLIE and Plasma: status and discussion

2008-11-21 Thread Aaron J. Seigo
something else i just remembered: we're going to want to think about the full round trip "experience": use case: widget wants to use a web service * it says "give me this web service" -> that should be a one liner with no jargon in it, just the URL of the service and return a Plasma::Service re

Re: Panel offenders

2008-11-21 Thread Guillaume Pothier
2008/11/21 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Friday 21 November 2008, Guillaume Pothier wrote: >> I would add Grid to this list, it currently messes up the panel when >> added to it. > > focus on what we're actually shipping first, please. Oops I didn't notice this was from playground, sorr

Re: JOLIE and Plasma: status and discussion

2008-11-21 Thread Aaron J. Seigo
On Thursday 20 November 2008, Fabrizio Montesi wrote: > --- > --- Service Exposure > --- > Example of service exposure: > Plasma::Service *s = ...; //Some plasma service obtained in some way. > Plasma::ServicePublishingJob *j = s->publish(); > > Here Plasma::Service::publish() is a wrapper for: >

Re: [PATCH] update desktop wallpaper outside plasma

2008-11-21 Thread Aaron J. Seigo
On Thursday 20 November 2008, sycao wrote: > Hi, does anyone has an requirement that be able to change wallpaper > outside plasma. This is important functionality, but the approach taken in your patch is not correct. I don't blame you, though, as the design of Plasma makes the DBus part of it a

Re: Panel offenders

2008-11-21 Thread Aaron J. Seigo
On Friday 21 November 2008, Guillaume Pothier wrote: > I would add Grid to this list, it currently messes up the panel when > added to it. focus on what we're actually shipping first, please. -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1

Re: Panel offenders

2008-11-21 Thread Guillaume Pothier
On Fri, Nov 21, 2008 at 11:44 AM, Marco Martin <[EMAIL PROTECTED]> wrote: > Hi all, > i see that some applet still screw up when added in the panel, a thing i would > like to have for 4.2 is a good behaviour of all the shipped as default ones, > from a quick test the offenders aren't many and solut

Review Request: make Calendar a popupÂpplet

2008-11-21 Thread Loic Marteau
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/276/ --- Review request for Plasma. Summary --- Hey Marco Here is the code to

Re: Panel offenders

2008-11-21 Thread Marco Martin
On Friday 21 November 2008, Loïc Marteau wrote: > Perhaps we can share the work on this > > I can take care of of the simplest case, I will start with calendar. ok, so to recap, you try to make the calendar a popupapplet? the browser is dispatched, extender applet, rssnow and nowplaying will be

Re: Panel offenders

2008-11-21 Thread Artur Souza(MoRpHeUz)
On Friday 21 November 2008 15:21:27 Alex Merry wrote: > Can you file a bug again widgets-nowplaying, please? If you could include > what version of Amarok you're using, that would be great. Sure =). Just in advance, it's svn's amarok.. Cheers and thanks! ---

Re: Panel offenders

2008-11-21 Thread Alex Merry
On Friday 21 November 2008 18:16:55 Artur Souza(MoRpHeUz) wrote: > Hi Alex, > > On Friday 21 November 2008 15:13:55 Alex Merry wrote: > > Sorry, I've been meaning to fix this for a while. I'll try to get down > > to it this weekend. > > I've a strange bug here regarding the now playing applet: som

Re: Panel offenders

2008-11-21 Thread Artur Souza(MoRpHeUz)
Hi Alex, On Friday 21 November 2008 15:13:55 Alex Merry wrote: > Sorry, I've been meaning to fix this for a while. I'll try to get down to > it this weekend. I've a strange bug here regarding the now playing applet: sometimes the play/pause button get disabled and because of that I have to dire

Re: Panel offenders

2008-11-21 Thread Alex Merry
On Friday 21 November 2008 14:44:44 Marco Martin wrote: > nowplayingsome options: popupthing as usual or display just the current > track or display just play/stop buttons (only buttins would be the saner?) Sorry, I've been meaning to fix this for a while. I'll try to get down to it this wee

Re: Panel offenders

2008-11-21 Thread Marco Martin
On Friday 21 November 2008, Loïc Marteau wrote: > Perhaps we can share the work on this yeah > I can take care of of the simplest case, I will start with calendar. weel, not soo simple: pretty immediate to make it a popupapplet, but just using the standard icon we have a calendar that always says

Re: Panel offenders

2008-11-21 Thread Marco Martin
On Friday 21 November 2008, Rob Scheepmaker wrote: > On Friday 21 November 2008 15:44:44 Marco Martin wrote: > > Hi all, > > i see that some applet still screw up when added in the panel, a thing i > > would like to have for 4.2 is a good behaviour of all the shipped as > > default ones, > > A very

Re: Panel offenders

2008-11-21 Thread Rob Scheepmaker
On Friday 21 November 2008 15:44:44 Marco Martin wrote: > Hi all, > i see that some applet still screw up when added in the panel, a thing i > would like to have for 4.2 is a good behaviour of all the shipped as > default ones, A very sensible idea :) > rssnow: also here just display a single ne

Re: Panel offenders

2008-11-21 Thread Loïc Marteau
Perhaps we can share the work on this I can take care of of the simplest case, I will start with calendar. Cheers Lo On Fri, Nov 21, 2008 at 3:44 PM, Marco Martin <[EMAIL PROTECTED]> wrote: > Hi all, > i see that some applet still screw up when added in the panel, a thing i would > like to have

Panel offenders

2008-11-21 Thread Marco Martin
Hi all, i see that some applet still screw up when added in the panel, a thing i would like to have for 4.2 is a good behaviour of all the shipped as default ones, from a quick test the offenders aren't many and solutions could be pretty simple (enough to be "bugfixes, eheh:") twitter popup

Re: Review Request: Bug with Plasma::Delegate text painter

2008-11-21 Thread Jesse Zamora
> On 2008-11-20 13:38:23, Aaron Seigo wrote: > > /trunk/KDE/kdelibs/plasma/delegate.cpp, lines 300-328 > > > > > > there's no point in painting this if the text fits, right? that would > > avoid painting this for ever

Re: Review Request: Bug with Plasma::Delegate text painter

2008-11-21 Thread Jesse L. Zamora
> there's no point in painting this if the text fits, right? that would > avoid painting this for every single item. > > > - Aaron Yes, that's why I check before painting the gradient if the titleRect or subTitleRect is too long, and if it is, I draw the gradient. I've used it here for awhi

Re: bug update

2008-11-21 Thread Petri Damstén
On Wednesday 19 November 2008 19:33:54 Aaron J. Seigo wrote: > > > what is the resolution of each of your monitors? > > > > Two 1280x1024 monitors. Played more with the panels: - remove /share/config/plasma-appletsrc - start plasma - add panel to second screen - move it to right side plasma(1791