Re: Review Request 109648: Implement the implicitWidth/implicitHeight of the chat plasmoid

2013-03-21 Thread Aleix Pol Gonzalez
> On March 21, 2013, 6:54 p.m., David Edmundson wrote: > > I'm very much against shipping any patches if we don't know why it fixes > > it. From what I can see from declarative/declarativeitemcontainer.cpp it > > just sets the minimumWidth to it... which I would have thought it would be > > do

Re: Review Request 109648: Implement the implicitWidth/implicitHeight of the chat plasmoid

2013-03-21 Thread Marco Martin
> On March 21, 2013, 6:54 p.m., David Edmundson wrote: > > I'm very much against shipping any patches if we don't know why it fixes > > it. From what I can see from declarative/declarativeitemcontainer.cpp it > > just sets the minimumWidth to it... which I would have thought it would be > > do

Re: Review Request 109648: Implement the implicitWidth/implicitHeight of the chat plasmoid

2013-03-21 Thread Àlex Fiestas
> On March 21, 2013, 7:11 p.m., Marco Martin wrote: > > Ship It! Marco, can you bring some light into David ramblings? Any doc on implicit/preferred/minimum width? - Àlex --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 109604: Model to provide easy access to KWin's Clients from QML

2013-03-21 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109604/#review29660 --- kwin/scripting/scripting_model.h

Re: Review Request 109648: Implement the implicitWidth/implicitHeight of the chat plasmoid

2013-03-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109648/#review29653 --- Ship it! Ship It! - Marco Martin On March 21, 2013, 5:37 p.

Re: Review Request 109648: Implement the implicitWidth/implicitHeight of the chat plasmoid

2013-03-21 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109648/#review29651 --- Ship it! I'm very much against shipping any patches if we don'

Re: Review Request 106110: Make autohide panel do not intercept events in fullscreen app

2013-03-21 Thread Thomas Lübking
> On March 20, 2013, 2 p.m., Aaron J. Seigo wrote: > > have you tested the following two use cases: > > > > * panel hides, a window becomes full screen at that point .. does the panel > > still unhide now? > > * window becomes full screen, panel hides (and with the patch no unhide > > trigger

Re: Review Request 106110: Make autohide panel do not intercept events in fullscreen app

2013-03-21 Thread Alexey Shmalko
> On March 20, 2013, 2 p.m., Aaron J. Seigo wrote: > > have you tested the following two use cases: > > > > * panel hides, a window becomes full screen at that point .. does the panel > > still unhide now? > > * window becomes full screen, panel hides (and with the patch no unhide > > trigger

Re: Review Request 109641: Fix border detection in Plasma dialogs

2013-03-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109641/ --- (Updated March 21, 2013, 5:41 p.m.) Status -- This change has been ma

Re: Review Request 109641: Fix border detection in Plasma dialogs

2013-03-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109641/#review29644 --- This review has been submitted with commit 4602e8155078a28968c

Review Request 109648: Implement the implicitWidth/implicitHeight of the chat plasmoid

2013-03-21 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109648/ --- Review request for Plasma, Telepathy and David Edmundson. Description

Re: Review Request 109641: Fix border detection in Plasma dialogs

2013-03-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109641/#review29641 --- Ship it! Ship It! - Marco Martin On March 21, 2013, 2:01 p.

Re: Participation in GSoC 2013

2013-03-21 Thread amanjot singh
On Thu, Mar 21, 2013 at 3:00 AM, Aaron J. Seigo wrote: > On Wednesday, March 20, 2013 23:29:51 amanjot singh wrote: > > It isn't fully developed yet, so I wanted to submit a proposal for the > same > > to make it a fully functional and a usable widget. > > What are the use cases you have in mind

Review Request 109641: Fix border detection in Plasma dialogs

2013-03-21 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109641/ --- Review request for Plasma. Description --- I had some problems with p

Re: Review Request 109595: active-welcome plasmoid: use plasmacomponents ToolButton

2013-03-21 Thread Michael Bohlender
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109595/ --- (Updated March 21, 2013, 1:02 p.m.) Review request for Plasma. Changes -

Re: Now Listen plasmoid

2013-03-21 Thread Alex Merry
On 21/03/13 09:48, Aaron J. Seigo wrote: On Thursday, March 21, 2013 00:00:52 you wrote: PlasmaCore.SvgItem { svg: Svg("nocover") } yes, or Svg { source: "nocover" } Does that really work? When I did Svg { imagePath: "nocover" }, QML complained that Svg is not a type (it's a global func

Re: Now Listen plasmoid

2013-03-21 Thread Aaron J. Seigo
On Thursday, March 21, 2013 11:09:29 Marco Martin wrote: > the svg could be searched over first in the theme, if not exists in > images/org.kde.nowplaying/foo.svg > as a fallback which will create an unexpected result if someone tries to name something widgets/foo.svg without realizing that there

Re: Nepomukcontroller rewritten in QML

2013-03-21 Thread Jörg Ehrichs
> > Well, the feeder agent's configure dialog can be invoked via D-Bus > See the configure method of interface org.freedesktop.Akonadi.Agent.Control > Thanks a lot. Sadly the akonadi feeders only option is the indle time delay. Not sure this is what the user expects, I've settle with kcm_akonadifo

Re: Now Listen plasmoid

2013-03-21 Thread Marco Martin
On Thursday 21 March 2013, Aaron J. Seigo wrote: > $PLASMOID_PACKAGE/images/ > $SYSTEM_SVG_THEME_PATH/ > $PLASMOID_PACKAGE/images/themed > > this could get tripped up if the file path is "themed/myimage.svg" but that > seems like a stretch .. we could have a completely different directory, > such

Re: Now Listen plasmoid

2013-03-21 Thread Aaron J. Seigo
On Thursday, March 21, 2013 00:00:52 you wrote: > PlasmaCore.SvgItem { > svg: Svg("nocover") > } yes, or Svg { source: "nocover" } > This seems to work, in that it finds content/images/nocover.svgz as > expected. However, the effect is exactly the same as my original code, > which was > > Pla

GSoC 2013

2013-03-21 Thread amanjot singh
This is Amanjot Singh, I took part in SoK last year, worked on the project "Global New Document Menu.." ( http://community.kde.org/GSoC/2012/Ideas#Project:_Global_.22New_document22_menu ) under Friedrich W. H. Kossebau. I developed a widget under this project and it is basically a template open