Re: AppletScript & configAccepted

2008-09-11 Thread Petri Damstén
On Friday 12 September 2008 06:24:08 Petri Damstén wrote: > patch for: > - configChanged functions in Applet and AppletScript > - connected to configXml configChanged() and applyClicked/okClicked (for > C++ applets) > - package typo mainconfiggui -> mainconfigui > - and some Q_UNUSED to remove warn

Re: QuickSand, an alternative interface

2008-09-11 Thread Ryan P. Bitanga
2008/9/11 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Wednesday 10 September 2008, Ryan P. Bitanga wrote: >> It's still experimental, and some things don't work (like the >> configuration dialog) but it should provide a slightly different >> experience from the current interface as well as a more fami

Re: QuickSand, an alternative interface

2008-09-11 Thread Ryan P. Bitanga
You're one to talk :p But trying it out for yourself is way better than a screenshot, or even a screencast. :) 2008/9/12 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Thursday 11 September 2008, Michael Rudolph wrote: >> A screenshot was posted earlier today (or tonight, yesterday or >> tomorrow, depen

Re: AppletScript & configAccepted

2008-09-11 Thread Petri Damstén
On Thursday 11 September 2008 17:47:27 Aaron J. Seigo wrote: > On Thursday 11 September 2008, Petri Damstén wrote: > > Applets connect to apply & ok to know when user accepted config, but if > > AppletScript uses configxml there seems to be no way to do that? Maybe > > configAccepted signal in appl

Re: AppletScript & configAccepted

2008-09-11 Thread Petri Damstén
On Thursday 11 September 2008 20:17:08 Aaron J. Seigo wrote: > *that* said ... what's wrong with simply having code/main the same as > html/index.html, and stuffing the rest of the html files into code/? I seem to lose color coding in e.g. kate if it's just main. Ruby (Ruboid) seems to specify it

Re: AppletScript & configAccepted

2008-09-11 Thread Petri Damstén
On Thursday 11 September 2008 22:06:42 Aaron J. Seigo wrote: > On Thursday 11 September 2008, Petri Damstén wrote: > > On Thursday 11 September 2008 20:17:08 Aaron J. Seigo wrote: > > > > - Background does not work currently. > > > > > > ... define? > > > > I use applet()->setBackgroundHints(Plasma

Re: Extender tutorial on techbase

2008-09-11 Thread Marco Martin
On Thursday 11 September 2008, Rob Scheepmaker wrote: > On Thursday 11 September 2008 17:06:10 Marco Martin wrote: > > with furter investigation i've found that it doesn't crash anymore if i > > remove line 194 of applet.cpp > > //destroy temporary extender items, or items that aren't detached, s

Re: Review Request: Clock: remove "Use local time zone", add enabling/disabling code for "clockDefaultsTo"

2008-09-11 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/189/#review186 --- Ship it! seems sensible to me - Marco On 2008-09-11 07:49:26,

Re: plasma design online

2008-09-11 Thread Riccardo Iaconelli
On Thursday 11 September 2008 20:25:04 Alex Merry wrote: > All the files are currently empty. Sorry, I had a little bug in the update script, try now. =) -Riccardo -- GPG key: 3D0F6376 When encrypting, please encrypt also for this subkey: 9EBD7FE1 - Pace Peace Paix Paz Frieden Pax Pokój Frið

Re: AppletScript & configAccepted

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Petri Damstén wrote: > On Thursday 11 September 2008 20:17:08 Aaron J. Seigo wrote: > > > - Background does not work currently. > > > > ... define? > > I use applet()->setBackgroundHints(Plasma::Applet::DefaultBackground); in > init but still get only white web page.

Re: plasma design online

2008-09-11 Thread Alex Merry
On Thursday 11 September 2008 14:04:18 Riccardo Iaconelli wrote: > hello, > so, I decided that we're not using markown for nothing, and now I'm > extracting readable webpages out of design file. Updated each 6 hours on > the hour from SVN, and yes, I need to put a better CSS there. All the files a

Re: AppletScript & configAccepted

2008-09-11 Thread Petri Damstén
On Thursday 11 September 2008 20:17:08 Aaron J. Seigo wrote: > personally, to keep the proper semantics in place for UI, config, etc, i'd > do something like: > > // copy the main applet structure > (*this) = Applet::packageStructure(); > > // now add our own stuff > addFileDefinition("webpage", "h

Re: AppletScript & configAccepted

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Petri Damstén wrote: > On Thursday 11 September 2008 17:47:27 Aaron J. Seigo wrote: > > there is a configChanged() signal from KCoreConfigSkeleton as well; we > > might be able to use that. it gets emitted when writeConfig() is > > called... s ... we could do sth

Re: QuickSand, an alternative interface

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Michael Rudolph wrote: > A screenshot was posted earlier today (or tonight, yesterday or > tomorrow, depending on time zone :-) here: > http://rbitanga.blogspot.com/2008/09/some-people-fall-in-love-and-touch-sky >.html kids and their blogs these days. ;) thanks... -

Re: AppletScript & configAccepted

2008-09-11 Thread Richard Dale
2008/9/11 Aaron J. Seigo <[EMAIL PROTECTED]> > On Thursday 11 September 2008, Richard Dale wrote: > > Then those wouldn't have any configXml object and this part of the logic > > would never be entered. I'm afraid I still don't see what the early exit > > code is doing. > > that branch is only ent

Re: Extender tutorial on techbase

2008-09-11 Thread Rob Scheepmaker
On Thursday 11 September 2008 17:06:10 Marco Martin wrote: > with furter investigation i've found that it doesn't crash anymore if i > remove line 194 of applet.cpp > //destroy temporary extender items, or items that aren't detached, so > their //configuration won't linger after a plasma restart.

Re: AppletScript & configAccepted

2008-09-11 Thread Petri Damstén
On Thursday 11 September 2008 17:47:27 Aaron J. Seigo wrote: > there is a configChanged() signal from KCoreConfigSkeleton as well; we > might be able to use that. it gets emitted when writeConfig() is called... > s ... we could do sth like: > > * connect d->configXml configChanged to a configCh

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Dario Freddi
> > On Thursday 11 September 2008 16:55:18 Aaron J. Seigo wrote: > > On Thursday 11 September 2008, Dario Freddi wrote: > > > Discussed on IRC and agreed only to return when reply is not valid, > > > included in the patch > > > > please be sure to add this to kdebase/workspace/plasma/design/CHAN

Re: QuickSand, an alternative interface

2008-09-11 Thread Michael Rudolph
2008/9/11 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Wednesday 10 September 2008, Ryan P. Bitanga wrote: >> It's still experimental, and some things don't work (like the >> configuration dialog) but it should provide a slightly different >> experience from the current interface as well as a more fami

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Sebastian Kügler
On Thursday 11 September 2008 16:55:18 Aaron J. Seigo wrote: > On Thursday 11 September 2008, Dario Freddi wrote: > > Discussed on IRC and agreed only to return when reply is not valid, > > included in the patch > > please be sure to add this to kdebase/workspace/plasma/design/CHANGELOG =) > > th

Re: [PATCH] Fifteen puzzle sizing change (and other fixes)

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Anthony Bryant wrote: > I was just wondering whether anyone's had any time to look at/try out my > patch. yes, i have. unfortunatley i'm working on file dialog stuff today and have been trying to keep patch merging down to just 1-3 per day. i do have your patch ope

Re: QuickSand, an alternative interface

2008-09-11 Thread Aaron J. Seigo
On Wednesday 10 September 2008, Ryan P. Bitanga wrote: > It's still experimental, and some things don't work (like the > configuration dialog) but it should provide a slightly different > experience from the current interface as well as a more familiar one > to QuickSilver users. screenshots or a

Re: Extender tutorial on techbase

2008-09-11 Thread Marco Martin
On Wednesday 10 September 2008, Marco Martin wrote: > On Wednesday 10 September 2008, Rob Scheepmaker wrote: > > On Wednesday 10 September 2008 15:02:23 Marco Martin wrote: > > > another problem: > > > adding a clock on the desktop when plasma is already started produces a > > > crash, at line 389

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Dario Freddi wrote: > Discussed on IRC and agreed only to return when reply is not valid, > included in the patch please be sure to add this to kdebase/workspace/plasma/design/CHANGELOG =) thanks for working on this ... great stuff. -- Aaron J. Seigo humru othro a

Re: AppletScript & configAccepted

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Richard Dale wrote: > Then those wouldn't have any configXml object and this part of the logic > would never be entered. I'm afraid I still don't see what the early exit > code is doing. that branch is only entered if there isn't a configXml + UI file. at which poin

Re: [PATCH] Fifteen puzzle sizing change (and other fixes)

2008-09-11 Thread Anthony Bryant
I was just wondering whether anyone's had any time to look at/try out my patch. Also, here's an updated diff that uses the current trunk. Thanks in advance, Anthony. Index: fifteen.cpp === --- fifteen.cpp (revision 859913) +++ fiftee

Review Request: Clock: remove "Use local time zone", add enabling/disabling code for "clockDefaultsTo"

2008-09-11 Thread Rafa? Mi?ecki
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/189/ --- Review request for Plasma. Summary --- This patch implements idea disc

Re: AppletScript & configAccepted

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Petri Damstén wrote: > Applets connect to apply & ok to know when user accepted config, but if > AppletScript uses configxml there seems to be no way to do that? Maybe > configAccepted signal in applet? I guess this could be good for Applets too > so they don't have t

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Dario Freddi
> On 2008-09-11 06:54:27, Sebastian Kügler wrote: > > Other than my comments, it looks good to me, though I wouldn't mind someone > > else having another look at it ... > > Alessandro Diaferia wrote: > the patch looks good to me too. just remove the whitespaces (highlighted > in red here).

Re: AppletScript & configAccepted

2008-09-11 Thread Richard Dale
2008/9/11 Aaron J. Seigo <[EMAIL PROTECTED]> > On Thursday 11 September 2008, Richard Dale wrote: > > 2008/9/11 Petri Damstén <[EMAIL PROTECTED]> > > > > > Hi, > > > > > > I have been playing with webkit AppletScript. There is a small example > > > (webkit/code/test.html) using plasma dataengines

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Alessandro Diaferia
> On 2008-09-11 06:54:27, Sebastian Kügler wrote: > > Other than my comments, it looks good to me, though I wouldn't mind someone > > else having another look at it ... the patch looks good to me too. just remove the whitespaces (highlighted in red here). =) - Alessandro --

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Dario Freddi
> On 2008-09-11 06:54:27, Sebastian Kügler wrote: > > /trunk/KDE/kdebase/workspace/plasma/dataengines/powermanagement/powermanagementengine.cpp, > > line 70 > > > > > > We can return here already, as the rest of the code

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Dario Freddi
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/188/ --- (Updated 2008-09-11 07:08:52.151574) Review request for Plasma. Summary -

Re: Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/188/#review182 --- Ship it! Other than my comments, it looks good to me, though I wo

Re: AppletScript & configAccepted

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Richard Dale wrote: > 2008/9/11 Petri Damstén <[EMAIL PROTECTED]> > > > Hi, > > > > I have been playing with webkit AppletScript. There is a small example > > (webkit/code/test.html) using plasma dataengines so I have added more > > functions (init(), engine.connectSo

Re: Review Request: [MOCKUP] Clock: remove "Use local time zone"

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Rafał Miłecki wrote: > Let's put QComboBox on bottom and let's enable it ONLY if there is at > least one time zone selected in KTimeZoneWidget? this implies always showing the local timezone in every clock. i guess i'm fine with that. -- Aaron J. Seigo humru othro

Review Request: Merging PowerDevil engine with base/powermanagementengine

2008-09-11 Thread Dario Freddi
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/188/ --- Review request for Plasma. Summary --- As agreed on IRC with sebas, I'

Re: Review Request: [MOCKUP] Clock: remove "Use local time zone"

2008-09-11 Thread Rafał Miłecki
2008/9/11 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Thursday 11 September 2008, Rafał Miłecki wrote: >> 2008/9/11 Hans Chen <[EMAIL PROTECTED]>: >> > As for the combobox, my first thought when I saw the picture was to put >> > it on top. However, I see a problem with this approach: it feels reversed

Re: Review Request: [MOCKUP] Clock: remove "Use local time zone"

2008-09-11 Thread Aaron J. Seigo
On Thursday 11 September 2008, Rafał Miłecki wrote: > 2008/9/11 Hans Chen <[EMAIL PROTECTED]>: > > As for the combobox, my first thought when I saw the picture was to put > > it on top. However, I see a problem with this approach: it feels reversed > > somehow. I think there is a high possibility

plasma design online

2008-09-11 Thread Riccardo Iaconelli
hello, so, I decided that we're not using markown for nothing, and now I'm extracting readable webpages out of design file. Updated each 6 hours on the hour from SVN, and yes, I need to put a better CSS there. Have a look: http://www.ruphy.org/plasma-design/ -Riccardo -- GPG key: 3D0F6376 When

Re: AppletScript & configAccepted

2008-09-11 Thread Richard Dale
2008/9/11 Petri Damstén <[EMAIL PROTECTED]> > Hi, > > I have been playing with webkit AppletScript. There is a small example > (webkit/code/test.html) using plasma dataengines so I have added more > functions (init(), engine.connectSource(), etc.) and plasma package support > so > it looks more li

Re: Review Request: [MOCKUP] Clock: remove "Use local time zone"

2008-09-11 Thread Rafał Miłecki
2008/9/11 Hans Chen <[EMAIL PROTECTED]>: > As for the combobox, my first thought when I saw the picture was to put it > on top. However, I see a problem with this approach: it feels reversed > somehow. I think there is a high possibility that users fail to see the > connection "Select a timezone i

Re: Review Request: [MOCKUP] Clock: remove "Use local time zone"

2008-09-11 Thread Hans Chen
2008/9/11 Sebastian Kügler <[EMAIL PROTECTED]> > On Wednesday 10 September 2008 23:12:30 Celeste Lyn Paul wrote: > > Checkboxes are much easier and intuitive for making multiple selections > > than the current selection method. As it is, theres no way to know you > can > > make multiple selection

Re: Review Request: [MOCKUP] Clock: remove "Use local time zone"

2008-09-11 Thread Sebastian Kügler
On Wednesday 10 September 2008 23:12:30 Celeste Lyn Paul wrote: > Checkboxes are much easier and intuitive for making multiple selections > than the current selection method.  As it is, theres no way to know you can > make multiple selections.  Checkboxes provide that "multi" affordance. There are

AppletScript & configAccepted

2008-09-11 Thread Petri Damstén
Hi, I have been playing with webkit AppletScript. There is a small example (webkit/code/test.html) using plasma dataengines so I have added more functions (init(), engine.connectSource(), etc.) and plasma package support so it looks more like a normal plasma applet. Applets connect to apply &