Created attachment 173848 [details] text overflow SUMMARY If alternate calendar is enabled, the date heading overflows STEPS TO REPRODUCE 1. Create a digital clock widget 2. In settings -> calendar -> available plugins, turn on Alternate Calendar. 3. Click on the widget OBSERVED RESULT The text overflows on the date heading. EXPECTED RESULT The text should not overflow. SOFTWARE/OS VERSIONS Windows: macOS: (available in the Info Center app, or by running `kinfo` in a terminal window) Linux/KDE Plasma: 6.6.38 / master KDE Plasma Version: master KDE Frameworks Version: master Qt Version: 6.7.2 ADDITIONAL INFORMATION
Created attachment 173857 [details] can't reproduce might be an issue of that chinese font in particular? (i assume the selected alternalte calendal plugin is chinese lunar calendar?) here is how it renders locally
Are you using a custom Plasma theme, Tusooa? Can you share a screenshot that shows more context? Does it reproduce in a new clean user account, with the only customization being that Alternate Calendar is enabled?
Created attachment 173898 [details] Full view of Moe
Created attachment 173899 [details] Full view of Breeze
(In reply to Nate Graham from comment #2) > Are you using a custom Plasma theme, Tusooa? Can you share a screenshot that > shows more context? Does it reproduce in a new clean user account, with the > only customization being that Alternate Calendar is enabled? Using Moe, the version I have can be downloaded at https://f.tusooa.xyz/moe.tar . Judging from the code, the events view just uses the header height of the month view, and hope it would be enough. Sadly, this is not true for all themes. It is unrelated to Chinese font, as the Julian calendar suffers from the same problem.
I notice your font size is quite large, and that's likely the indirect cause of the issue, which triggers the bug due to the code issue you found. I can reproduce this with an alternate calendar set and both the Moe and Breeze Plasma themes with 16pt Noto Sans. It's more obvious with Moe, but still clearly an issue with Breeze too. What font and font size are you using?
(In reply to Nate Graham from comment #6) > I notice your font size is quite large, and that's likely the indirect > cause of the issue, which triggers the bug due to the code issue you found. > I can reproduce this with an alternate calendar set and both the Moe and > Breeze Plasma themes with 16pt Noto Sans. It's more obvious with Moe, but > still clearly an issue with Breeze too. > > What font and font size are you using? Sans Serif 15pt (which is DejaVu Sans configured by fontconfig), small font being 13pt. Monospace is 13 pt, being Ubuntu Mono by fontconfig.
I'm tempted to close this as RESOLVED INTENTIONAL since it appears you're drastically changing the font size as a way to avoid using the normal scaling system, which is not formally supported, and the Fonts KCM even shows a warning about this when you set the font size to 14pt or larger. :) However I could be persuaded to keep the Bugzilla ticker open if you plan to submit a patch that fixes it. Are you planning to do so?
(In reply to Nate Graham from comment #8) > I'm tempted to close this as RESOLVED INTENTIONAL since it appears you're > drastically changing the font size as a way to avoid using the normal > scaling system, which is not formally supported, and the Fonts KCM even > shows a warning about this when you set the font size to 14pt or larger. :) > > However I could be persuaded to keep the Bugzilla ticker open if you plan to > submit a patch that fixes it. Are you planning to do so? > 14pt or larger except that it still does not look correct when I set the font to 13pt; the spacing does not look correct. I will try to fix it.
Created attachment 174033 [details] 13pt font; missing bottom spacing
Created attachment 174034 [details] 14pt font; overflow
Yes, I can confirm that with the MOE theme.
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/4746
Git commit c083f4674fe09aeaad4fc129604749ee5e710f80 by Nate Graham, on behalf of tusooa the west wind. Committed on 26/09/2024 at 15:07. Pushed by ngraham into branch 'master'. Fix potential overflow of agenda heading of digital clock The original approach to determining the heading height is to just use the height of the MonthView and hope it would be enough. This assumption is untrue for many font/theme combinations and can cause spacing issues even if the height happens to be enough. We fix this by synchronizing the height of both headers to be the larger one. M +6 -1 applets/digital-clock/package/contents/ui/CalendarView.qml M +3 -1 components/calendar/qml/MonthViewHeader.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/c083f4674fe09aeaad4fc129604749ee5e710f80
Git commit fead85fc2245864af2b6f0bb3ec36c8f12d3fda1 by Nate Graham. Committed on 26/09/2024 at 15:29. Pushed by ngraham into branch 'Plasma/6.2'. Fix potential overflow of agenda heading of digital clock The original approach to determining the heading height is to just use the height of the MonthView and hope it would be enough. This assumption is untrue for many font/theme combinations and can cause spacing issues even if the height happens to be enough. We fix this by synchronizing the height of both headers to be the larger one. (cherry picked from commit c083f4674fe09aeaad4fc129604749ee5e710f80) Co-authored-by: tusooa the west wind <tusooa@kazv.moe> M +6 -1 applets/digital-clock/package/contents/ui/CalendarView.qml M +3 -1 components/calendar/qml/MonthViewHeader.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/fead85fc2245864af2b6f0bb3ec36c8f12d3fda1