extragear/graphics/digikam/libs/imageproperties

SVN commit 1004617 by aclemens:

This commit fixes the caption issue described in the ML for me.
Do we really need this connection?

CCMAIL:[hidden email]

 M  +2 -2      captionedit.cpp  


--- trunk/extragear/graphics/digikam/libs/imageproperties/captionedit.cpp #1004616:1004617
@@ -74,8 +74,8 @@
     connect(d->altLangStrEdit, SIGNAL(signalModified()),
             this, SIGNAL(signalModified()));
 
-    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const QString&)),
-            this, SLOT(slotSelectionChanged(const QString&)));
+//    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const QString&)),
+//            this, SLOT(slotSelectionChanged(const QString&)));
 
     connect(d->altLangStrEdit, SIGNAL(signalAddValue(const QString&, const QString&)),
             this, SLOT(slotAddValue(const QString&, const QString&)));
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

This is not good, because author name will not be filled in text edit.

I know where is the problem now. it's a signals / slots loop. I will fix it

Gilles

2009/7/30 Andi Clemens <[hidden email]>:

> SVN commit 1004617 by aclemens:
>
> This commit fixes the caption issue described in the ML for me.
> Do we really need this connection?
>
> CCMAIL:[hidden email]
>
>  M  +2 -2      captionedit.cpp
>
>
> --- trunk/extragear/graphics/digikam/libs/imageproperties/captionedit.cpp #1004616:1004617
> @@ -74,8 +74,8 @@
>     connect(d->altLangStrEdit, SIGNAL(signalModified()),
>             this, SIGNAL(signalModified()));
>
> -    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const QString&)),
> -            this, SLOT(slotSelectionChanged(const QString&)));
> +//    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const QString&)),
> +//            this, SLOT(slotSelectionChanged(const QString&)));
>
>     connect(d->altLangStrEdit, SIGNAL(signalAddValue(const QString&, const QString&)),
>             this, SLOT(slotAddValue(const QString&, const QString&)));
> _______________________________________________
> Digikam-devel mailing list
> [hidden email]
> https://mail.kde.org/mailman/listinfo/digikam-devel
>

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Ok :-)

Andi

On Thursday 30 July 2009 19:32:41 Gilles Caulier wrote:

> This is not good, because author name will not be filled in text edit.
>
> I know where is the problem now. it's a signals / slots loop. I will fix it
>
> Gilles
>
> 2009/7/30 Andi Clemens <[hidden email]>:
> > SVN commit 1004617 by aclemens:
> >
> > This commit fixes the caption issue described in the ML for me.
> > Do we really need this connection?
> >
> > CCMAIL:[hidden email]
> >
> >  M  +2 -2      captionedit.cpp
> >
> >
> > --- trunk/extragear/graphics/digikam/libs/imageproperties/captionedit.cpp
> > #1004616:1004617 @@ -74,8 +74,8 @@
> >     connect(d->altLangStrEdit, SIGNAL(signalModified()),
> >             this, SIGNAL(signalModified()));
> >
> > -    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const
> > QString&)), -            this, SLOT(slotSelectionChanged(const
> > QString&)));
> > +//    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const
> > QString&)), +//            this, SLOT(slotSelectionChanged(const
> > QString&)));
> >
> >     connect(d->altLangStrEdit, SIGNAL(signalAddValue(const QString&,
> > const QString&)), this, SLOT(slotAddValue(const QString&, const
> > QString&))); _______________________________________________
> > Digikam-devel mailing list
> > [hidden email]
> > https://mail.kde.org/mailman/listinfo/digikam-devel
>
> _______________________________________________
> Digikam-devel mailing list
> [hidden email]
> https://mail.kde.org/mailman/listinfo/digikam-devel

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

Bad news. the bug is back when i rool combo box. SDound like th
eproblem is outside. i'm lost

Gilles

2009/7/30 Andi Clemens <[hidden email]>:

> Ok :-)
>
> Andi
>
> On Thursday 30 July 2009 19:32:41 Gilles Caulier wrote:
>> This is not good, because author name will not be filled in text edit.
>>
>> I know where is the problem now. it's a signals / slots loop. I will fix it
>>
>> Gilles
>>
>> 2009/7/30 Andi Clemens <[hidden email]>:
>> > SVN commit 1004617 by aclemens:
>> >
>> > This commit fixes the caption issue described in the ML for me.
>> > Do we really need this connection?
>> >
>> > CCMAIL:[hidden email]
>> >
>> >  M  +2 -2      captionedit.cpp
>> >
>> >
>> > --- trunk/extragear/graphics/digikam/libs/imageproperties/captionedit.cpp
>> > #1004616:1004617 @@ -74,8 +74,8 @@
>> >     connect(d->altLangStrEdit, SIGNAL(signalModified()),
>> >             this, SIGNAL(signalModified()));
>> >
>> > -    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const
>> > QString&)), -            this, SLOT(slotSelectionChanged(const
>> > QString&)));
>> > +//    connect(d->altLangStrEdit, SIGNAL(signalSelectionChanged(const
>> > QString&)), +//            this, SLOT(slotSelectionChanged(const
>> > QString&)));
>> >
>> >     connect(d->altLangStrEdit, SIGNAL(signalAddValue(const QString&,
>> > const QString&)), this, SLOT(slotAddValue(const QString&, const
>> > QString&))); _______________________________________________
>> > Digikam-devel mailing list
>> > [hidden email]
>> > https://mail.kde.org/mailman/listinfo/digikam-devel
>>
>> _______________________________________________
>> Digikam-devel mailing list
>> [hidden email]
>> https://mail.kde.org/mailman/listinfo/digikam-devel
>
> _______________________________________________
> Digikam-devel mailing list
> [hidden email]
> https://mail.kde.org/mailman/listinfo/digikam-devel
>

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel