diff --git a/src/contributing.md b/src/contributing.md index ed3b278b..6950bae0 100644 --- a/src/contributing.md +++ b/src/contributing.md @@ -86,7 +86,7 @@ Use `Layout.preferredWidth` and `Layout.preferredHeight` attached properties to Follow [Qt's recommendations](https://doc.qt.io/qt-5/qtqml-documents-scope.html) if appropriate -- Consider dynamic scoping. Component `id`s are not accessible outside the component's scope. If required, the component can expose the instance through properties binding. E.g. `readonly property ComponentType exposedComponent: privateInstanceId` +- Avoid dynamic scoping in out-of-line components, see more: https://doc.qt.io/qt-5/qtqml-documents-scope.html#component-instance-hierarchy - Example ```qml @@ -101,6 +101,8 @@ Follow [Qt's recommendations](https://doc.qt.io/qt-5/qtqml-documents-scope.html) } ``` +- Component `id`s are not accessible outside the component's scope. If required, the component can expose the instance through properties binding. E.g. `readonly property ComponentType exposedComponent: privateInstanceId` + - If in doubt, explicitly use an instance variable to access properties - If the scope is clear and there is no ambiguity, use the property directly for readability @@ -153,4 +155,4 @@ Try scenarios - Resize window to check the behavior for each case - Visually validate that each control is behaving as expected -- Add controls with different properties that affect control size behavior \ No newline at end of file +- Add controls with different properties that affect control size behavior