[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: dropped URL handling in containment,
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2009-10-28 15:39:41
Message-ID: 200910280939.46296.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On October 27, 2009, Sebastian Kügler wrote:
> On Tuesday 27 October 2009 17:19:24 Aaron J. Seigo wrote:
> >   On October 27, 2009, Sebastian Kügler wrote:
> > > Is this something I should pursue? It sounds simple enough ...
> >
> > sure; i don't see any downside to it and the results could be
> > interesting; worst case is we'd have a failed experimnt on our hands and
> > one more way not  to do it (in the words of Edison ;) but i think this
> > could work out very well..
> 
> The attached patch seems to work. I've not really tested it, just confirmed
>  that it works at least in one case (emailmessage, added
>  X-Plasma-DropUrlPatterns=akonadi:* to its desktop file) -- it's late and I
>  should sleep. If it's conceptually OK, I'll clean it up and submit it to
>  review board.

doing: 

KPluginInfo::List allApplets = KPluginInfo::fromServices(offers, "exist [X-
Plasma-DropUrlPatterns]")

should eliminate all the entries that don't have any DropUrlPatterns without 
having to create and then cycle through *all* the plasma applets installed.

this shouldn't be needed:

    QHash<KJob*, KPluginInfo::List> dropPlugins;

the URL can be retrieved in mimeTypeRetrieved with job->url()

but in general it could be an interesting addition... 

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks

["signature.asc" (application/pgp-signature)]

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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic