Res: KRunner kills keyboard?

2008-09-01 Thread dantti85-dev
something is killing kded4,
you can avoid logout and login by just
starting kded4 again.
you can try to start kded4 on a shell
and open KRunner to see if you find something.

Daniel.


- Mensagem original 
De: Guillaume Pothier <[EMAIL PROTECTED]>
Para: plasma-devel@kde.org
Enviadas: Segunda-feira, 1 de Setembro de 2008 14:26:19
Assunto: KRunner kills keyboard?

Hi, since today or yesterday, whenever I open KRunner I loose my
keyboard... I cannot type anything in any application, the only
solution is to log out and back in. Anybody else having the issue?
Cheers,
g
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel



  Novos endereços, o Yahoo! que você conhece. Crie um email novo com a sua 
cara @ymail.com ou @rocketmail.com.
http://br.new.mail.yahoo.com/addresses___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Res: Application Launcher and the ALT+F1 shortcut

2008-09-17 Thread dantti85-dev
Yep, 
i got this problem too,
i think some app is stealing this signal.
i solved this rmoving my .kde4 :P
But i definitely don't promote this kind of
"solving problems" :)

Daniel.



- Mensagem original 
De: Michael Jansen <[EMAIL PROTECTED]>
Para: plasma-devel@kde.org
Enviadas: Quarta-feira, 17 de Setembro de 2008 7:57:35
Assunto: Application Launcher and the ALT+F1 shortcut


Hi

i tried to find out why the ALT+F1 shortcut isn't working for me. It is 
catched and send to plasma but seems to get lost somewhere. To find out where 
i applied the following patch:

diff --git a/workspace/libs/plasma/applet.cpp 
b/workspace/libs/plasma/applet.cpp
index 795f81a..e47999f 100644
--- a/workspace/libs/plasma/applet.cpp
+++ b/workspace/libs/plasma/applet.cpp
@@ -252,7 +252,7 @@ void Applet::restore(KConfigGroup &group)

void AppletPrivate::setFocus()
{
-kDebug() << "setting focus";
+kDebug() << "setting focus to " << q->name();
 q->setFocus(Qt::ShortcutFocusReason);
}

Now i can see 

plasma(4030)/libplasma Plasma::AppletPrivate::setFocus: setting focus to  
"Application Launcher"

when pressing ALT+F1. But still no open menu.

Mike

-- 
Michael Jansen

http://www.michael-jansen.biz
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel



  Novos endereços, o Yahoo! que você conhece. Crie um email novo com a sua 
cara @ymail.com ou @rocketmail.com.
http://br.new.mail.yahoo.com/addresses___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Res: Review Request: This patch add an easy way to up/down brightness only scrolling on battery plasmoid top.

2008-12-01 Thread dantti85-dev
>sorry, what does moving the mouse wheel over the battery icon have to do >with 
>the brightness of my screen?
>
>at the very least it requires onscreen notification, but it really seems >like 
>an odd feature.

I do agree, and imho it could be changed to do another thing, like changing 
power profiles WITH a on screen notification. this makes much more sense to me, 
what about that?

Daniel.





De: Aaron J. Seigo <[EMAIL PROTECTED]>
Para: plasma-devel@kde.org
Enviadas: Segunda-feira, 1 de Dezembro de 2008 15:05:21
Assunto: Re: Review Request: This patch add an easy way to up/down brightness 
only scrolling on battery plasmoid top.

On Monday 01 December 2008, Sebastian Kügler wrote:
> On Monday 01 December 2008 14:32:39 Gökmen GÖKSEL wrote:
> > On Monday 01 December 2008 03:14:17 Aaron Seigo wrote:
> > > hm. i actually take that "Ship it!" back .. i don't think it makes any
> > > sense to change the screen brightness when scrolling over the battery
> > > icon. coupled with the fact that there is no user feedback when this
> > > happens (other than the screen getting dimmer or brighter ;) this is
> > > just asking to annoy people.
> > >
> > > click on on the battery to get the controls; how hard is that? =)
> >
> > KPowerSave was provide that and it was a very useful shortcut I guess :)
>
> I'd be OK with it as a config option, off by default (and no UI before 4.2
> because of string freeze).

sorry, what does moving the mouse wheel over the battery icon have to do with 
the brightness of my screen?

at the very least it requires onscreen notification, but it really seems like 
an odd feature.

and the problem with features is that you can't take them back once offered.

-- 
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 Software


  Veja quais são os assuntos do momento no Yahoo! +Buscados
http://br.maisbuscados.yahoo.com___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Configuring time and date from context menu of digital-clock applet

2009-05-28 Thread dantti85-dev

>> PS: By the way, there is a bug about time configuration. By default, update
>> interval of digital-clock plasmoid is 60 seconds when showSeconds is false,
>> and 1 second if it's true. If you change time from clock kcm and
>> showSeconds is false, digital-clock applet refreshes itself 1 minute later.
>> Maybe updateInterval should be hardcoded as 1 second regardless of
>> showSeconds variable.
>
>hmm, not sure, would be  making it to do way more work just for the occasion 
>of something that happens very rarely

I'd suggest having a DBus method for that, whenever the clock needs to update
we call org.kde.system-clock-refresh and attached applications like plasma 
clocks
would get it right.

Also for kde 4.4 i'd like to patch kcm to use policy-kit, and do the auth 
before the user
changes the time, this way the apply button
would not waste the enter password time (ie the user set the seconds manually,
and while typing the password the seconds aren't right anymore).

Cheers,
Daniel.



- Mensagem original 
De: Marco Martin 
Para: plasma-devel@kde.org
Enviadas: Quinta-feira, 28 de Maio de 2009 8:39:15
Assunto: Re: Configuring time and date from context menu of digital-clock applet

On Thursday 28 May 2009, Gökçen Eraslan wrote:
> Hi,
>
> A patch for $subject is attached. This was the default behaviour in KDE3,
> now users have to open systemsettings to configure date/time.
>
> Any reviews, comments?

could make sense i miss that feature too...
however it will have to wait since we're now in string freeze, b

> PS: By the way, there is a bug about time configuration. By default, update
> interval of digital-clock plasmoid is 60 seconds when showSeconds is false,
> and 1 second if it's true. If you change time from clock kcm and
> showSeconds is false, digital-clock applet refreshes itself 1 minute later.
> Maybe updateInterval should be hardcoded as 1 second regardless of
> showSeconds variable.

hmm, not sure, would be  making it to do way more work just for the occasion 
of something that happens very rarely

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



  Veja quais são os assuntos do momento no Yahoo! +Buscados
http://br.maisbuscados.yahoo.com
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Kickoff fix

2009-05-29 Thread dantti85-dev
Hi,
there is a bug in kickoff when dealing with old applications
that install .desktop at applnk. KSycoca returns a relative
path and as David Faure said it should be handled in kickoff,
in the case what happens is that kvirc from kde3 when added
to favorites looses it's icon, and when added to the desktop
does not work at all.

I've made a patch so please take a look, if it's ok, i'll commit.

Thanks,
Daniel.



  Veja quais são os assuntos do momento no Yahoo! +Buscados
http://br.maisbuscados.yahoo.comIndex: plasma/applets/kickoff/core/models.cpp
===
--- plasma/applets/kickoff/core/models.cpp	(revisão 975387)
+++ plasma/applets/kickoff/core/models.cpp	(cópia de trabalho)
@@ -59,7 +59,9 @@
 
 QStandardItem *item = 0;
 
-if (url.isLocalFile() && urlString.endsWith(".desktop")) {
+// We can't check if it's a local file since legacy .desktop at applnk
+// receive relative paths
+if (urlString.endsWith(".desktop")) {
 // .desktop files may be services (type field == 'Application' or 'Service')
 // or they may be other types such as links.
 //
Index: plasma/applets/kickoff/ui/contextmenufactory.cpp
===
--- plasma/applets/kickoff/ui/contextmenufactory.cpp	(revisão 975387)
+++ plasma/applets/kickoff/ui/contextmenufactory.cpp	(cópia de trabalho)
@@ -38,6 +38,7 @@
 #include 
 #include 
 #include 
+#include 
 
 // Plasma
 #include 
@@ -154,7 +155,15 @@
 
 if (desktop && desktop->immutability() == Plasma::Mutable) {
 addToDesktopAction->setText(i18n("Add to Desktop"));
-actions << addToDesktopAction;
+// There might be relative paths, we need to locate them
+if (kurl.isRelative()) {
+kurl = KStandardDirs::locate("apps", url);
+if (!kurl.isEmpty()) {
+actions << addToDesktopAction;
+}
+} else {
+actions << addToDesktopAction;
+}
 }
 }
 
@@ -234,7 +243,7 @@
 Plasma::Containment *desktop = corona->containmentForScreen(containment->screen());
 if (desktop) {
 QVariantList args;
-args << url;
+args << kurl.url();
 desktop->addApplet("icon", args);
 }
 }
@@ -246,7 +255,7 @@
 Plasma::Containment *panel = d->applet->containment();
 if (panel) {
 QVariantList args;
-args << url;
+args << kurl.url();
 
 // move it to the middle of the panel
 QRectF rect(panel->geometry().width() / 2, 0, 150, panel->boundingRect().height());
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Kickoff fix

2009-05-30 Thread dantti85-dev

The url for these relative paths are like
"Internet/kvirc.desktop", and how does
that http://foo/.../thing.desktop works? I mean
how can one input those?
We can test the url protocol but it actually depends more
on it being passed, like these relatives don't pass.
This patch is not completely actually, after i sent
the email, i remembered that i forgot to put code to
add it to panel, (but it's quite easy to fix..), I'll try to finish
tomorrow.

Daniel.



- Mensagem original 
De: Aaron J. Seigo 
Para: plasma-devel@kde.org
Enviadas: Sábado, 30 de Maio de 2009 13:03:14
Assunto: Re: Kickoff fix

On Friday 29 May 2009, dantti85-...@yahoo.com.br wrote:
> I've made a patch so please take a look, if it's ok, i'll commit.

hm.. actually ... this will now break if someone puts 
http://foo.com/somewhere/thing.desktop won't it?

i wonder if we can test the url's protocol, rather than test it for being 
local. what's the full url for these relative-path-.desktop files?

-- 
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 Software


  Veja quais são os assuntos do momento no Yahoo! +Buscados
http://br.maisbuscados.yahoo.com
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Kickoff fix

2009-05-31 Thread dantti85-dev

What about doing the old test (isLocalFile || isRelativePath()), this would
keep the old behavior and add support for relative paths, i believe
something like http://path_to/thing.desktop will never be relative?

Daniel.



- Mensagem original 
De: Aaron J. Seigo 
Para: plasma-devel@kde.org
Enviadas: Domingo, 31 de Maio de 2009 2:50:25
Assunto: Re: Kickoff fix

On Saturday 30 May 2009, dantti85-...@yahoo.com.br wrote:
> The url for these relative paths are like
> "Internet/kvirc.desktop", and how does
> that http://foo/.../thing.desktop works? I mean
> how can one input those?

that method is used for the creation of items for most of the views, and it's 
not hard to get such urls in, for instance, recently used documents.

> We can test the url protocol but it actually depends more
> on it being passed, like these relatives don't pass.

sure; but in the case of the relative path, what does protocol() return? i 
imagine an empty string? probably checking for either file or QString() would 
work?

> This patch is not completely actually, after i sent
> the email, i remembered that i forgot to put code to
> add it to panel, (but it's quite easy to fix..), I'll try to finish
> tomorrow.

"add to panel" and "add to desktop" is such a horrible hack it's not even 
funny. i really wish, in retrospect, that i'd been a bit stricter and not let 
those feature/bugs in. i was catching enough flak from the people who were 
contributing such patches at the time that i eventually just gave in. *sigh*

-- 
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 Software


  Veja quais são os assuntos do momento no Yahoo! +Buscados
http://br.maisbuscados.yahoo.com
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Kickoff fix

2009-05-31 Thread dantti85-dev
>On Sunday 31 May 2009, dantti85-...@yahoo.com.br wrote:
>> What about doing the old test (isLocalFile || isRelativePath()), this would
>> keep the old behavior and add support for relative paths, i believe
>> something like http://path_to/thing.desktop will never be relative?
>
>hm.. probably not. 
>
>so, let's try isRelativePath and see if that brings any new edge cases with 
>it. hopefully not :) add a comment in there about all this and we should be 
>golden.
Comments added :D

>thanks for fixing this ...
No problem :D, this seems a bit rare case but i got upset with that gear :P

Now the attached diff seems to fix better the problem, if
you don't have anything against it I'll commit.

Cheers,
Daniel.



  Veja quais são os assuntos do momento no Yahoo! +Buscados
http://br.maisbuscados.yahoo.comIndex: applets/kickoff/core/models.cpp
===
--- applets/kickoff/core/models.cpp	(revisão 975387)
+++ applets/kickoff/core/models.cpp	(cópia de trabalho)
@@ -59,7 +59,10 @@
 
 QStandardItem *item = 0;
 
-if (url.isLocalFile() && urlString.endsWith(".desktop")) {
+// Match files ending with ".desktop" and being local or having a relative
+// path. For instance applications that still installs .desktop files at
+// /usr/share/applnk, like KVirc 3
+if (urlString.endsWith(".desktop") && (url.isLocalFile() || url.isRelative())) {
 // .desktop files may be services (type field == 'Application' or 'Service')
 // or they may be other types such as links.
 //
Index: applets/kickoff/ui/contextmenufactory.cpp
===
--- applets/kickoff/ui/contextmenufactory.cpp	(revisão 975387)
+++ applets/kickoff/ui/contextmenufactory.cpp	(cópia de trabalho)
@@ -38,6 +38,7 @@
 #include 
 #include 
 #include 
+#include 
 
 // Plasma
 #include 
@@ -149,7 +150,15 @@
 if ((d->applet) && (kurl.scheme() != "leave")) {
 Plasma::Containment *containment = d->applet->containment();
 
-if (containment && containment->corona()) {
+// There might be relative paths for .desktop installed in
+// /usr/shar/applnk, we need to locate them
+bool urlFound = true;
+if (kurl.isRelative() && kurl.url().endsWith(".desktop")) {
+kurl = KStandardDirs::locate("apps", url);
+urlFound = !kurl.isEmpty();
+}
+
+if (urlFound && containment && containment->corona()) {
 Plasma::Containment *desktop = containment->corona()->containmentForScreen(containment->screen());
 
 if (desktop && desktop->immutability() == Plasma::Mutable) {
@@ -158,7 +167,7 @@
 }
 }
 
-if (containment && containment->immutability() == Plasma::Mutable &&
+if (urlFound && containment && containment->immutability() == Plasma::Mutable &&
 containment->containmentType() == Plasma::Containment::PanelContainment) {
 addToPanelAction->setText(i18n("Add to Panel"));
 actions << addToPanelAction;
@@ -234,7 +243,7 @@
 Plasma::Containment *desktop = corona->containmentForScreen(containment->screen());
 if (desktop) {
 QVariantList args;
-args << url;
+args << kurl.url();
 desktop->addApplet("icon", args);
 }
 }
@@ -246,7 +255,7 @@
 Plasma::Containment *panel = d->applet->containment();
 if (panel) {
 QVariantList args;
-args << url;
+args << kurl.url();
 
 // move it to the middle of the panel
 QRectF rect(panel->geometry().width() / 2, 0, 150, panel->boundingRect().height());
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel