From 8999b35fcf8a89827106a2db486d28e274ab300c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thorbj=C3=B8rn=20Lindeijer?= <thorbjorn.lindeijer@nokia.com> Date: Wed, 21 Apr 2010 14:50:35 +0200 Subject: [PATCH] Only escape the code tooltip when it is used as rich text Since the fix in 01a0ec161cf5e814b53151a040afc2ac1acd69f8, sometimes & would turn up in the tooltips due to them being escaped but not interpreted as HTML. --- src/plugins/cppeditor/cpphoverhandler.cpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/plugins/cppeditor/cpphoverhandler.cpp b/src/plugins/cppeditor/cpphoverhandler.cpp index a2e270b3e10..2c91f5cba4b 100644 --- a/src/plugins/cppeditor/cpphoverhandler.cpp +++ b/src/plugins/cppeditor/cpphoverhandler.cpp @@ -413,21 +413,17 @@ void CppHoverHandler::updateHelpIdAndTooltip(TextEditor::ITextEditor *editor, in } } - if (!formatTooltip.isEmpty()) { + if (!formatTooltip.isEmpty()) m_toolTip = formatTooltip; - } - - if (!m_toolTip.isEmpty()) - m_toolTip = Qt::escape(m_toolTip); if (!m_helpId.isEmpty() && !helpLinks.isEmpty()) { if (showF1) { m_toolTip = QString(QLatin1String("<table><tr><td valign=middle><nobr>%1</td>" "<td><img src=\":/cppeditor/images/f1.png\"></td></tr></table>")) - .arg(m_toolTip); + .arg(Qt::escape(m_toolTip)); } editor->setContextHelpId(m_helpId); } else if (!m_toolTip.isEmpty() && Qt::mightBeRichText(m_toolTip)) { - m_toolTip = QString(QLatin1String("<nobr>%1")).arg(m_toolTip); + m_toolTip = QString(QLatin1String("<nobr>%1")).arg(Qt::escape(m_toolTip)); } } -- GitLab