As from what happens with the current approach the view that lists available wallpapers is connected to the modified() SLOT through the activated() SIGNAL. But when the global preferences are set to use Double Click you have to double click each item in the view in order to see the preview before applying the wallpaper. This is a little annoying since there's apparently no need for the view to "wait" for a double click before previewing the wallpaper. In addition to this the behavior is not so intuitive.<br>
<br>This patch comes then:<br><br><br>Index: image/image.cpp<br>===================================================================<br>--- image/image.cpp (revisione 1022077)<br>+++ image/image.cpp (copia locale)<br>
@@ -127,7 +127,7 @@ QWidget* Image::createConfigurationInter<br> fillMetaInfo(b);<br> }<br> }<br>- connect(m_uiImage.m_view, SIGNAL(activated(const QModelIndex &)), this, SLOT(pictureChanged(const QModelIndex &)));<br>
+ connect(m_uiImage.m_view, SIGNAL(clicked(const QModelIndex &)), this, SLOT(pictureChanged(const QModelIndex &)));<br><br> m_uiImage.m_pictureUrlButton->setIcon(KIcon("document-open"));<br>
connect(m_uiImage.m_pictureUrlButton, SIGNAL(clicked()), this, SLOT(showFileDialog()));<br><br>Is it ok for you to commit it?<br clear="all"><br>-- <br>Alessandro Diaferia<br>KDE Developer<br>