Bug 121160 - Change the layout mode icon
Summary: Change the layout mode icon
Status: RESOLVED INTENTIONAL
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: unspecified Solaris
: NOR wishlist
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-01 16:47 UTC by Nicolas Brisset
Modified: 2006-05-24 23:54 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.