Bug 339588 - QML binding/anchor loops in org.kde.plasma.notifications/contents/ui/NotificationPopup.qml
Summary: QML binding/anchor loops in org.kde.plasma.notifications/contents/ui/Notific...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (show other bugs)
Version: master
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: 1.0
Assignee: Martin Klapetek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-01 22:14 UTC by Elias Probst
Modified: 2015-06-22 16:53 UTC (History)
0 users

See Also:
Latest Commit: http://commits.kde.org/plasma-workspace/707a9b0bc74af62cdf1fe04cf11c6163a5006518
Version Fixed In: 5.3.2
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Elias Probst 2014-10-01 22:14:59 UTC 
Running latest KF5/P5 from git master + Qt 5.4.0 alpha.

Reproduce it using:
notify-send "a" "b"

plasmashell logs:
1#
file:///usr/share/plasma/plasmoids/org.kde.plasma.notifications/contents/ui/NotificationPopup.qml:136:13: QML Text: Binding loop detected for property "height"

2#
file:///usr/share/plasma/plasmoids/org.kde.plasma.notifications/contents/ui/NotificationPopup.qml:136:13: QML Text: Possible anchor loop detected on vertical anchor.
Comment 1 Martin Klapetek 2014-10-03 12:15:54 UTC 
Are you still getting this? I don't see it.
Comment 2 Martin Klapetek 2014-10-03 12:27:11 UTC 
The error happened with custom patch applied, closing.
Comment 3 Elias Probst 2014-10-03 14:19:30 UTC 
Can be reproduced after calling "notify-send a b" at least 4 times after plasmashell restart.
Comment 4 Martin Klapetek 2014-10-03 14:19:59 UTC 
Yup, indeed.
Comment 5 Elias Probst 2014-12-04 14:22:56 UTC 
Still seeing this in plasmashell's log:

file:///usr/share/plasma/plasmoids/org.kde.plasma.notifications/contents/ui/NotificationPopup.qml:102:23: QML Text: Binding loop detected for property "height"

plasma-workspace@2d44facb
Comment 6 Elias Probst 2015-01-10 00:34:25 UTC 
Still valid for plasma-workspace@156e1dd:

file:///usr/share/plasma/plasmoids/org.kde.plasma.notifications/contents/ui/NotificationPopup.qml:103:23: QML Text: Binding loop detected for property "height"
Comment 7 Martin Klapetek 2015-06-22 16:53:23 UTC 
Git commit 707a9b0bc74af62cdf1fe04cf11c6163a5006518 by Martin Klapetek.
Committed on 22/06/2015 at 16:53.
Pushed by mklapetek into branch 'Plasma/5.3'.

[notifications] Rework the notifications sizing code

This reworks the notification sizing computation to use Layouts. This
should fix all the popup sizing issues there are (including a binding
loop error).

Now it also properly elides after max 4 lines of text.

REVIEW: 124149
FIXED-IN: 5.3.2
Related: bug 349142

M  +75   -73   applets/notifications/package/contents/ui/NotificationItem.qml
M  +1    -0    applets/notifications/package/contents/ui/NotificationPopup.qml

http://commits.kde.org/plasma-workspace/707a9b0bc74af62cdf1fe04cf11c6163a5006518