]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Word-wrap KMessageWidget text
authorNate Graham <nate@kde.org>
Tue, 12 Feb 2019 19:21:34 +0000 (12:21 -0700)
committerNate Graham <nate@kde.org>
Sun, 17 Feb 2019 21:07:28 +0000 (14:07 -0700)
Summary:
Word-wrap KMessageWidget text to prevent the window from being widened
when the text is very long.

CCBUG: 404232

Only `CCBUG: ` because word wrap only works where QLabel detects a word boundary;
it will not wrap a string with no spaces or other word boundaries (e.g.
"aaaaaaaaaaaaaaaaaaaa...")

Test Plan:
- Create a new text file with a 300-character name that has spaces in it
- In the KMessageWidget's error message, the file name is wrapped at the word
boundaries

Reviewers: #dolphin, #frameworks, cfeck, elvisangelaccio

Reviewed By: #dolphin, cfeck, elvisangelaccio

Subscribers: elvisangelaccio, cfeck, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D18968

src/dolphinviewcontainer.cpp

index 767af64e1c6b103236c15cc517f8653e939ee8f2..4d808d023b40f0d60f9a9e6bec49cf0ca0dc2357 100644 (file)
@@ -312,6 +312,10 @@ void DolphinViewContainer::showMessage(const QString& msg, MessageType type)
 
     m_messageWidget->setText(msg);
 
+    // TODO: wrap at arbitrary character positions once QLabel can do this
+    // https://bugreports.qt.io/browse/QTBUG-1276
+    m_messageWidget->setWordWrap(true);
+
     switch (type) {
     case Information: m_messageWidget->setMessageType(KMessageWidget::Information); break;
     case Warning:     m_messageWidget->setMessageType(KMessageWidget::Warning); break;