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

List:       kde-commits
Subject:    KDE/kdebase/workspace/libs/taskmanager
From:       Anton Kreuzkamp <akreuzkamp () web ! de>
Date:       2010-11-14 19:22:32
Message-ID: 20101114192232.B60E3AC8A0 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1197054 by akreuzkamp:

Fixes bug that caused the launcher-tooltips to show lowercased names

 M  +5 -5      taskactions.cpp  


--- trunk/KDE/kdebase/workspace/libs/taskmanager/taskactions.cpp #1197053:1197054
@@ -313,11 +313,11 @@
     setText(i18n("&Pin Task"));
     setCheckable(true);
     if (item->itemType() == TaskItemType) {
-        m_name = qobject_cast< TaskItem* >(item)->task()->classClass().toLower();
+        m_name = qobject_cast< TaskItem* >(item)->task()->classClass();
     } else {
-        m_name = item->name().toLower();
+        m_name = item->name();
     }
-    setChecked(m_groupingStrategy->findLauncher(m_name));
+    setChecked(m_groupingStrategy->findLauncher(m_name.toLower()));
     if (!m_groupingStrategy->findLauncher(m_name)) {
         // Search for applications which are executable and case-insensitively match \
                the windowclass of the task and
         // See http://techbase.kde.org/Development/Tutorials/Services/Traders#The_KTrader_Query_Language
 @@ -340,8 +340,8 @@
 
 void ToggleLauncherActionImpl::toggleLauncher()
 {
-    if (m_groupingStrategy->findLauncher(m_name)) {
-        m_groupingStrategy->removeLauncher(m_groupingStrategy->findLauncher(m_name));
 +    if (m_groupingStrategy->findLauncher(m_name.toLower())) {
+        m_groupingStrategy->removeLauncher(m_groupingStrategy->findLauncher(m_name.toLower()));
  } else if (m_url.isValid()) {
         if (m_url.isLocalFile() && KDesktopFile::isDesktopFile(m_url.toLocalFile())) \
{  m_groupingStrategy->addLauncher(m_url);


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