From 1b89bcbf5e274d90737f3d0f0068164b4cb503ca Mon Sep 17 00:00:00 2001 From: Eike Ziller Date: Thu, 23 Jun 2022 10:27:26 +0200 Subject: Fix application of value to StringAspect/LineEditDisplay It was not applying the value when changing the text via a completer. That lead to confusing result where values were cut off (and also saved in the settings as such) at seemingly arbitrary cases. Ensure that the text from the line edit ends up in the aspect by also applying it on editingFinished(). One example where that exposed, was the executable aspect in the RemoteLinuxCustomRunConfiguration. Change-Id: I65c434fd675eeead76a73f680c126fd204d7c996 Reviewed-by: Christian Kandeler --- src/libs/utils/aspects.cpp | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/libs/utils/aspects.cpp') diff --git a/src/libs/utils/aspects.cpp b/src/libs/utils/aspects.cpp index 1af70360c1..79f6428a4a 100644 --- a/src/libs/utils/aspects.cpp +++ b/src/libs/utils/aspects.cpp @@ -1138,6 +1138,9 @@ void StringAspect::addToLayout(LayoutBuilder &builder) &FancyLineEdit::textEdited, this, &StringAspect::setValue); + connect(d->m_lineEditDisplay, &FancyLineEdit::editingFinished, this, [this] { + setValue(d->m_lineEditDisplay->text()); + }); } } if (d->m_useResetButton) { -- cgit v1.2.1