Bug 403518

Summary: Cropped notification indicator and unexpected transition shadow when zoomed
Product: [Unmaintained] lattedock Reporter: Viorel-Cătălin Răpițeanu <rapiteanu.catalin>
Component: applicationAssignee: Michail Vourlakos <mvourlakos>
Status: RESOLVED FIXED    
Severity: minor    
Priority: NOR    
Version First Reported In: git (master)   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Application switch highlight issue.

Description Viorel-Cătălin Răpițeanu 2019-01-22 21:40:36 UTC
SUMMARY:
In dock form, the notification indicator (in case of large numbers, like +9999), gets cropped by the neighbor application icon, even if that's transparent. Also, as a related note, when you zoom between these 2 items, you get a small visible vertical rectangle on each icon (see attached video). 

STEPS TO REPRODUCE
1. Add next to each other 2 round icons (like Steam and Spotify) and set the notification indicator for the application on the right to +9999.
2. Slowly zoom between those 2 application.

OBSERVED RESULT
1. The notification gets cropped, even if there is enough space to fully display  it.
2. You can see a small rectangle (application box), when you transition between them.

SOFTWARE/OS VERSIONS
Monitor resolution: 3840 x 2160
Linux/KDE Plasma: 5.14.90
KDE Plasma Version: 5.14.90
KDE Frameworks Version: 5.54.0
Qt Version: 5.12.0-3
Comment 1 Viorel-Cătălin Răpițeanu 2019-01-22 21:44:06 UTC
Video of the issue: https://drive.google.com/open?id=1ZR5jXB5UWxtHlD3Hr8DEIiUoKrx2Lott
Comment 2 Michail Vourlakos 2019-01-23 20:48:34 UTC
Git commit c3ddce3c864bc85ef8b547fd767efdf19b805c6e by Michail Vourlakos.
Committed on 23/01/2019 at 20:47.
Pushed by mvourlakos into branch 'master'.

improve Tasks IconItem visuals/badges positioning

--in order to be able to draw correctly the badges when
they exceed the tasks boundaries the effects needed to
be repositioned in Task/IconItem. So now shadows/colorizers
etc. provide better freedom from code point of view.

M  +94   -42   plasmoid/package/contents/ui/task/IconItem.qml

https://commits.kde.org/latte-dock/c3ddce3c864bc85ef8b547fd767efdf19b805c6e
Comment 3 Viorel-Cătălin Răpițeanu 2019-01-23 21:26:53 UTC
The notification badge is displayed correctly right now. The issue of the transition shadow remained. Should I open a new ticket for that?

When you set the notification for one app at 9999+ and you switch between icons remained. To see this easily, set the zoom level at 5% and use huge application icons.
Comment 4 Viorel-Cătălin Răpițeanu 2019-01-23 21:29:33 UTC
Created attachment 117625 [details]
Application switch highlight issue.
Comment 5 Michail Vourlakos 2019-01-23 21:51:36 UTC
Git commit 37558dd20c7215e4164adb21ebf746e7441a9a3c by Michail Vourlakos.
Committed on 23/01/2019 at 21:50.
Pushed by mvourlakos into branch 'master'.

dont include tasks badges at brightness effects

M  +57   -42   plasmoid/package/contents/ui/task/IconItem.qml

https://commits.kde.org/latte-dock/37558dd20c7215e4164adb21ebf746e7441a9a3c
Comment 6 Michail Vourlakos 2019-01-23 21:53:10 UTC
(In reply to Viorel-Cătălin Răpițeanu from comment #3)
> The issue of the transition shadow remained. Should I open a new ticket for that?

yes please and provide a video demonstrating with the latest commits because in your current video I can not notice any problem with shadows
Comment 7 Viorel-Cătălin Răpițeanu 2019-01-23 22:07:02 UTC
> Git commit 37558dd20c7215e4164adb21ebf746e7441a9a3c by Michail Vourlakos.
Everything is working as expected using the latest commit. Thanks.
Comment 8 Viorel-Cătălin Răpițeanu 2019-01-23 22:11:18 UTC
Related to this change, there is another small papercut when the application with the notification badge is the first application in the dock. I'll add a video for that issue soon.
Comment 9 Viorel-Cătălin Răpițeanu 2019-01-23 22:18:53 UTC
Badge crop when the dock is not focused:
https://drive.google.com/open?id=1_9HRguzUrW78LSSYSBdG8nXZvr6JV90h
Comment 10 Michail Vourlakos 2019-01-24 00:26:27 UTC
(In reply to Viorel-Cătălin Răpițeanu from comment #9)
> Badge crop when the dock is not focused:
> https://drive.google.com/open?id=1_9HRguzUrW78LSSYSBdG8nXZvr6JV90h

Unfortunately this can be solved by making sure that the Info badge is always at the limits of the task size. That means that 9999+ could just change to 999+
Comment 11 Viorel-Cătălin Răpițeanu 2019-01-24 12:03:12 UTC
That could be a solution.
Comment 12 Michail Vourlakos 2019-01-24 13:31:57 UTC
Git commit 9851a71293a1bd4841a34597c4369489d26d7112 by Michail Vourlakos.
Committed on 24/01/2019 at 13:29.
Pushed by mvourlakos into branch 'master'.

elide InfoBadge for first task when needed

M  +4    -0    liblatte2/qml/BadgeText.qml
M  +4    -1    plasmoid/package/contents/ui/task/ProgressOverlay.qml

https://commits.kde.org/latte-dock/9851a71293a1bd4841a34597c4369489d26d7112
Comment 13 Viorel-Cătălin Răpițeanu 2019-01-24 13:40:47 UTC
That looks really good. Thanks for looking into this.