From cde02af8f6606334b42af77c94a2c85cf87b15fb Mon Sep 17 00:00:00 2001 From: Axel Spoerl Date: Tue, 13 May 2025 10:31:47 +0200 Subject: [PATCH] qdockarealayout.cpp: add comments to apply(bool aninmate) bool traps apply() of various classes sadly take a boolean argument, to indicate whether an animation should be triggered. Removing this bool trap would require central enumeration. At least document in apply() calls, what the boolean argument is for. Task-number: QTBUG-136716 Pick-to: 6.8 6.5 Change-Id: I5de59fb2813207c09d9de3cfb9ca147c18d943de Reviewed-by: Richard Moe Gustavsen (cherry picked from commit bf38ab9990f30151c148dbff77c8c9bf3982f097) Reviewed-by: Qt Cherry-pick Bot --- src/widgets/widgets/qdockarealayout.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/widgets/widgets/qdockarealayout.cpp b/src/widgets/widgets/qdockarealayout.cpp index 8d243106fc7..8b0901b05b4 100644 --- a/src/widgets/widgets/qdockarealayout.cpp +++ b/src/widgets/widgets/qdockarealayout.cpp @@ -3365,7 +3365,7 @@ int QDockAreaLayout::separatorMove(const QList &separator, const QPoint &or else setGrid(&list, nullptr); - apply(false); + apply(/* animate = */ false); return delta; } @@ -3379,7 +3379,7 @@ int QDockAreaLayoutInfo::separatorMove(const QList &separator, const QPoint delta = pick(info->o, dest - origin); if (delta != 0) delta = info->separatorMove(index, delta); - info->apply(false); + info->apply(/* animate = */ false); return delta; }