Review Request 128460: ktp-text-ui: Added a filter for geo uri support

Alexandr Akulich akulichalexander at gmail.com
Sat Aug 6 21:18:44 UTC 2016


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

(Updated Aug. 7, 2016, 2:18 a.m.)


Review request for Telepathy.


Changes
-------

- Added zoom optional argument support
- Used QStringRef instead of QString where possible
- Fixed constness
- Fixed style of CMakeFiles.txt
- Plaintext of the geo now will be replaced by href.


Repository: ktp-text-ui


Description
-------

Added a geopoint filter which adds an iframe with a map for each geo:<double,double> in message (RFC 5870).
Uses OpenStreetMap as "backend".


Diffs (updated)
-----

  filters/CMakeLists.txt 8118b13 
  filters/geopoint/CMakeLists.txt PRE-CREATION 
  filters/geopoint/geopoint-filter.h PRE-CREATION 
  filters/geopoint/geopoint-filter.cpp PRE-CREATION 
  filters/geopoint/ktptextui_message_filter_geopoint.desktop.cmake PRE-CREATION 

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


Testing
-------

It works for me.


Thanks,

Alexandr Akulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20160806/452d4324/attachment.html>


More information about the KDE-Telepathy mailing list