From kde-core-devel  Tue Aug 22 11:59:32 2006
From: =?utf-8?q?Jaros=C5=82aw_Staniek?= <js () iidea ! pl>
Date: Tue, 22 Aug 2006 11:59:32 +0000
To: kde-core-devel
Subject: [patch] do not display links (KUrlLabel) with ampersands
Message-Id: <200608221359.32380.js () iidea ! pl>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=115624806520733
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--Boundary-00=_kGv6ELhHqfsxXGr"

--Boundary-00=_kGv6ELhHqfsxXGr
Content-Type: text/plain;
  charset="us-ascii"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline


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

--Boundary-00=_kGv6ELhHqfsxXGr
Content-Type: text/x-diff;
  charset="us-ascii";
  name="kacceleratormanager.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
	filename="kacceleratormanager.patch"

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;
 

--Boundary-00=_kGv6ELhHqfsxXGr--