Review Request 128841: Fix message classes for history items

Mariusz Glebocki mglb at arccos-1.net
Mon Sep 5 20:17:02 UTC 2016


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

Review request for Telepathy.


Repository: ktp-text-ui


Description
-------

"incoming" and "outgoing" classes for history messages are swapped - they are used for local to remote and remote to local messages, respectively. This bug is visible in styles that use a single .html file for sent and received messages, relying on a class.
I don't know which theme could be used to test it - I spotted this while playing with theme creation. Everything without an "Outgoing" directory and with distinguishable incoming/outgoing messages should be ok.


Diffs
-----

  lib/adium-theme-message-info.cpp ce1d5cd 

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


Testing
-------

- compile/run: OK
- talk with someone, close the window, open the chat again, check the last chat message's direction: OK


Thanks,

Mariusz Glebocki

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20160905/956b6dcc/attachment.html>


More information about the KDE-Telepathy mailing list