[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 <<a \
href="mailto:aseigo@kde.org">aseigo@kde.org</a>>:</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>> Now looks like He is missing their properties, I can only \
see the<br>> properties to:<br>> QObject, QWidget and QFrame but nothing about \
KColorPatch. <br><br>that'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've not made extensive use of qt'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 EE75 D6B7 2EB1 A7F1 DB43<br><br>KDE core developer sponsored by \
Trolltech<br><br><br>>> Visit <a \
href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub"> \
http://mail.kde.org/mailman/listinfo/kde-devel#unsub</a> to unsubscribe \
<<<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