| Summary: | Incorrect ""today" highlight in clock/calendar view | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] libplasma | Reporter: | Scott Harvey <scott> |
| Component: | components | Assignee: | Scott Harvey <scott> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ab4bd, abrouwers, acoguemovil, colinnkeenan, fabian, kde, lehaemail, nate, roland, samuel.a.ward |
| Priority: | NOR | ||
| Version: | 5.46.0 | ||
| Target Milestone: | --- | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| URL: | https://bugreports.qt.io/browse/QTBUG-29328 | ||
| Latest Commit: | https://commits.kde.org/plasma-framework/d15f0fa8322dc2632f15024457bea41e7dd058d7 | Version Fixed In: | |
| Attachments: |
Various clocks, same calendar error
hwclock output |
||
Created attachment 112739 [details]
hwclock output
If someone in authority (and another timezone) can confirm this I'll see if I can patch it. Appears to be a bug within Qt (QDate not functioning properly). May not be fixable at this time. If it's a Qt bug, we need a Qt bug report. :) Can you find or file one? Thanks! I'll ask Fabian to file one. He's the one who actually did all the digging and reported that "QDate is basically FUBAR". I'm just doing the housekeeping. :) (In reply to Scott Harvey from comment #5) > I'll ask Fabian to file one. He's the one who actually did all the digging > and reported that "QDate is basically FUBAR". I'm just doing the > housekeeping. :) It got reported for Qt 5.0.2 already, after my ping the priority got bumped from "Low" to "Somewhat important" :-/ *** Bug 393975 has been marked as a duplicate of this bug. *** *** Bug 394995 has been marked as a duplicate of this bug. *** Git commit d15f0fa8322dc2632f15024457bea41e7dd058d7 by Fabian Vogt. Committed on 04/06/2018 at 08:33. Pushed by fvogt into branch 'master'. Use QDateTime for interfacing with QML Summary: QDate from/to JS Date has unexpected results, so use QDateTime to have a timezone assigned to prevent conversions. That way the date values are consistent. The behaviour got changed with Qt 5.11 (see the linked bug report) which lead to plasma showing the wrong dates in the calendar. Test Plan: Tested with and without this patch on Qt 5.10.1 and 5.11.0. Now the correct date is displayed for -0004 and +0001 timezones. Reviewers: #plasma, #frameworks, davidedmundson Reviewed By: #plasma, davidedmundson Subscribers: ngraham, Zren, sharvey, kde-frameworks-devel Tags: #frameworks Differential Revision: https://phabricator.kde.org/D13222 M +13 -7 src/declarativeimports/calendar/calendar.cpp M +12 -5 src/declarativeimports/calendar/calendar.h https://commits.kde.org/plasma-framework/d15f0fa8322dc2632f15024457bea41e7dd058d7 *** Bug 395620 has been marked as a duplicate of this bug. *** *** Bug 395634 has been marked as a duplicate of this bug. *** *** Bug 395836 has been marked as a duplicate of this bug. *** *** Bug 395464 has been marked as a duplicate of this bug. *** |
Created attachment 112738 [details] Various clocks, same calendar error There appears to be a bug in org.plasma.component.calendar that highlights the wrong day for "today". It's a day behind. I tried it in every clock plasmoid that displays a calendar and got the same incorrect result. They all use the same library. I also checked `hwclock` on my machine and it's accurate. System: KF5.47, Qt5.11 (Tumbleweed Krypton)