Review Request 113605: Fix a a signal/slot signature mistmatch Tp::ContactPtr to KTp::ContactPtr

Diane Trout diane at ghic.org
Mon Nov 4 16:29:49 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113605/
-----------------------------------------------------------

(Updated Nov. 4, 2013, 4:29 p.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy.


Repository: ktp-send-file


Description
-------

Signature updates for change from Tp::ContactPtr to KTp::ContactPtr
    
The signal signature for ContactGridWidget::selectionChanged changed in ktp-common-internals and so the connection to the OK button in the UI never got the message to enable itself.


Diffs
-----

  mainwindow.h 397f6b9 
  mainwindow.cpp 87fb0f7 

Diff: http://git.reviewboard.kde.org/r/113605/diff/


Testing
-------

1) Launch dolphin 
2) right click on a file
3) Actions->Send file to IM Contact.
4) Select IM contact.

Without patch, the OK button never lights up, with patch, OK button enables and I can send a file.

Unfortunately ktp contactlist still seems to have the send file UI element grayed out.


Thanks,

Diane Trout

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20131104/7d916d9a/attachment.html>


More information about the KDE-Telepathy mailing list