[Kst] [Bug 121165] Harmonize view object RMB menus in zoom/layout/drawing modes

[George Staikos]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=121165         
staikos kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From staikos kde org  2006-06-18 18:07 -------
SVN commit 552634 by staikos:

Moved edit to the top of the layout menu.  I also shuffled a couple other items
that seemed randomly placed.  Importantly, I pushed delete down so it's not too
close to the top where the menu may show.  Destructive commands should be a bit
harder to reach.  Similar commands are grouped together.

BUG: 121165


 M  +12 -12    kstviewobject.cpp  


--- trunk/extragear/graphics/kst/src/libkstapp/kstviewobject.cpp #552633:552634
 @ -1123,8 +1123,8  @
     menu->insertTitle(tagName());
   }
 
-  if (_layoutActions & Delete) {
-    menu->insertItem(i18n("&Delete"), this, SLOT(deleteObject()));
+  if (_layoutActions & Edit) {
+    menu->insertItem(i18n("&Edit..."), this, SLOT(edit()));
     rc = true;
   }
 
 @ -1133,6 +1133,16  @
     rc = true;
   }
 
+  if (_layoutActions & Delete) {
+    menu->insertItem(i18n("&Delete"), this, SLOT(deleteObject()));
+    rc = true;
+  }
+
+  if (_layoutActions & Rename) {
+    menu->insertItem(i18n("Re&name..."), this, SLOT(rename()));
+    rc = true;
+  }
+
   if (_layoutActions & Raise) {
     index = menu->insertItem(i18n("&Raise"), this, SLOT(raise()));
     rc = true;
 @ -1165,16 +1175,6  @
     }
   }
 
-  if (_layoutActions & Rename) {
-    menu->insertItem(i18n("Re&name..."), this, SLOT(rename()));
-    rc = true;
-  }
-
-  if (_layoutActions & Edit) {
-    menu->insertItem(i18n("&Edit..."), this, SLOT(edit()));
-    rc = true;
-  }
-
   if (_layoutActions & MoveTo) {
     int i = 0;
     bool hasEntry = false;