Skip to content
Verified Commit 3db63a5d authored by ivan tkachenko's avatar ivan tkachenko 🗯
Browse files

BreadcrumbControl: Fix errors from wrong indexing

Binding on the result of indexing a list property with an index property
means relying on two independently and asynchronously updated
properties, and that won't work well, especially on boundary value like
last index. Solution is to guard expressions against null to prepare
them for Qt-flavored "eventual consistency" of signals propagation.

Fixes errors in e.g. Plasmoid configuration window when switching tabs:

    file:///usr/lib/qml/org/kde/kirigami.2/private/globaltoolbar/BreadcrumbControl.qml:16:5: Unable to assign [undefined] to QQuickItem*
    file:///usr/lib/qml/org/kde/kirigami.2/private/globaltoolbar/BreadcrumbControl.qml:16:5: Unable to assign [undefined] to QQuickItem*
    file:///usr/lib/qml/org/kde/kirigami.2/private/globaltoolbar/BreadcrumbControl.qml:16:5: Unable to assign [undefined] to QQuickItem*
parent a4d8afc9
Pipeline #379929 passed with stage
in 4 minutes and 23 seconds
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment