Bug 190786 - Buttons on notifications are misplaced to the right (Qt4.5.1)
Summary: Buttons on notifications are misplaced to the right (Qt4.5.1)
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: notifications (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: Rob Scheepmaker
URL:
Keywords:
: 190734 191012 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-04-27 02:47 UTC by Lisandro Damián Nicanor Pérez Meyer
Modified: 2009-04-29 17:33 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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. ***