Bug 121160

Summary:	Change the layout mode icon
Product:	[Applications] kst	Reporter:	Nicolas Brisset <nicolas.brisset>
Component:	general	Assignee:	kst
Status:	RESOLVED INTENTIONAL
Severity:	wishlist
Priority:	NOR
Version:	1.x
Target Milestone:	---
Platform:	unspecified
OS:	Solaris
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Nicolas Brisset 2006-02-01 16:47:22 UTC

Version:           1.2.0_devel (using KDE 3.4.0, compiled sources)
Compiler:          gcc version 3.4.3
OS:                SunOS (sun4u) release 5.8

This is a very small request, but important usability-wise: the layout mode icon is not very standard and people tend to not find it. An arrow (or 4-way arrow) would be better.

Comment 1 Andrew Walker 2006-02-01 23:52:21 UTC

Proposed patch:

Index: kst.cpp
===================================================================
--- kst.cpp     (revision 504664)
+++ kst.cpp     (working copy)
@@ -809,7 +809,7 @@
   _gfxLabelAction->setToolTip(i18n("Draw label"));
   _gfxLabelAction->setWhatsThis(i18n("Draw label"));

-  LayoutAction = new KRadioAction(i18n("Layout Mode"), "kst_layoutmode", 0,
+  LayoutAction = new KRadioAction(i18n("Layout Mode"), "move", 0,
                                   this, SLOT(toggleMouseMode()),
                                   actionCollection(), "layoutmode_action");
   LayoutAction->setExclusiveGroup("gfx");

Comment 2 Andrew Walker 2006-02-21 00:51:07 UTC

I think we should either change the icon as suggested or mark the bug as INVALID.

Comment 3 George Staikos 2006-02-21 05:20:35 UTC

   Instead of using the system icon (which is defined by the theme), we should 
change the icon we install.  (If the art czar wants it changed. :-))

Comment 4 Andrew Walker 2006-05-24 23:54:50 UTC

Doesn't seem to be any desire to change the icon for this.