Bug 440627 - A pinned calendar displays the wrong number of dots
Summary: A pinned calendar displays the wrong number of dots
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Digital Clock (show other bugs)
Version: 5.22.4
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-05 09:15 UTC by popov895
Modified: 2021-08-09 15:56 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.86


Attachments
Dots on days tab (31.55 KB, image/png)
2021-08-05 09:15 UTC, popov895
Details
Dots on months tab (39.80 KB, image/png)
2021-08-05 09:16 UTC, popov895
Details

Note You need to log in before you can comment on or make changes to this bug.
Description popov895 2021-08-05 09:15:34 UTC
Created attachment 140539 [details]
Dots on days tab

SUMMARY

A pinned calendar displays a different number of dots every time you open its settings dialog. At first I thought this problem only related to my openSUSE Tumbleweed, but after switching to KDE neon, it appeared here as well.

After a little research, I came up with some information:
* the number of dots changes only for days with events
* dots are also displayed on the months tab because the day cell has no clipping
* the number of dots always changes cyclically: 0 => real number of events => 42 => 0 => ... (the number 42 is most likely related to the number of days displayed on the days tabs)
* looks like every new opening of the settings dialog affects how the DelegateModel calculates the rootIndex (https://invent.kde.org/frameworks/plasma-framework/-/blob/master/src/declarativeimports/calendar/qml/DayDelegate.qml#L84)

STEPS TO REPRODUCE

1. Open Digital Clock applet
2. Check "Keep Open" button in the top right corner
3. Open Settings dialog (clicking "Configure Digital Clock..." button or menuitem with the same name in the context menu) and close it
4. Repeat step 3 several times

SOFTWARE/OS VERSIONS

Operating System: KDE neon 5.22
KDE Plasma Version: 5.22.4
KDE Frameworks Version: 5.84.0
Qt Version: 5.15.3
Kernel Version: 5.11.0-25-generic (64-bit)
Graphics Platform: X11
Comment 1 popov895 2021-08-05 09:16:21 UTC
Created attachment 140540 [details]
Dots on months tab
Comment 2 Nate Graham 2021-08-05 14:04:56 UTC
Huh, I thought we had fixed this. :(

If it's reproducible and you've already dug into the code, please feel free to submit a merge request.
Comment 3 popov895 2021-08-05 15:47:34 UTC
Is it just me again?
Comment 4 Bug Janitor Service 2021-08-06 19:42:54 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/309
Comment 5 Bug Janitor Service 2021-08-07 08:55:12 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/310
Comment 6 Nate Graham 2021-08-09 15:56:06 UTC
Git commit 19fe24ef9377178413438d7749630053995af229 by Nate Graham, on behalf of Eugene Popov.
Committed on 09/08/2021 at 15:56.
Pushed by ngraham into branch 'master'.

[Calendar] Fix pinned calendar displaying the wrong number of dots

Use assignment instead of binding for the rootIndex property of the DelegateModel.
FIXED-IN: 5.86

M  +2    -1    src/declarativeimports/calendar/qml/DayDelegate.qml

https://invent.kde.org/frameworks/plasma-framework/commit/19fe24ef9377178413438d7749630053995af229