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

List:       kde-devel
Subject:    Re: Qt Designer crashed with some kde4 widgets
From:       "=?ISO-8859-1?Q?Percy_Camilo_Trive=F1o_Aucahuasi?=" <orgyforever () gmail ! com>
Date:       2008-01-10 5:36:07
Message-ID: 579229230801092136x5f088ea9hb8dd8a0d85ea3c9e () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hi Aaron, I did two patch, one for kcolordialog.cpp [kcolorpatchcpp.dif] and
other for kcolordialog.h [kcolorpatchh.dif], now KColorPatch have a property
called color and works fine in Qt Designer.

Greetings

pd: btw this is my first patch :-).



2008/1/9, Aaron J. Seigo <aseigo@kde.org>:
>
> On Wednesday 09 January 2008, Percy Camilo Triveņo Aucahuasi wrote:
> > Now looks like He is missing their properties, I can only see the
> > properties to:
> > QObject, QWidget and QFrame but nothing about KColorPatch.
>
> that's because none are defined. (take a look in kcolordialog.h where this
> class is defined; man, i dislike multiple classes in the same file =)
>
> so ... adding an appropriate Q_PROPERTY macro to the header would do it.
> since
> we've not made extensive use of qt's property system until now and some of
> these classes are quite old, not surprising.
>
> doing up patches for the various classes in kdeui would be very cool =)
>
> --
> 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
>
>
> >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to
> unsubscribe <<
>
>
>

[Attachment #5 (text/html)]

Hi Aaron, I did two patch, one for kcolordialog.cpp [kcolorpatchcpp.dif] and other \
for kcolordialog.h [kcolorpatchh.dif], now KColorPatch have a property called color \
                and works fine in Qt Designer.<br><br>Greetings<br><br>
pd: btw this is my first patch :-).<br><br><br><br><div><span \
class="gmail_quote">2008/1/9, Aaron J. Seigo &lt;<a \
href="mailto:aseigo@kde.org">aseigo@kde.org</a>&gt;:</span><blockquote \
class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt \
0pt 0.8ex; padding-left: 1ex;"> On Wednesday 09 January 2008, Percy Camilo Triveņo \
Aucahuasi wrote:<br>&gt; Now looks like He is missing their properties, I can only \
see the<br>&gt; properties to:<br>&gt; QObject, QWidget and QFrame but nothing about \
KColorPatch. <br><br>that&#39;s because none are defined. (take a look in \
kcolordialog.h where this<br>class is defined; man, i dislike multiple classes in the \
same file =)<br><br>so ... adding an appropriate Q_PROPERTY macro to the header would \
do it. since <br>we&#39;ve not made extensive use of qt&#39;s property system until \
now and some of<br>these classes are quite old, not surprising.<br><br>doing up \
                patches for the various classes in kdeui would be very cool \
                =)<br><br>
--<br>Aaron J. Seigo<br>humru othro a kohnu se<br>GPG Fingerprint: 8B8B 2209 0C6F \
7C47 B1EA&nbsp;&nbsp;EE75 D6B7 2EB1 A7F1 DB43<br><br>KDE core developer sponsored by \
Trolltech<br><br><br>&gt;&gt; Visit <a \
href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub"> \
http://mail.kde.org/mailman/listinfo/kde-devel#unsub</a> to unsubscribe \
&lt;&lt;<br><br><br></blockquote></div><br>


["kcolorpatchcpp.dif" (video/dv)]
["kcolorpatchh.dif" (video/dv)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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

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