diff options
author | Edward Welbourne <edward.welbourne@qt.io> | 2017-11-27 14:18:24 +0100 |
---|---|---|
committer | Edward Welbourne <edward.welbourne@qt.io> | 2017-11-30 20:21:42 +0000 |
commit | c27348985d528030d2b7d69f1734a26e421172b6 (patch) | |
tree | 3e9aa9e25205637b3457c80103c4ddf5106c0d14 /src/platformsupport/services | |
parent | 454bbbe787003be8ba41999ad5e3237b02deafd7 (diff) |
Simplify date-time formatting in textForRole()
QAbstractItemDelegatePrivate::textForRole() was formatting date and
time separately, then gluing the parts back together with space.
QLocale can do that just fine itself (it has a toString() overload
that handles a QDateTime) and might even (some day) do it better. To
my mild surprise, this proved sufficient to fix a problem with
date-time display in tool-tips, when the date-time includes a zone.
Extracted the date-time part of an existing selftest into a test of
its own and extended it to test times of each spec-type; verified that
the non-local spec cases of this all failed before this fix.
Task-number: QTBUG-61069
Change-Id: I6d6be0c27be9a557d8afc3ced200a10b2aaff816
Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
Diffstat (limited to 'src/platformsupport/services')
0 files changed, 0 insertions, 0 deletions