[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    [patch] do not display links (KUrlLabel) with ampersands
From:       Jarosław_Staniek <js () iidea ! pl>
Date:       2006-08-22 11:59:32
Message-ID: 200608221359.32380.js () iidea ! pl
[Download RAW message or body]

OK to commit, or is there cleaner way to do this?

Optionally we could provide a way to display accelerators for links, e.g. 
stronger underline, or displaying a letter in bold.

-- 
regards / pozdrawiam, Jaroslaw Staniek
 Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
 Kexi & KOffice: http://www.kexi-project.org, http://www.koffice.org
 KDE3 & KDE4 Libraries for MS Windows: http://kdelibs.com, http://www.kde.org

["kacceleratormanager.patch" (text/x-diff)]

Index: kdecore/kacceleratormanager.cpp
===================================================================
--- kdecore/kacceleratormanager.cpp	(wersja 575825)
+++ kdecore/kacceleratormanager.cpp	(kopia robocza)
@@ -235,6 +235,9 @@
         // we possibly reserved an accel, but we won't set it as it looks silly
         if ( qobject_cast<Q3GroupBox*>( it->m_widget ) )
              continue;
+        // links look weird with ampersands
+        if ( qobject_cast<QLabel*>( it->m_widget ) && \
it->m_widget->inherits("KURLLabel") ) +             continue;
 
         kDebug(131) << "write " << cnt << " " << \
it->m_widget->metaObject()->className() << " " <<contents[cnt].accelerated() << endl; \




[prev in list] [next in list] [prev in thread] [next in thread]