Bug 190786

Summary: Buttons on notifications are misplaced to the right (Qt4.5.1)
Product: [Plasma] plasma4 Reporter: Lisandro Damián Nicanor Pérez Meyer <perezmeyer>
Component: notificationsAssignee: Rob Scheepmaker <rob>
Status: RESOLVED FIXED    
Severity: normal CC: andresbajotierra, budinero, grosser.meister.morti, kde-bugs, mail, pali.rohar
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: Version Fixed In:

Description Lisandro Damián Nicanor Pérez Meyer 2009-04-27 02:47:22 UTC 
Version:            (using KDE 4.2.2)
OS:                Linux
Installed from:    Debian testing/unstable Packages

From: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=525756
Original reporter: Facundo Aguilera <budinero@gmail.com>

Kopete notifications has no the "Ignore" and "Show" buttons when messages arrived.
Comment 1 Lisandro Damián Nicanor Pérez Meyer 2009-04-27 02:48:18 UTC 
It is possible that this happens with Qt 4.5.0, but I'm not sure.
Comment 2 Lisandro Damián Nicanor Pérez Meyer 2009-04-27 03:20:06 UTC 
My mistake: Qt 4.5.1
Comment 3 Facundo Aguilera 2009-04-27 03:34:19 UTC 
Yes, I think the same
Comment 4 Pali Rohár 2009-04-27 08:07:40 UTC 
Yes, I know. It show all messages (incomming and outcomming) immediately.
It is bug in skype protocol. It is on TODO list.
Comment 5 George Kiagiadakis 2009-04-27 20:31:01 UTC 
This seems like a plasma/Qt related bug. It happens with all notifications that have actions and it seems to be triggered by Qt 4.5.1, as it used to work fine with Qt 4.5.0.

Here is a testcase to show the problem:
Assuming that you have Qt 4.5.1 installed, the two actions (buttons) "foo" and "bar" should not appear on the popup. However, you can show them if you click twice on the application icon that appears on the top left corner of the popup.
--------------
#include <QtCore/QCoreApplication>
#include <QtCore/QStringList>
#include <QtCore/QTimer>
#include <knotification.h>
#include <kcomponentdata.h>
#include <klocale.h>
#include <kaboutdata.h>

int main(int argc, char **argv)
{
        QCoreApplication a(argc, argv);
        KComponentData d(KAboutData("kwrited", 0, ki18n("kwrited-test"), "1.0"));
        KNotification *notification = new KNotification("NewMessage", 0, KNotification::Persistent);
        notification->setComponentData(d);
        notification->setText( "hello world" );
        notification->setActions(QStringList() << "foo" << "bar");
        notification->sendEvent();
        QTimer::singleShot(5000, &a, SLOT(quit()));
        return a.exec();
}
-----------

I compiled that with "g++ test.cpp -lkdeui -I/usr/include/qt4" on a debian system. I have put the app name to be "kwrited", so that it uses the kwrited.notifyrc file that is installed with kdebase/workspace/kwrited, as it was easier for me.
Comment 6 Facundo Aguilera 2009-04-27 21:56:12 UTC 
The buttons are in the popup, but you can not see them.
I can see a portion of one button at the bottom right of notification. 

Please, report a bug for plasma notifications and extend it.
Comment 7 Dario Andres 2009-04-28 01:11:13 UTC 
This seems to be bug 190734. Confirm it and I will mark it as duplicate.
Thanks
Comment 8 Facundo Aguilera 2009-04-28 01:17:31 UTC 
Yes, it is.
Comment 9 Aaron J. Seigo 2009-04-28 01:23:05 UTC 
*** Bug 190734 has been marked as a duplicate of this bug. ***
Comment 10 Aaron J. Seigo 2009-04-28 01:55:07 UTC 
SVN commit 960131 by aseigo:

simple enough fix; thanks to George Kiagiadakis for the nice testcase
BUG:190786


 M  +1 -1      notificationwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=960131
Comment 11 Aaron J. Seigo 2009-04-28 01:56:07 UTC 
SVN commit 960132 by aseigo:

backport fix for bug #190786
CCBUG:190786


 M  +1 -1      notificationwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=960132
Comment 12 Dario Andres 2009-04-29 17:33:21 UTC 
*** Bug 191012 has been marked as a duplicate of this bug. ***