Skip to content

Conversation

@Rello
Copy link
Collaborator

@Rello Rello commented Sep 17, 2025

UnifiedSearchInputContainer: Binding loop detected for property "implicitWidth":
qrc:/qt-project.org/imports/QtQuick/NativeStyle/controls/DefaultTextField.qml:16:5
UnifiedSearchInputContainer: Binding loop detected for property "rightPadding":
qrc:/qml/src/gui/tray/UnifiedSearchInputContainer.qml:28:5
UnifiedSearchInputContainer: Binding loop detected for property "rightPadding":
qrc:/qml/src/gui/tray/UnifiedSearchInputContainer.qml:28:5
UnifiedSearchInputContainer: Binding loop detected for property "rightPadding":

Updated the unified search TextField paddings to derive from the leading and trailing control widths, removing the binding loop that stemmed from referencing the field’s own width.

@Rello Rello changed the title fix: Binding loop detected fix: log - Binding loop detected Sep 17, 2025
@Rello Rello force-pushed the bugfix/bindWarning branch from 3cb5047 to b468bb8 Compare September 17, 2025 03:23
@sonarqubecloud
Copy link

@Rello Rello added this to the 3.18.0 milestone Sep 17, 2025
> UnifiedSearchInputContainer: Binding loop detected for property "implicitWidth":
qrc:/qt-project.org/imports/QtQuick/NativeStyle/controls/DefaultTextField.qml:16:5
> UnifiedSearchInputContainer: Binding loop detected for property "rightPadding":
qrc:/qml/src/gui/tray/UnifiedSearchInputContainer.qml:28:5
> UnifiedSearchInputContainer: Binding loop detected for property "rightPadding":
qrc:/qml/src/gui/tray/UnifiedSearchInputContainer.qml:28:5
> UnifiedSearchInputContainer: Binding loop detected for property "rightPadding":

Updated the unified search TextField paddings to derive from the leading and trailing control widths, removing the binding loop that stemmed from referencing the field’s own width.

Signed-off-by: Rello <Rello@users.noreply.github.com>
@Rello Rello force-pushed the bugfix/bindWarning branch from b468bb8 to 3a1c5ab Compare September 17, 2025 08:08
@github-actions
Copy link

Artifact containing the AppImage: nextcloud-appimage-pr-8761.zip

Digest: sha256:f336e10094271d433cd8ae76be1a02f88283d136257cde172926938713d2df91

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@Rello Rello enabled auto-merge September 17, 2025 08:25
@Rello Rello merged commit a9ba5ad into master Sep 17, 2025
16 of 17 checks passed
@Rello Rello deleted the bugfix/bindWarning branch September 17, 2025 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants