[Kde-pim] Review Request 119080: fix timezone config in calendar view

Xuetian Weng wengxt at gmail.com
Wed Jul 16 07:38:54 BST 2014


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

(Updated July 16, 2014, 6:38 a.m.)


Review request for KDEPIM and Sergio Luis Martins.


Changes
-------

add a enum for role in model.


Repository: kdepim


Description
-------

old implementation uses i18n string in return value, which will make timezone configuration not working for other language, this commit uses Qt::UserRole to store the timezone name information.


Diffs (updated)
-----

  calendarviews/agenda/timescaleconfigdialog.cpp 2dbe3e0 

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


Testing (updated)
-------

change timezone under zh_CN now works.


Thanks,

Xuetian Weng

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list