Bug 445459 - SelectableLabel.qml:36:5: QML TextArea: Binding loop detected for property "implicitHeight"
Summary: SelectableLabel.qml:36:5: QML TextArea: Binding loop detected for property "i...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (show other bugs)
Version: master
Platform: openSUSE Linux
: NOR normal
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords:
: 461830 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-11-14 07:20 UTC by Fushan Wen
Modified: 2022-11-14 22:33 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.27
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Fushan Wen 2021-11-14 07:20:14 UTC
SUMMARY
When a notification with icon appears, a binding loop will be detected.

file:///usr/share/plasma/plasmoids/org.kde.plasma.notifications/contents/ui/SelectableLabel.qml:36:5: QML TextArea: Binding loop detected for property "implicitHeight"

Example: Element new message notification

Operating System: openSUSE Tumbleweed 20211110
KDE Plasma Version: 5.23.80
KDE Frameworks Version: 5.88.0
Qt Version: 5.15.2
Graphics Platform: X11
Comment 1 Bug Janitor Service 2021-11-14 07:25:33 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/380
Comment 2 Fushan Wen 2021-11-15 02:21:58 UTC
Git commit a7fd79d32ee497786c12357a04a4a71dabd0c03b by Fushan Wen.
Committed on 15/11/2021 at 02:19.
Pushed by fusionfuture into branch 'master'.

PC3.TextArea: Delay binding on implicitHeight

This eliminates the binding loop warnings in desktop notifications.

M  +7    -4    src/declarativeimports/plasmacomponents3/TextArea.qml

https://invent.kde.org/frameworks/plasma-framework/commit/a7fd79d32ee497786c12357a04a4a71dabd0c03b
Comment 3 Fushan Wen 2021-11-24 00:58:06 UTC
Reopened
Comment 5 Nate Graham 2022-10-31 14:50:35 UTC
Git commit ea062b22c027a5e1c0833c4cf7444d24053c1491 by Nate Graham, on behalf of Fushan Wen.
Committed on 31/10/2022 at 14:47.
Pushed by ngraham into branch 'master'.

applets/notification: port NotificationItem to anchor-based positioning

Layout + ScrollView produces unpredictable results, and since ScrollView
depends on the content height to adjust its size properties, but
re-layout will create unnecessary size adjustments for ScrollView in
Layout, there will be many binding loop warnings in the log.
Anchor-based positioning does not have those issues and is more
efficient.

Test: run

while [ 1 -eq 1 ]; do notify-send -a "Notification Test" "Evil Layout" "$(openssl rand -base64 64)";sleep 1;done

and watch the log of plasmashell.
FIXED-IN: 5.27

M  +217  -231  applets/notifications/package/contents/ui/NotificationItem.qml

https://invent.kde.org/plasma/plasma-workspace/commit/ea062b22c027a5e1c0833c4cf7444d24053c1491
Comment 6 Nate Graham 2022-11-14 22:33:15 UTC
*** Bug 461830 has been marked as a duplicate of this bug. ***