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

List:       kde-panel-devel
Subject:    Re: bug #163229: %U in .desktop prevents kickoff search to find
From:       "alessandro diaferia" <alediaferia () gmail ! com>
Date:       2008-06-10 16:39:29
Message-ID: 65627f3a0806100939i35a99edcha53eb8e9db92f045 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


2008/6/10 Aaron J. Seigo <aseigo@kde.org>:

> On Tuesday 10 June 2008, alessandro diaferia wrote:
> > As you can see looking for Okular in kickoff's search line the app is not
> > found.
> > It actually doesn't depend on the %U, however i prepared this patch to
> > solve the problem.
>
> nice =)
>
> > I deeply analyzed the issue so for any question just ask.
>
> i take it this is the important part of the patch:
>
> -                if (execFields.contains(service->exec())) {
> +                if (execFields.contains(service->exec()) && service-
> >noDisplay()) {
>
> so .. a previous item set to noDisplay() was interfering? interesting.
> makes
> sense; please commit =)
>
> --
> 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 Trolltech
>
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel@kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>
>

Hei Aaron.. I'd like to commit but i don't have an account :):)
-- 
Alessandro

[Attachment #5 (text/html)]

<br><br><div class="gmail_quote">2008/6/10 Aaron J. Seigo &lt;<a \
href="mailto:aseigo@kde.org">aseigo@kde.org</a>&gt;:<br><blockquote \
class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt \
0pt 0.8ex; padding-left: 1ex;"> <div class="Ih2E3d">On Tuesday 10 June 2008, \
alessandro diaferia wrote:<br> &gt; As you can see looking for Okular in \
kickoff&#39;s search line the app is not<br> &gt; found.<br>
&gt; It actually doesn&#39;t depend on the %U, however i prepared this patch to<br>
&gt; solve the problem.<br>
<br>
</div>nice =)<br>
<div class="Ih2E3d"><br>
&gt; I deeply analyzed the issue so for any question just ask.<br>
<br>
</div>i take it this is the important part of the patch:<br>
<br>
- &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;if \
(execFields.contains(service-&gt;exec())) {<br> + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \
&nbsp; &nbsp; &nbsp;if (execFields.contains(service-&gt;exec()) &amp;&amp; \
service-<br> &gt;noDisplay()) {<br>
<br>
so .. a previous item set to noDisplay() was interfering? interesting. makes<br>
sense; please commit =)<br>
<font color="#888888"><br>
--<br>
Aaron J. Seigo<br>
humru othro a kohnu se<br>
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA &nbsp;EE75 D6B7 2EB1 A7F1 DB43<br>
<br>
KDE core developer sponsored by Trolltech<br>
<br>
</font><br>_______________________________________________<br>
Panel-devel mailing list<br>
<a href="mailto:Panel-devel@kde.org">Panel-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/panel-devel" \
target="_blank">https://mail.kde.org/mailman/listinfo/panel-devel</a><br> \
<br></blockquote></div><br><br clear="all">Hei Aaron.. I&#39;d like to commit but i \
don&#39;t have an account :):)<br>-- <br>Alessandro



_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel


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