From b26143561af5fe476f0c5ee933d428ace1a66ad0 Mon Sep 17 00:00:00 2001 From: Nikolai Kosjar Date: Thu, 23 Mar 2017 13:30:11 +0100 Subject: [PATCH] Clang: Make diagnostic display adapt to size of the info bar Change-Id: Ibc7431234b8a68c8e2b0c3f25387de96b203c4c5 Reviewed-by: Tim Jenssen --- .../clangdiagnostictooltipwidget.cpp | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/src/plugins/clangcodemodel/clangdiagnostictooltipwidget.cpp b/src/plugins/clangcodemodel/clangdiagnostictooltipwidget.cpp index 42c77883bfc..62870880fdf 100644 --- a/src/plugins/clangcodemodel/clangdiagnostictooltipwidget.cpp +++ b/src/plugins/clangcodemodel/clangdiagnostictooltipwidget.cpp @@ -112,10 +112,16 @@ private: label->setTextFormat(Qt::RichText); label->setText(text); label->setTextInteractionFlags(Qt::TextBrowserInteraction); - // Using "setWordWrap(true)" alone will wrap the text already for small - // widths, so do not require word wrapping until we hit limits. - if (m_displayHints.limitWidth && label->sizeHint().width() > widthLimit()) { - label->setMaximumWidth(widthLimit()); + + if (m_displayHints.limitWidth) { + const int limit = widthLimit(); + // Using "setWordWrap(true)" alone will wrap the text already for small + // widths, so do not require word wrapping until we hit limits. + if (label->sizeHint().width() > limit) { + label->setMaximumWidth(limit); + label->setWordWrap(true); + } + } else { label->setWordWrap(true); } @@ -141,7 +147,7 @@ private: QString htmlText(const QVector &diagnostics) { // For debugging, add: style='border-width:1px;border-color:black' - QString text = ""; + QString text = "
"; foreach (const ClangBackEnd::DiagnosticContainer &diagnostic, diagnostics) text.append(tableRows(diagnostic));