From 1ab372a0afebbcf214f8521b83e0996f30adeae1 Mon Sep 17 00:00:00 2001 From: Martin Rotter Date: Fri, 19 Jan 2024 12:09:53 +0100 Subject: [PATCH] do not save splitter position on resize if position is somehow 0 --- src/librssguard/gui/feedmessageviewer.cpp | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/src/librssguard/gui/feedmessageviewer.cpp b/src/librssguard/gui/feedmessageviewer.cpp index e096ab5d8..323807a38 100644 --- a/src/librssguard/gui/feedmessageviewer.cpp +++ b/src/librssguard/gui/feedmessageviewer.cpp @@ -125,11 +125,20 @@ void FeedMessageViewer::onFeedSplitterResized() { void FeedMessageViewer::onMessageSplitterResized() { qDebugNN << LOGSEC_GUI << "Message splitter moved."; + QList sizes = m_messageSplitter->sizes(); + + if (sizes.size() == 2 && (sizes[0] == 0 || sizes[1] == 0)) { + qWarningNN << LOGSEC_GUI << "Some of splitter position is 0."; + return; + } + + QVariant siz = toVariant(sizes); + if (m_messageSplitter->orientation() == Qt::Orientation::Vertical) { - qApp->settings()->setValue(GROUP(GUI), GUI::SplitterMessagesVertical, toVariant(m_messageSplitter->sizes())); + qApp->settings()->setValue(GROUP(GUI), GUI::SplitterMessagesVertical, siz); } else { - qApp->settings()->setValue(GROUP(GUI), GUI::SplitterMessagesHorizontal, toVariant(m_messageSplitter->sizes())); + qApp->settings()->setValue(GROUP(GUI), GUI::SplitterMessagesHorizontal, siz); } }