Review Request 117543: Don't leave group chat when window is closed

Dan Vrátil dvratil at redhat.com
Thu Apr 17 13:07:35 UTC 2014


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

(Updated April 17, 2014, 3:07 p.m.)


Review request for Telepathy.


Changes
-------

As David suggested, we now only keep the channels in TelepathyChatUi when they don't have any tab assigned. This also fixes duplicated notifications, reconnection and makes the code cleaner.


Repository: ktp-text-ui


Description
-------

When "Don't leave chat room when window is closed" settings is enabled, the channel is not closed when user closes the tab or window, but is maintained by TelepathyChatUi. The channel can be left via Conversation -> Leave room action.


Diffs (updated)
-----

  app/chat-tab.h 0bb9dcf 
  app/chat-tab.cpp 5c9308a 
  app/chat-window.h 72bbd1d 
  app/chat-window.cpp a7da574 
  app/telepathy-chat-ui.h 97bc4b7 
  app/telepathy-chat-ui.cpp 33150b8 
  config/behavior-config.h d57fd90 
  config/behavior-config.cpp eeb3597 
  config/behavior-config.ui c8e731c 
  lib/chat-widget.cpp 3929e45 
  lib/notify-filter.h f929ce3 
  lib/notify-filter.cpp 6807dac 
  lib/text-chat-config.h e0ba24f 
  lib/text-chat-config.cpp 57c7c0c 

Diff: https://git.reviewboard.kde.org/r/117543/diff/


Testing
-------


Thanks,

Dan Vrátil

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140417/078f3aa6/attachment.html>


More information about the KDE-Telepathy mailing list