Re: [nepomuk-kde] Plasma activities and Nepomuk

[Hari krishna Anandhan]

On Sun, Aug 9, 2009 at 1:09 PM, Marco Martin wrote:
> there was also the idea of plasmoids changing their contents on
> activity change,that now i suppose would be from activity type
> change... (and the activity name being just a mnrmonic name for the
> user)

Activity name is unique for each activity and activity type is shared
between related activities (as there can be different activities with
the same type). So, I would say that plasmoids might use either
activity name or activity type depending upon the plasmoid's
purpose...

- Listing plasmoids - which just list things specific to the acitvity
- like contacts, resources associated with the activity, would filter
using the activity name
- Communication plasmoids - like Mail, etc would depend upon activity type

We would have to come up with a list of scenarios, plasmoids and their
usage to conclusively answer this ;)

Cheers,
Hari
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: [nepomuk-kde] Plasma activities and Nepomuk

[Hari krishna Anandhan]

Hi Leo,

On Sun, Aug 9, 2009 at 12:02 AM, Leo Sauermann wrote:
> I think you guys still did not check out the links I have posted,
> because if you did, you would be talking about "UserWorkContextThreads"
> which are the high-level "user activity" you are talking about.

Actually I had read them completely! Last time, me and Sebastian Trüg
had agreed that ContextThreads (given in that onto) are more aligned
with tracking NOPs (medium and long term), and are not currently
suited to what we need right now! I am sure we would be using most of
it at later phases when we bring NOPs into the picture, but for now we
have much simpler needs ;)

Cheers,
Hari
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Chani]

On August 10, 2009 08:28:47 Chani wrote:
> On August 10, 2009 00:13:41 Patrick Aljord wrote:
> > This is kind of unrelated but wouldn't it be a good thing to turn the
> > upload backends into a plasma data engine? I can see other plasmoids
> > making use of uploading pics and even kde apps like ksnapshot but I
> > have no idea whether kde apps can make use of plasma engines.
> 
> +1
> 
> and I believe there was some vague intention to someday make it possible
>  for other apps to access dataengines; I don't think anyone's had time to
>  actively work on it though?

correction: any app can of course load dataengines via libplasma, like 
plasmaengineexplorer does. what they can't do atm is share one engine over 
several processes; each process runs its own instance of the dataengine.

-- 
This message brought to you by eevil bananas and the number 3.
www.chani3.com


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

[tutorial][python] problems with self.engine.connectSource

[Maik Beckmann]

Hello,

The code in this tutorial
  - http://techbase.kde.org/Development/Tutorials/Plasma/PythonPlasmoid
after getting it to work by changing
  charge = data[QString("Charge Percent")].toInt()[0]
to
  charge = data[QString("Charge Percent")] 
gives
{{{ 
Connecting to battery 
/org/freedesktop/Hal/devices/computer_power_supply_battery_BAT0
Unrecognized character: /   
 
Unrecognized character: /   
 
ERROR: syntax error
}}}

The line which triggers this is 
  self.engine.connectSource(battery, self)

What's wrong?

-- Maik

PS; kde-4.3 on arch linux.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Artur Souza (MoRpHeUz)]

On Monday 10 August 2009, 14:00 Michal Dutkiewicz wrote:
> By the way, I could try to rewrite this applet from scratch and add for
>  example support for error messages (SIH returns different error messages
>  for example), that could be passed as signal parameter.

Actually rewriting it to support more features is probably not the best way to 
go (I've been there once hehe). The correct way should be to separate this and 
use the plasma infrastructure (dataengines/services) to have the backends.

Because every time a new pastebin/imagebin server shows up with a new feature 
and then we have to update this thing. Also, the way it is now makes it hard to 
extend. So, if you want to rewrite, let's start by separating the backend's 
code from the frontend. And then this time we can make it work properly :) What 
do you think ?

Cheers,

--
Artur Duque de Souza
openBossa Research Labs
INdT - Instituto Nokia de Tecnologia
--
Blog: http://blog.morpheuz.cc
PGP: 0xDBEEAAC3 @ wwwkeys.pgp.net
--


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Michal Dutkiewicz]

> On 2009-08-10 16:51:06, Artur de Souza (MoRpHeUz) wrote:
> > /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp,
> >  line 57
> > 
> >
> > Minor but important: not sure if kdeplasma-addons have coding style 
> > guidelines but following kdelibs, "if" statements should have the "{" on 
> > the same line :)
> >

Fixed in new diff.

Personally I don't use in my work that coding style (I was using it for some 
years, but new line is more readable for me and not only me ;-)).

By the way, I could try to rewrite this applet from scratch and add for example 
support for error messages (SIH returns different error messages for example), 
that could be passed as signal parameter.


- Michal


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1251/#review1964
---


On 2009-08-10 16:56:55, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1251/
> ---
> 
> (Updated 2009-08-10 16:56:55)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This patch adds support for Simplest Image Hosting 
> (http://simplest-image-hosting.net/) to pastebin applet.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinConfig.ui 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui 
> 1008396 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp
>  PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/CMakeLists.txt 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/backends.h 1008396 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.h 
> PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/1251/diff
> 
> 
> Testing
> ---
> 
> Works fine except showing error icon even if upload is successful (there is 
> success notification and link is added to history).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Michal Dutkiewicz]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1251/
---

(Updated 2009-08-10 16:56:55.220998)


Review request for Plasma.


Changes
---

Fixed coding style.


Summary
---

This patch adds support for Simplest Image Hosting 
(http://simplest-image-hosting.net/) to pastebin applet.


Diffs (updated)
-

  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinConfig.ui 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui 1008396 
  
/trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp 
PRE-CREATION 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/CMakeLists.txt 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/backends.h 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.h 
PRE-CREATION 

Diff: http://reviewboard.kde.org/r/1251/diff


Testing
---

Works fine except showing error icon even if upload is successful (there is 
success notification and link is added to history).


Thanks,

Michal

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Artur de Souza (MoRpHeUz)]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1251/#review1964
---



/trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp


Minor but important: not sure if kdeplasma-addons have coding style 
guidelines but following kdelibs, "if" statements should have the "{" on the 
same line :)



- Artur


On 2009-08-10 16:41:16, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1251/
> ---
> 
> (Updated 2009-08-10 16:41:16)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This patch adds support for Simplest Image Hosting 
> (http://simplest-image-hosting.net/) to pastebin applet.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/CMakeLists.txt 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/backends.h 1008396 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.h 
> PRE-CREATION 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp
>  PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinConfig.ui 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui 
> 1008396 
> 
> Diff: http://reviewboard.kde.org/r/1251/diff
> 
> 
> Testing
> ---
> 
> Works fine except showing error icon even if upload is successful (there is 
> success notification and link is added to history).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Michal Dutkiewicz]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1251/
---

(Updated 2009-08-10 16:41:16.451067)


Review request for Plasma.


Changes
---

Fixed showing error notification on success.
This was indeed second call of that method with empty data (strange, there 
should be no redirection, there was comment about that in imageshack backend).


Summary
---

This patch adds support for Simplest Image Hosting 
(http://simplest-image-hosting.net/) to pastebin applet.


Diffs (updated)
-

  /trunk/KDE/kdeplasma-addons/applets/pastebin/CMakeLists.txt 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/backends.h 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.h 
PRE-CREATION 
  
/trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp 
PRE-CREATION 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinConfig.ui 1008396 
  /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui 1008396 

Diff: http://reviewboard.kde.org/r/1251/diff


Testing
---

Works fine except showing error icon even if upload is successful (there is 
success notification and link is added to history).


Thanks,

Michal

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Michal Dutkiewicz]

> On 2009-08-10 12:00:23, Artur de Souza (MoRpHeUz) wrote:
> > /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp,
> >  line 54
> > 
> >
> > maybe you get the error post icon even after a successful post because 
> > it tries to retrieve more data and then you fall in this place. not sure 
> > though and right now I don't have the time to test. Check form where 
> > postError() is being emited on your backend code. (probably from here)

Thanks for the hint, I'll test it with qDebug(). ;-)


> On 2009-08-10 12:00:23, Artur de Souza (MoRpHeUz) wrote:
> > /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui, line 
> > 57
> > 
> >
> > With more plugins being added we need to properly fix this config tab 
> > now. We postponed this as much as we could but now we should do it 
> > properly. I'll come with something later (plane to tokamak? hehe).

Yes, this should be done cleaner, maybe it is possible to create kind of 
service for backends and use them through data engine, as mentioned earlier?


- Michal


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1251/#review1950
---


On 2009-08-09 21:08:23, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1251/
> ---
> 
> (Updated 2009-08-09 21:08:23)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This patch adds support for Simplest Image Hosting 
> (http://simplest-image-hosting.net/) to pastebin applet.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/CMakeLists.txt 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/backends.h 1008396 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.h 
> PRE-CREATION 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp
>  PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinConfig.ui 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui 
> 1008396 
> 
> Diff: http://reviewboard.kde.org/r/1251/diff
> 
> 
> Testing
> ---
> 
> Works fine except showing error icon even if upload is successful (there is 
> success notification and link is added to history).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: activities support in opendesktop widget

[Marco Martin]

> On 2009-08-10 15:43:54, Chani wrote:
> > /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.cpp, line 51
> > 
> >
> > just a little detail... using [] will create the key if it doesn't 
> > exist, so generally using .value("foo") is cheaper. probably doesn't matter 
> > in this case, just thought I'd point it out.

yeah, in this case ["message"] should always exist but good point


- Marco


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1253/#review1957
---


On 2009-08-10 10:54:19, Marco Martin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1253/
> ---
> 
> (Updated 2009-08-10 10:54:19)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> no, not plasma activities :p
> this adds the friends activity support to the opendesktop plasmoid and 
> selects it by default, so it doesn't look as a static thing anymore but shows 
> some "news"
> 
> still to do:
> configurable refresh time
> avatar pictures to make it more kolorful
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/CMakeLists.txt 1009325 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.h 
> PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.cpp 
> PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.h 1009325 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.cpp 1009325 
> 
> Diff: http://reviewboard.kde.org/r/1253/diff
> 
> 
> Testing
> ---
> 
> 
> Screenshots
> ---
> 
> 
>   http://reviewboard.kde.org/r/1253/s/170/
> 
> 
> Thanks,
> 
> Marco
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: activities support in opendesktop widget

[Chani]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1253/#review1957
---



/trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.cpp


just a little detail... using [] will create the key if it doesn't exist, 
so generally using .value("foo") is cheaper. probably doesn't matter in this 
case, just thought I'd point it out.


- Chani


On 2009-08-10 10:54:19, Marco Martin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1253/
> ---
> 
> (Updated 2009-08-10 10:54:19)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> no, not plasma activities :p
> this adds the friends activity support to the opendesktop plasmoid and 
> selects it by default, so it doesn't look as a static thing anymore but shows 
> some "news"
> 
> still to do:
> configurable refresh time
> avatar pictures to make it more kolorful
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/CMakeLists.txt 1009325 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.h 
> PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.cpp 
> PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.h 1009325 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.cpp 1009325 
> 
> Diff: http://reviewboard.kde.org/r/1253/diff
> 
> 
> Testing
> ---
> 
> 
> Screenshots
> ---
> 
> 
>   http://reviewboard.kde.org/r/1253/s/170/
> 
> 
> Thanks,
> 
> Marco
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Chani]

On August 10, 2009 00:13:41 Patrick Aljord wrote:
> This is kind of unrelated but wouldn't it be a good thing to turn the
> upload backends into a plasma data engine? I can see other plasmoids
> making use of uploading pics and even kde apps like ksnapshot but I
> have no idea whether kde apps can make use of plasma engines.

+1

and I believe there was some vague intention to someday make it possible for 
other apps to access dataengines; I don't think anyone's had time to actively 
work on it though?

-- 
This message brought to you by eevil bananas and the number 3.
www.chani3.com


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: activities support in opendesktop widget

[Sebastian Kügler]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1253/#review1953
---

Ship it!


Code looks clean and correct, nice work :)

I really, really tried to find things wrong with it, but couldn't find anything 
worse than a bit of stray whitespace in the license header. Go for it.


/trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.h


stray whitespace :)


- Sebastian


On 2009-08-10 10:54:19, Marco Martin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1253/
> ---
> 
> (Updated 2009-08-10 10:54:19)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> no, not plasma activities :p
> this adds the friends activity support to the opendesktop plasmoid and 
> selects it by default, so it doesn't look as a static thing anymore but shows 
> some "news"
> 
> still to do:
> configurable refresh time
> avatar pictures to make it more kolorful
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/CMakeLists.txt 1009325 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.h 
> PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.cpp 
> PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.h 1009325 
>   /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.cpp 1009325 
> 
> Diff: http://reviewboard.kde.org/r/1253/diff
> 
> 
> Testing
> ---
> 
> 
> Screenshots
> ---
> 
> 
>   http://reviewboard.kde.org/r/1253/s/170/
> 
> 
> Thanks,
> 
> Marco
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: New systray spec

[Marco Martin]

On Sunday 09 August 2009, Riccardo Iaconelli wrote:
> On Sunday 09 August 2009 16:01:05 Marco Martin wrote:
> > so if someone feels like giving it a read to point out most obviously
> > missing/wrong stuff is welcome :D
>
> just some brain.dump() of stuff that may be missing in the popup text:
>
> - basic alignment, could be useful to show data in a grid-like fashion. may
> be unuseful though, as you can just print "Label: Value" without caring too
> much of spacing and alignment

yeah and amarok too.
probably is possible with a table, since the tooltip subtext is a qlabel, so 
with the usual poor but working support for html (i suppose tooltips of 
ktorrent and amarok already use html?).
what i'm a bit on the fence is to put it in the spec, hmm...

> - does the image always need an alt text? i'm thinking of amarok's album
> art

you can send empty strings yes

> - font colors. ktorrent uses it to discriminate between upload/download.
> but then you could run into problems like black on black or similar issues.
> maybe defining some "color codes" (much like the kde color classes are
> doing) so you say "this text is warning text" and so on, and let the
> implementation deal with it?
or it could just present in different ways the tooltips of icons in different 
states (passive, active, requesting attention)
hardcoded colors can become invisible on the plasma theme
color codes requires preprocessing of the string, but hmm yeah could be an 
idea..
as usual yeah the  tag probably works, but is not really nice to propose 
that tag in a 2009 standard :p

and from that comes the question: do we filter out the tags not covered in the 
standard or try to display everything as is? what are supposed to do the 
Notification protocol? (not NotificationItem, really Notification:p)
>
> nothing else comes to my mind right now... thoughts?
>
> Bye,
> -Riccardo


-- 
Marco Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Adding a runner to KDE

[Jan Gerrit Marker]

Hello,
I committed my source code to playground. Could you have a look at it?
(I added me to Copyright, is it OK?)

Jan
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Artur de Souza (MoRpHeUz)]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1251/#review1950
---



/trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp


maybe you get the error post icon even after a successful post because it 
tries to retrieve more data and then you fall in this place. not sure though 
and right now I don't have the time to test. Check form where postError() is 
being emited on your backend code. (probably from here)



/trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp


Yes, we can remove all this dummy casts.



/trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui


With more plugins being added we need to properly fix this config tab now. 
We postponed this as much as we could but now we should do it properly. I'll 
come with something later (plane to tokamak? hehe).


- Artur


On 2009-08-09 21:08:23, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1251/
> ---
> 
> (Updated 2009-08-09 21:08:23)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This patch adds support for Simplest Image Hosting 
> (http://simplest-image-hosting.net/) to pastebin applet.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/CMakeLists.txt 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/backends.h 1008396 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.h 
> PRE-CREATION 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp
>  PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinConfig.ui 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui 
> 1008396 
> 
> Diff: http://reviewboard.kde.org/r/1251/diff
> 
> 
> Testing
> ---
> 
> Works fine except showing error icon even if upload is successful (there is 
> success notification and link is added to history).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Artur Souza (MoRpHeUz)]

On Monday 10 August 2009, 03:41 Chani wrote:
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.
> cpp 
> 
> 42 + 13?

If I'm not wrong, this was taken from flickr upload plugin on kipi_plugins.

--
Artur Duque de Souza
openBossa Research Labs
INdT - Instituto Nokia de Tecnologia
--
Blog: http://blog.morpheuz.cc
PGP: 0xDBEEAAC3 @ wwwkeys.pgp.net
--


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request: activities support in opendesktop widget

[Marco Martin]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1253/
---

Review request for Plasma.


Summary
---

no, not plasma activities :p
this adds the friends activity support to the opendesktop plasmoid and selects 
it by default, so it doesn't look as a static thing anymore but shows some 
"news"

still to do:
configurable refresh time
avatar pictures to make it more kolorful


Diffs
-

  /trunk/KDE/kdeplasma-addons/applets/opendesktop/CMakeLists.txt 1009325 
  /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.h PRE-CREATION 
  /trunk/KDE/kdeplasma-addons/applets/opendesktop/activitywidget.cpp 
PRE-CREATION 
  /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.h 1009325 
  /trunk/KDE/kdeplasma-addons/applets/opendesktop/opendesktop.cpp 1009325 

Diff: http://reviewboard.kde.org/r/1253/diff


Testing
---


Screenshots
---


  http://reviewboard.kde.org/r/1253/s/170/


Thanks,

Marco

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Patrick Aljord]

This is kind of unrelated but wouldn't it be a good thing to turn the
upload backends into a plasma data engine? I can see other plasmoids
making use of uploading pics and even kde apps like ksnapshot but I
have no idea whether kde apps can make use of plasma engines.

-Pat
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplest Image Hosting support for pastebin applet

[Michal Dutkiewicz]

> On 2009-08-10 06:41:57, Chani wrote:
> > /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp,
> >  line 43
> > 
> >
> > 42 + 13?

This is taken from Imageshack backend, strange notation indeed. ;-)


> On 2009-08-10 06:41:57, Chani wrote:
> > /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp, line 101
> > 
> >
> > why are all these image server classes being cast to themselves?

This is also question to applet developer, I can later create patch that fixes 
thinks like that and also adds possibility to show error message (at least this 
backend can show different error messages, API returns codes).


- Michal


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1251/#review1948
---


On 2009-08-09 21:08:23, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1251/
> ---
> 
> (Updated 2009-08-09 21:08:23)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This patch adds support for Simplest Image Hosting 
> (http://simplest-image-hosting.net/) to pastebin applet.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/CMakeLists.txt 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/backends.h 1008396 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.h 
> PRE-CREATION 
>   
> /trunk/KDE/kdeplasma-addons/applets/pastebin/backends/simplestimagehosting.cpp
>  PRE-CREATION 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinConfig.ui 1008396 
>   /trunk/KDE/kdeplasma-addons/applets/pastebin/pastebinServersConfig.ui 
> 1008396 
> 
> Diff: http://reviewboard.kde.org/r/1251/diff
> 
> 
> Testing
> ---
> 
> Works fine except showing error icon even if upload is successful (there is 
> success notification and link is added to history).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel