https://bugs.kde.org/show_bug.cgi?id=390330

Chris Holland <zrenf...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zrenf...@gmail.com

--- Comment #3 from Chris Holland <zrenf...@gmail.com> ---
This is because DaysCalendar.qml uses:
https://github.com/KDE/plasma-framework/blob/8497832a8bf088a9011052841a50863ab40bf062/src/declarativeimports/calendar/qml/DaysCalendar.qml#L315

Assumes firstDayOfWeek is 0 (aka Sunday). Using firstDayOfWeek=6 (Saturday)
breaks this logic.

    text:
Qt.locale(Qt.locale().uiLanguages[0]).dayName(calendarBackend.firstDayOfWeek +
index, Locale.ShortFormat)

instead of:
    calendarBackend.firstDayOfWeek + index
we need to use:
    (calendarBackend.firstDayOfWeek + index) % 7

I'm assuming the number of days in a week doesn't change (7), though I'm not
sure if there's a variable we can use to substitute.
http://doc.qt.io/qt-5/qml-qtqml-locale.html#weekDays-prop

You should be able to modify this locally, then restart plasmashell, or wait
for a patch.
/usr/lib/x86_64-linux-gnu/qt5/qml/org/kde/plasma/calendar/DaysCalendar.qml

I'm not sure if locales that start on Saturday or Monday also experience this.
Gonna check.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to