Hello:
<br>
<br>You are correct, the present code (code in subversion) doesn't handle the case for KDE < 3.3.  One way to correct the problem is to add the diff code marked by  #else that I sent. 
<br>
<br>regards
<br>
<br>Mike
<br>
<br><blockquote style="border-left: 2px solid #666666; border-top: 1px solid #eeeeee; margin: 15px 20px;">
<div style="color: #333333; font: bold 11px tahoma,geneva,helvetica,arial,sans-serif; padding: 5px 10px;">Tom Albers wrote:</div>
<div style="color: #333333; font: 11px tahoma,geneva,helvetica,arial,sans-serif; padding: 5px 10px;">Mike Seymour (sent by Nabble.com) wrote:
<br>> Hello:
<br>> 
<br>> Actually, the change in the SVN souce will leave the forward and back
<br>> buttons blank in the image properties dialog.  I added the code below
<br>> which seemed to give the buttons for those of us stuck with KDE < 3.3
<br>> cordialement
<br>
<br>Hi,
<br>
<br>I dont undestand this patch, it seems to remove the special handling for
<br>KDE<3.3. Maybe the patch is the wrong way around? Anyway the current
<br>code looks like:
<br>
<br>    if (m_mode == MULTI)
<br>    {
<br>#if KDE_IS_VERSION(3,3,0)
<br>        setButtonGuiItem(User1, KStdGuiItem::guiItem(KStdGuiItem::Forward));
<br>        setButtonGuiItem(User2, KStdGuiItem::guiItem(KStdGuiItem::Back));
<br>#endif
<br>        enableButton(User1, m_currItem->nextItem() != 0);
<br>        enableButton(User2, m_currItem->prevItem() != 0);
<br>    }
<br>
<br>Let me know what to change..
<br>
<br>Toma
<br>
<br>
<br>> Mike
<br>> 
<br>> *** imageproperties.cpp 2005-11-22 22:40:41.023253880 +0100
<br>> --- imageproperties.cpp.bak     2005-11-16 15:18:32.000000000 +0100
<br>> ***************
<br>> *** 59,74 ****
<br>>   {
<br>>       if (m_mode == MULTI)
<br>>       {
<br>> -     #if KDE_IS_VERSION(3,3,0)
<br>>           setButtonGuiItem(User1,
<br>> KStdGuiItem::guiItem(KStdGuiItem::Forward));
<br>>           setButtonGuiItem(User2, KStdGuiItem::guiItem(KStdGuiItem::Back));
<br>> -     #else
<br>> -       setButtonText(User1,
<br>>  KStdGuiItem::guiItem(KStdGuiItem::Forward).plainTe
<br>> xt() );
<br>> -       setButtonTip( User1,
<br>> KStdGuiItem::guiItem(KStdGuiItem::Forward).toolTip(
<br>> ) );
<br>> -       setButtonText(User2,
<br>>  KStdGuiItem::guiItem(KStdGuiItem::Back).plainText(
<br>> ) );
<br>> -       setButtonTip( User2,
<br>> KStdGuiItem::guiItem(KStdGuiItem::Back).toolTip() )
<br>> ;
<br>> 
<br>> -     #endif
<br>>           enableButton(User1, m_currItem->nextItem() != 0);
<br>>           enableButton(User2, m_currItem->prevItem() != 0);
<br>>       }
<br>> --- 59,67 ----
<br>> 
<br>> 
<br>> 
<br>>     Gilles Caulier wrote:
<br>>     Le Mardi 15 Novembre 2005 00:14, Uwe Reder a ?crit :
<br>>     > Hi!
<br>>     >
<br>>     > I run KDE 3.2.3. Compilation breaks in
<br>>     > digikam-0.8.0-rc/digikam/digikam/imageproperties.cpp with
<br>>     > setButtonGuiItem() undefined. It seams setButtonGuiItem() appeared
<br>>     with
<br>>     > KDE 3.3. There must be a way to work around setButtonGuiItem()
<br>>     since it is
<br>>     > the only method (as far as I can see) that is used from 3.3.
<br>>     >
<br>>     > I took a rather simple approach to solve the problem: I backported
<br>>     > setButtonGuiItem() to 3.2.3's libkdeui.so. Works perfectly for me :-)
<br>>     >
<br>>     > Uwe.
<br>> 
<br>>     Fixed on svn to ignore this method if compiling under kde 3.2.x like
<br>>     this :
<br>> 
<br>>     ImageProperties::ImageProperties(enum Mode mode, QWidget* parent,
<br>>                                      AlbumIconView* view, AlbumIconItem*
<br>>     currItem,
<br>>                                      QRect* selectionArea, uint* imageData,
<br>>                                      int imageWidth, int imageHeight)
<br>>         : KDialogBase(Tabbed, QString::null,
<br>>                       (mode == MULTI) ? Help|User1|User2|Stretch|Close :
<br>>     Help|
<br>>     Stretch|Close,
<br>>                       Close, parent, 0, true, true),
<br>>           m_view(view), m_currItem(currItem), m_mode(mode)
<br>>     {
<br>>         if (m_mode == MULTI)
<br>>         {
<br>>     #if KDE_IS_VERSION(3,3,0)
<br>>             setButtonGuiItem(User1,
<br>>     KStdGuiItem::guiItem(KStdGuiItem::Forward));
<br>>             setButtonGuiItem(User2,
<br>>     KStdGuiItem::guiItem(KStdGuiItem::Back));
<br>>     #endif
<br>>             enableButton(User1, m_currItem->nextItem() != 0);
<br>>             enableButton(User2, m_currItem->prevItem() != 0);
<br>>         }
<br>>     ...
<br>> 
<br>>     Regards
<br>> 
<br>>     -- 
<br>>     Gilles Caulier
<br>> 
<br>>     _______________________________________________
<br>>     Digikam-devel mailing list
<br>>     Digikam-devel@kde.org
<br>>     <a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_top" rel="nofollow">https://mail.kde.org/mailman/listinfo/digikam-devel</a>
<br>> 
<br>> 
<br>> ------------------------------------------------------------------------
<br>> Sent from the digikam-devel
<br>> <<a href="http://www.nabble.com/setButtonGuiItem%28%29-from-KDE-3.3-t552361.html#a1607996>" target="_top" rel="nofollow">http://www.nabble.com/setButtonGuiItem%28%29-from-KDE-3.3-t552361.html#a1607996></a>
<br>> forum at Nabble.com.
<br>> 
<br>> 
<br>> ------------------------------------------------------------------------
<br>> 
<br>> _______________________________________________
<br>> Digikam-devel mailing list
<br>> Digikam-devel@kde.org
<br>> <a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_top" rel="nofollow">https://mail.kde.org/mailman/listinfo/digikam-devel</a>
<br>
<br>_______________________________________________
<br>Digikam-devel mailing list
<br>Digikam-devel@kde.org
<br><a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_top" rel="nofollow">https://mail.kde.org/mailman/listinfo/digikam-devel</a>
</div>
</blockquote>

<br><hr align="left" width="30%">
Sent from the <a href="http://www.nabble.com/setButtonGuiItem%28%29-from-KDE-3.3-t552361.html#a1667460">digikam-devel</a> forum at Nabble.com.