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