Alt+drag/click is reserved in KDE.

For those who were wondering (like myself), it is used to move a window.

> ------- For those who were wondering (like myself), it is used to move a
> window.


Oh yeah...  Grumble...

I still think that a modifier key for layout mode would be *very* convenient.

Any ideas from anyone on what we could do?  shift and ctrl are taken.

Should we toggle with some other key?

sigh...

cbn

Maybe Esc ?

 I think it needs to be a modifier key, which Esc doesn't qualify as.

What do you mean by "needs" ? Is this a "philosophical" rule or a technical requiement ?
Because as things stand, I don't see any other choice of "special" key easily accessible with the left hand (sorry left-handed people out there !). The only other possibility might be a combination like Shift+Ctrl+mouse or something else. Personally, I'd prefer Esc...

On Monday 20 February 2006 04:38, Nicolas Brisset wrote:
> 10:38 ------- What do you mean by "needs" ? Is this a "philosophical" rule
> or a technical requiement ? Because as things stand, I don't see any other
> choice of "special" key easily accessible with the left hand (sorry
> left-handed people out there !). The only other possibility might be a
> combination like Shift+Ctrl+mouse or something else. Personally, I'd prefer
> Esc...


  In order to be properly portable, and perhaps to work at all on X11, it has 
to be a modifier key.  ESC is not a modifier key.  QMouseEvent only contains 
the following:

Qt::ShiftButton - a Shift key on the keyboard is also pressed. 
Qt::ControlButton - a Ctrl key on the keyboard is also pressed. 
Qt::AltButton - an Alt key on the keyboard is also pressed. 
Qt::MetaButton - a Meta key on the keyboard is also pressed. 
Qt::Keypad - a keypad button is pressed.

   Holding ESC will just generate a sequence of keys based on the repeat rate 
of the keyboard anyway, I think.

Hum, that sounds bad... I've just experimented with modifier combinations (e.g. Shift+Ctrl, ALT+Ctrl, etc) and it seems there is a notion of precedence. Maybe it's due to the way kst handles multiple simultaneous modifier keys, but otherwise that's not an option either. Meta keys are grabbed by the environment, and the keypad lies on the wrong side of the keyboard. So, what can we do ?

Another alternative would be to provide a toggle function. So (for example) hitting L enters layout mode and remembers the previous mode. Hitting L again returns the user to the previous mode. Not as convenient as the modifier key but better than nothing.

I agree. I can't think of a better alternative...

How about assigning function keys to the modes, rather than a toggle?
F1 is taken, but
F2-F4: zoom modes, F5: layout, F6-F?, view object modes, etc...
These would also have to be in the Right Button Menu.

I think if the user wants to setup such shortcuts they can already do so
using the
existing functionality. Should we let the user pick whatever they want, or
do we
hardcode it for them? My preference would be the former.

  We don't have a default though...  If we set one, they can still change it.  
That's provided for by KDE.  From the sounds of it, we just need to assign a 
default to the action here and that will solve everything.

OK.  Lets make F2-Fn be the defaults for selecting the various mouse modes.

Is there a way of telling the user what the keys are?

The shortcuts will appear on the menu.

I just changed it by hand locally to F2 - F10.  It is really great!  The only issue is that F5 was assigned to reload by default, as it is in konqueror.  I think this is OK to override.  Thoughts?

Created attachment 14800 [details]
Proposed patch that uses F2-F11.

On Tuesday 21 February 2006 18:08, netterfield@astro.utoronto.ca wrote:
> Is there a way of telling the user what the keys are?


  They're visible from the key configuration dialog and from any menu entry 
that shows the action.

On Tuesday 21 February 2006 18:45, netterfield@astro.utoronto.ca wrote:
> 00:45 ------- I just changed it by hand locally to F2 - F10.  It is really
> great!  The only issue is that F5 was assigned to reload by default, as it
> is in konqueror.  I think this is OK to override.  Thoughts?


   I really think it should remain as reload.  This is consistent across so 
many applications now, it would be a shame to break that.

Created attachment 14803 [details]
Proposed patch that uses F2-F4, F6-F12

The F2-F4, F6-F12 patch is approved.  Please apply.

SVN commit 512514 by arwalker:

BUG:121994 Create default shortcuts for all modes

 M  +30 -20    kst.cpp  


--- trunk/extragear/graphics/kst/src/libkstapp/kst.cpp #512513:512514
@@ -449,7 +449,8 @@
   connect(PauseAction, SIGNAL(toggled(bool)), this, SLOT(updatePausedState(bool)));
 
   /************/
-  XYZoomAction = new KRadioAction(i18n("XY Mouse &Zoom"), "kst_zoomxy", 0,
+  XYZoomAction = new KRadioAction(i18n("XY Mouse &Zoom"), "kst_zoomxy",
+                                  KShortcut(Key_F2),
                                   this, SLOT(toggleMouseMode()),
                                   actionCollection(), "zoomxy_action");
   XYZoomAction->setExclusiveGroup("gfx");
@@ -459,7 +460,8 @@
   XYZoomAction->setChecked(true);
 
   /************/
-  XZoomAction = new KRadioAction(i18n("&X Mouse Zoom"), "kst_zoomx", 0,
+  XZoomAction = new KRadioAction(i18n("&X Mouse Zoom"), "kst_zoomx",
+                                 KShortcut(Key_F3),
                                  this, SLOT(toggleMouseMode()),
                                  actionCollection(), "zoomx_action");
   XZoomAction->setExclusiveGroup("gfx");
@@ -468,7 +470,8 @@
                                  "X axis (CTRL-mouse also does this)"));
 
   /************/
-  YZoomAction = new KRadioAction(i18n("&Y Mouse Zoom"), "kst_zoomy", 0,
+  YZoomAction = new KRadioAction(i18n("&Y Mouse Zoom"), "kst_zoomy",
+                                  KShortcut(Key_F4),
                                   this, SLOT(toggleMouseMode()),
                                   actionCollection(), "zoomy_action");
   YZoomAction->setExclusiveGroup("gfx");
@@ -765,53 +768,60 @@
   _tiedZoomAction->setWhatsThis(i18n("Apply zoom actions to all plots\n"
                                      "(not just the active one)."));
 
-  _gfxRectangleAction = new KRadioAction(i18n("&Rectangle"), "kst_gfx_rectangle", 0,
+  _gfxRectangleAction = new KRadioAction(i18n("&Rectangle"), "kst_gfx_rectangle", 
+                                  KShortcut(Key_F8),
                                   this, SLOT(toggleMouseMode()),
                                   actionCollection(), "rectangle_action");
   _gfxRectangleAction->setExclusiveGroup("gfx");
   _gfxRectangleAction->setToolTip(i18n("Draw rectangle"));
   _gfxRectangleAction->setWhatsThis(i18n("Draw rectangle"));
 
-  _gfxEllipseAction = new KRadioAction(i18n("&Ellipse"), "kst_gfx_ellipse", 0,
+  _gfxEllipseAction = new KRadioAction(i18n("&Ellipse"), "kst_gfx_ellipse",
+                                  KShortcut(Key_F9),
                                   this, SLOT(toggleMouseMode()),
                                   actionCollection(), "ellipse_action");
   _gfxEllipseAction->setExclusiveGroup("gfx");
   _gfxEllipseAction->setToolTip(i18n("Draw ellipse"));
   _gfxEllipseAction->setWhatsThis(i18n("Draw ellipse"));
 
-  _gfxPictureAction = new KRadioAction(i18n("&Picture"), "kst_gfx_picture", 0,
-                                       this, SLOT(toggleMouseMode()),
-                                       actionCollection(), "picture_action");
+  _gfxPictureAction = new KRadioAction(i18n("&Picture"), "kst_gfx_picture",
+                                   KShortcut(Key_F12),
+                                   this, SLOT(toggleMouseMode()),
+                                   actionCollection(), "picture_action");
   _gfxPictureAction->setExclusiveGroup("gfx");
   _gfxPictureAction->setToolTip(i18n("Insert picture"));
   _gfxPictureAction->setWhatsThis(i18n("Insert picture"));
 
-  _gfxArrowAction = new KRadioAction(i18n("&Arrow"), "kst_gfx_arrow", 0,
-                                     this, SLOT(toggleMouseMode()),
-                                     actionCollection(), "arrow_action");
+  _gfxArrowAction = new KRadioAction(i18n("&Arrow"), "kst_gfx_arrow",
+                                   KShortcut(Key_F11),
+                                   this, SLOT(toggleMouseMode()),
+                                   actionCollection(), "arrow_action");
   _gfxArrowAction->setExclusiveGroup("gfx");
   _gfxArrowAction->setToolTip(i18n("Draw arrow"));
   _gfxArrowAction->setWhatsThis(i18n("Draw arrow"));
 
-  _gfxLineAction = new KRadioAction(i18n("&Line"), "kst_gfx_line", 0,
-                                  this, SLOT(toggleMouseMode()),
-                                  actionCollection(), "line_action");
+  _gfxLineAction = new KRadioAction(i18n("&Line"), "kst_gfx_line",
+                                   KShortcut(Key_F10),
+                                   this, SLOT(toggleMouseMode()),
+                                   actionCollection(), "line_action");
   _gfxLineAction->setExclusiveGroup("gfx");
   _gfxLineAction->setToolTip(i18n("Draw line"));
   _gfxLineAction->setWhatsThis(i18n("Draw line"));
   _gfxLineAction->setChecked(true);
 
   /************/
-  _gfxLabelAction = new KRadioAction(i18n("L&abel"), "text", 0,
-                                this, SLOT(toggleMouseMode()),
-                                actionCollection(), "label_action");
+  _gfxLabelAction = new KRadioAction(i18n("L&abel"), "text",
+                                   KShortcut(Key_F7),
+                                   this, SLOT(toggleMouseMode()),
+                                   actionCollection(), "label_action");
   _gfxLabelAction->setExclusiveGroup("gfx");
   _gfxLabelAction->setToolTip(i18n("Draw label"));
   _gfxLabelAction->setWhatsThis(i18n("Draw label"));
 
-  LayoutAction = new KRadioAction(i18n("Layout Mode"), "kst_layoutmode", 0,
-                                  this, SLOT(toggleMouseMode()),
-                                  actionCollection(), "layoutmode_action");
+  LayoutAction = new KRadioAction(i18n("Layout Mode"), "kst_layoutmode",
+                                   KShortcut(Key_F6),
+                                   this, SLOT(toggleMouseMode()),
+                                   actionCollection(), "layoutmode_action");
   LayoutAction->setExclusiveGroup("gfx");
   LayoutAction->setToolTip(i18n("Layout mode"));
   LayoutAction->setWhatsThis(i18n("Use this mode to move, resize, and group plots."));