Bug 79385 - some icons in kwikdisk menu are too large (patch included)
Summary: some icons in kwikdisk menu are too large (patch included)
Status: RESOLVED FIXED
Alias: None
Product: kwikdisk
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Michael Kropfberger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-04-10 16:01 UTC by felixhaedicke
Modified: 2004-04-12 18:35 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Fix icon size (819 bytes, patch)
2004-04-10 16:02 UTC, felixhaedicke 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description felixhaedicke 2004-04-10 16:01:42 UTC 
Version:            (using KDE KDE 3.2.1)
Installed from:    Compiled From Sources
Compiler:          GCC 3.3.3 
OS:          Linux

When the kwikdisk icon is clicked, three of the icons (for configure kwikdisk, help and quit) are much too large, larger than the other icons.
Comment 1 felixhaedicke 2004-04-10 16:02:37 UTC 
Created attachment 5590 [details]
Fix icon size

Attached a fix
Comment 2 mcamen 2004-04-12 18:30:52 UTC 
CVS commit by mcamen: 

Fix icon size. Patch by felixhaedicke@web.de.

CCMAIL: 79385-done@bugs.kde.org


  M +3 -3      kwikdisk.cpp   1.52


--- kdeutils/kdf/kwikdisk.cpp  #1.51:1.52
@@ -426,9 +426,9 @@ void DockWidget::updateDFDone( void )
 
   mPopupMenu->insertItem(
-    BarIcon( "configure" ),
+    SmallIcon( "configure" ),
     i18n("&Configure KwikDisk..."), this, SLOT(settingsBtnClicked()),0);
 
   mPopupMenu->insertItem(
-    BarIcon( "help" ),
+    SmallIcon( "help" ),
     KStdGuiItem::help().text(), this, SLOT(invokeHelp()),0);
 
@@ -436,5 +436,5 @@ void DockWidget::updateDFDone( void )
 
   mPopupMenu->insertItem(
-    BarIcon( "exit" ),
+    SmallIcon( "exit" ),
     KStdGuiItem::quit().text(), this, SLOT(quit()) );
Comment 3 mcamen 2004-04-12 18:35:15 UTC 
KDE >= 3.2.3 will contain the fix (KDE 3.2.2 is already tagged).