]> git.lyx.org Git - lyx.git/blobdiff - src/lyxfunc.C
fixes/cleanup to dispatch/getStatus stuff; fixes the bug where unwanted function...
[lyx.git] / src / lyxfunc.C
index 860b6958bca394340912fb1dfb0beaa81d9a84de..c76ebbdd9ccf82295d657bb4c23415bf1232df5d 100644 (file)
 
 #include <config.h>
 
-#include "Lsstream.h"
-
-#include <time.h>
-#include <locale.h>
-#include <utility> 
-#include <algorithm> 
-
-#include <cstdlib>
-#include <cctype>
-
 #ifdef __GNUG__
 #pragma implementation
 #endif
 
-#include "support/lyxalgo.h"
+#include "lyxfunc.h"
 #include "version.h"
 #include "kbmap.h"
-#include "lyxfunc.h"
+#include "lyxrow.h"
 #include "bufferlist.h"
 #include "BufferView.h"
 #include "ColorHandler.h"
 #include "lyx_main.h"
 #include "lyx_cb.h"
 #include "LyXAction.h"
+#include "debug.h"
+#include "lyxrc.h"
+#include "lyxtext.h"
+#include "gettext.h"
+#include "Lsstream.h"
+#include "trans_mgr.h"
+#include "layout.h"
+#include "WorkArea.h"
+#include "bufferview_funcs.h"
+#include "minibuffer.h"
+#include "vspace.h"
+#include "LyXView.h"
+#include "lyx_gui_misc.h"
+#include "FloatList.h"
+#include "converter.h"
+#include "exporter.h"
+#include "importer.h"
+#include "FontLoader.h"
+#include "TextCache.h"
+#include "lyxfind.h"
+#include "undo_funcs.h"
+#include "ParagraphParameters.h"
+#include "figureForm.h"
+
 #include "insets/inseturl.h"
 #include "insets/insetlatexaccent.h"
 #include "insets/insettoc.h"
 #include "insets/insetmarginal.h"
 #include "insets/insetminipage.h"
 #include "insets/insetfloat.h"
+#if 0
 #include "insets/insetlist.h"
-#include "insets/insettabular.h"
 #include "insets/insettheorem.h"
+#endif
+#include "insets/insettabular.h"
 #include "insets/insetcaption.h"
+
 #include "mathed/formulamacro.h"
 #include "mathed/math_cursor.h"
 #include "mathed/math_inset.h"
-#include "minibuffer.h"
-#include "vspace.h"
-#include "LyXView.h"
-#include "lyx_gui_misc.h"
+
+#include "frontends/FileDialog.h"
+#include "frontends/Dialogs.h"
+#include "frontends/Toolbar.h"
+#include "frontends/Menubar.h"
+#include "frontends/Alert.h"
+
+#include "support/lyxalgo.h"
+#include "support/LAssert.h"
 #include "support/filetools.h"
 #include "support/FileInfo.h"
 #include "support/syscall.h"
 #include "support/lstrings.h"
 #include "support/path.h"
 #include "support/lyxfunctional.h"
-#include "debug.h"
-#include "lyxrc.h"
-#include "lyxtext.h"
-#include "gettext.h"
-#include "trans_mgr.h"
-#include "layout.h"
-#include "WorkArea.h"
-#include "bufferview_funcs.h"
-#include "frontends/FileDialog.h"
-#include "frontends/Dialogs.h"
-#include "frontends/Toolbar.h"
-#include "frontends/Menubar.h"
-#include "FloatList.h"
-#include "converter.h"
-#include "exporter.h"
-#include "importer.h"
-#include "FontLoader.h"
-#include "TextCache.h"
-#include "lyxfind.h"
-#include "undo_funcs.h"
+
+#include <ctime>
+#include <clocale>
+#include <cstdlib>
+#include <cctype>
+
+#include <utility> 
+#include <algorithm> 
+
 
 using std::pair;
 using std::make_pair; 
@@ -98,7 +109,6 @@ using std::find_if;
 
 extern BufferList bufferlist;
 extern LyXServer * lyxserver;
-extern int greek_kb_flag;
 extern bool selection_possible;
 extern void MenuSendto();
 
@@ -110,8 +120,6 @@ extern LyXAction lyxaction;
 // (alkis)
 extern tex_accent_struct get_accent(kb_action action);
 
-extern LyXTextClass::size_type current_layout;
-
 extern void ShowLatexLog();
 
 
@@ -150,17 +158,15 @@ MiniBufferController mb_ctrl;
 
 /* === globals =========================================================== */
 
-// Initialization of static member var
-bool LyXFunc::show_sc = true;
-
 
 LyXFunc::LyXFunc(LyXView * o)
-       : owner(o)
+       : owner(o),
+       keyseq(toplevel_keymap.get(), toplevel_keymap.get()),
+       cancel_meta_seq(toplevel_keymap.get(), toplevel_keymap.get())
 {
        meta_fake_bit = 0;
        lyx_dead_action = LFUN_NOACTION;
        lyx_calling_dead_action = LFUN_NOACTION;
-       setupLocalKeymap();
 }
 
 
@@ -199,14 +205,15 @@ void LyXFunc::handleKeyFunc(kb_action action)
 {
        char c = keyseq.getiso();
 
-       if (keyseq.length != -1) c = 0;
+       if (keyseq.length() > 1) {
+               c = 0;
+       }
        
        owner->getIntl()->getTrans()
                .deadkey(c, get_accent(action).accent, TEXT(false));
-       // Need to reset, in case the minibuffer calls these
+       // Need to clear, in case the minibuffer calls these
        // actions
-       keyseq.reset();
-       keyseq.length = 0;
+       keyseq.clear();
        // copied verbatim from do_accent_char
        owner->view()->update(TEXT(false),
               BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
@@ -235,7 +242,7 @@ void LyXFunc::processKeySym(KeySym keysym, unsigned int state)
                return;
        }
 
-       // Can we be sure that this will work for all X-Windows
+       // Can we be sure that this will work for all X Window
        // implementations? (Lgb)
        // This code snippet makes lyx ignore some keys. Perhaps
        // all of them should be explictly mentioned?
@@ -276,14 +283,15 @@ void LyXFunc::processKeySym(KeySym keysym, unsigned int state)
        // Dont remove this unless you know what you are doing.
        meta_fake_bit = 0;
                
-       if (action == 0) action = LFUN_PREFIX;
+       // can this happen now ? 
+       if (action == LFUN_NOACTION) {
+               action = LFUN_PREFIX;
+       }
 
        if (lyxerr.debugging(Debug::KEY)) {
-               string buf;
-               keyseq.print(buf);
                lyxerr << "Key [action="
                       << action << "]["
-                      << buf << "]"
+                      << keyseq.print() << "]"
                       << endl;
        }
 
@@ -291,13 +299,11 @@ void LyXFunc::processKeySym(KeySym keysym, unsigned int state)
        // why not return already here if action == -1 and
        // num_bytes == 0? (Lgb)
 
-       if (keyseq.length > 1 || keyseq.length < -1) {
-               string buf;
-               keyseq.print(buf);
-               owner->message(buf);
+       if (keyseq.length() > 1 && !keyseq.deleted()) {
+               owner->message(keyseq.print());
        }
 
-       if (action == -1) {
+       if (action == LFUN_UNKNOWN_ACTION) {
                // It is unknown, but what if we remove all
                // the modifiers? (Lgb)
                action = keyseq.addkey(keysym, 0);
@@ -307,75 +313,49 @@ void LyXFunc::processKeySym(KeySym keysym, unsigned int state)
                               << "Action now set to ["
                               << action << "]" << endl;
                }
-               if (action == -1) {
+               if (action == LFUN_UNKNOWN_ACTION) {
                        owner->message(_("Unknown function."));
                        return;
                }
        }
 
        if (action == LFUN_SELFINSERT) {
-               // This is very X dependant.
+               // This is very X dependent.
                unsigned int c = keysym;
+               string argument;
                
-               switch (c & 0x0000FF00) {
-                       // latin 1 byte 3 = 0
-               case 0x00000000: break;
-                       // latin 2 byte 3 = 1
-               case 0x00000100:
-                       // latin 3 byte 3 = 2
-               case 0x00000200:
-                       // latin 4 byte 3 = 3
-               case 0x00000300:
-                       // latin 8 byte 3 = 18 (0x12)
-               case 0x00001200:
-                       // latin 9 byte 3 = 19 (0x13)
-               case 0x00001300:
-                       c &= 0x000000FF;
-                       break;
-               default:
-                       c = 0;
-                       break;
-               }
+               c = kb_keymap::getiso(c);
+
                if (c > 0)
                        argument = static_cast<char>(c);
+               
+               dispatch(LFUN_SELFINSERT, argument);
                lyxerr[Debug::KEY] << "SelfInsert arg[`"
                                   << argument << "']" << endl;
        }
-       
-        bool tmp_sc = show_sc;
-       show_sc = false;
-       dispatch(action, argument);
-       show_sc = tmp_sc;
-       
-       //return 0;
+       else
+               verboseDispatch(action, false);
 } 
 
 
-func_status::value_type LyXFunc::getStatus(int ac) const
+FuncStatus LyXFunc::getStatus(int ac) const
 {
-       return getStatus(ac, string());
+       kb_action action;
+       string argument;
+       action = lyxaction.retrieveActionArg(ac, argument);
+       return getStatus(action, argument);
 }
 
 
-func_status::value_type LyXFunc::getStatus(int ac,
-                                          string const & not_to_use_arg) const
+FuncStatus LyXFunc::getStatus(kb_action action,
+                             string const & argument) const
 {
-       kb_action action;
-       func_status::value_type flag = func_status::OK;
-       string argument;
+       FuncStatus flag;
        Buffer * buf = owner->buffer();
        
-       if (lyxaction.isPseudoAction(ac)) 
-               action = lyxaction.retrieveActionArg(ac, argument);
-       else {
-               action = static_cast<kb_action>(ac);
-               if (!not_to_use_arg.empty())
-                       argument = not_to_use_arg; // exept here
-       }
-       
        if (action == LFUN_UNKNOWN_ACTION) {
                setErrorMessage(N_("Unknown action"));
-               return func_status::Unknown;
+               return flag.unknown(true);
        }
        
        // Check whether we need a buffer
@@ -389,14 +369,13 @@ func_status::value_type LyXFunc::getStatus(int ac,
                                                   LyXAction::ReadOnly)) {
                                // no
                                setErrorMessage(N_("Document is read-only"));
-                               flag |= func_status::Disabled;
+                               flag.disabled(true);
                        }
                } else {
                        // no
                        setErrorMessage(N_("Command not allowed with"
                                           "out any document open"));
-                       flag |= func_status::Disabled;
-                       return flag;
+                       return flag.disabled(true);
                }
        }
 
@@ -421,6 +400,11 @@ func_status::value_type LyXFunc::getStatus(int ac,
        case LFUN_SPELLCHECK:
                disable = lyxrc.isp_command == "none";
                break;
+#ifndef HAVE_LIBAIKSAURUS
+       case LFUN_THESAURUS_ENTRY:
+               disable = true;
+               break;
+#endif
        case LFUN_RUNCHKTEX:
                disable = lyxrc.chktex_command == "none";
                break;
@@ -439,7 +423,8 @@ func_status::value_type LyXFunc::getStatus(int ac,
        case LFUN_TABULAR_FEATURE:
                disable = true;
                if (owner->view()->theLockingInset()) {
-                       func_status::value_type ret = func_status::Disabled;
+                       FuncStatus ret;
+                       //ret.disabled(true);
                        if (owner->view()->theLockingInset()->lyxCode() == Inset::TABULAR_CODE) {
                                ret = static_cast<InsetTabular *>
                                        (owner->view()->theLockingInset())->
@@ -454,13 +439,11 @@ func_status::value_type LyXFunc::getStatus(int ac,
                        disable = false;
                } else {
                        static InsetTabular inset(*owner->buffer(), 1, 1);
-                       func_status::value_type ret;
+                       FuncStatus ret;
 
                        disable = true;
                        ret = inset.getStatus(argument);
-                       if ((ret & func_status::ToggleOn) ||
-                           (ret & func_status::ToggleOff))
-                               flag |= func_status::ToggleOff;
+                       if (ret.onoff(true) || ret.onoff(false)) flag.setOnOff(false);
                }
                break;
 
@@ -502,17 +485,14 @@ func_status::value_type LyXFunc::getStatus(int ac,
                                break;
                        }
                        if (argument.empty()) {
-                               flag = func_status::OK;
+                               flag.clear();
                                break;
                        }
                        if (!contains("tcb", argument[0])) {
                                disable = true;
                                break;
                        }
-                       if (argument[0] == align) 
-                               flag |= func_status::ToggleOn;
-                       else
-                               flag |= func_status::ToggleOff;
+                       flag.setOnOff(argument[0] == align);
                } else
                        disable = true;
                break;
@@ -528,17 +508,14 @@ func_status::value_type LyXFunc::getStatus(int ac,
                                break;
                        }
                        if (argument.empty()) {
-                               flag = func_status::OK;
+                               flag.clear();
                                break;
                        }
                        if (!contains("lcr", argument[0])) {
                                disable = true;
                                break;
                        }
-                       if (argument[0] == align) 
-                               flag |= func_status::ToggleOn;
-                       else
-                               flag |= func_status::ToggleOff;
+                       flag.setOnOff(argument[0] == align);
                } else
                        disable = true;
                break;
@@ -546,25 +523,18 @@ func_status::value_type LyXFunc::getStatus(int ac,
        case LFUN_MATH_MUTATE: {
                Inset * tli = owner->view()->theLockingInset();
                if (tli && (tli->lyxCode() == Inset::MATH_CODE)) {
-                       MathInsetTypes type = mathcursor->par()->GetType();
-                       func_status::value_type box = func_status::ToggleOff;
+                       MathInsetTypes type = mathcursor->formula()->getType();
                        if (argument == "inline") {
-                               if (type == LM_OT_SIMPLE)
-                                       box = func_status::ToggleOn;
+                               flag.setOnOff(type == LM_OT_SIMPLE);
                        } else if (argument == "display") {
-                               if (type == LM_OT_EQUATION)
-                                       box = func_status::ToggleOn;
+                               flag.setOnOff(type == LM_OT_EQUATION);
                        } else if (argument == "eqnarray") {
-                               if (type == LM_OT_EQNARRAY)
-                                       box = func_status::ToggleOn;
+                               flag.setOnOff(type == LM_OT_EQNARRAY);
                        } else if (argument == "align") {
-                               if (type == LM_OT_ALIGN)
-                                       box = func_status::ToggleOn;
+                               flag.setOnOff(type == LM_OT_ALIGN);
                        } else {
-                               box = func_status::OK;
                                disable = true;
                        }
-                       flag |= box;
                } else
                        disable = true;
                break;
@@ -572,6 +542,7 @@ func_status::value_type LyXFunc::getStatus(int ac,
 
        // we just need to be in math mode to enable that
        case LFUN_MATH_SIZE: 
+       case LFUN_MATH_SPACE:
        case LFUN_MATH_LIMITS: 
        case LFUN_MATH_NONUMBER: 
        case LFUN_MATH_NUMBER:
@@ -631,10 +602,10 @@ func_status::value_type LyXFunc::getStatus(int ac,
        case LFUN_INSET_LIST:
                code = Inset::LIST_CODE;
                break;
-#endif
        case LFUN_INSET_THEOREM:
                code = Inset::THEOREM_CODE;
                break;
+#endif
        case LFUN_INSET_CAPTION:
                code = Inset::CAPTION_CODE;
                break;
@@ -687,7 +658,6 @@ func_status::value_type LyXFunc::getStatus(int ac,
        case LFUN_LDOTS:
        case LFUN_END_OF_SENTENCE:
        case LFUN_PROTECTEDSPACE:
-       case LFUN_BREAKLINE:
                code = Inset::SPECIALCHAR_CODE;
                break;
        default:
@@ -700,41 +670,75 @@ func_status::value_type LyXFunc::getStatus(int ac,
        }
 
        if (disable)
-               flag |= func_status::Disabled;
+                       flag.disabled(true);
        
-       // the font related functions
-       func_status::value_type box = func_status::ToggleOff;
-       LyXFont const & font =
-               TEXT(false)->real_current_font;
+       // A few general toggles
        switch (action) {
-       case LFUN_EMPH:
-               if (font.emph() == LyXFont::ON)
-                       box = func_status::ToggleOn;
-               break;
-       case LFUN_NOUN:
-               if (font.noun() == LyXFont::ON)
-                       box = func_status::ToggleOn;
-               break;
-       case LFUN_BOLD:
-               if (font.series() == LyXFont::BOLD_SERIES)
-                       box = func_status::ToggleOn;
-               break;
-#ifndef NO_LATEX
-       case LFUN_TEX:
-               if (font.latex() == LyXFont::ON)
-                       box = func_status::ToggleOn;
-               break;
-#endif
        case LFUN_READ_ONLY_TOGGLE:
-               if (buf->isReadonly())
-                       box = func_status::ToggleOn;
+               flag.setOnOff(buf->isReadonly());
+               break;
+       case LFUN_APPENDIX:
+               flag.setOnOff(TEXT(false)->cursor.par()->params().startOfAppendix());
                break;
        default:
-               box = func_status::OK;
                break;
        }
-       flag |= box;
 
+       // the font related toggles
+       if (!mathcursor) {
+               LyXFont const & font = TEXT(false)->real_current_font;
+               switch (action) {
+               case LFUN_EMPH:
+                       flag.setOnOff(font.emph() == LyXFont::ON);
+                       break;
+               case LFUN_NOUN:
+                       flag.setOnOff(font.noun() == LyXFont::ON);
+                       break;
+               case LFUN_BOLD:
+                       flag.setOnOff(font.series() == LyXFont::BOLD_SERIES);
+                       break;
+               case LFUN_SANS:
+                       flag.setOnOff(font.family() == LyXFont::SANS_FAMILY);
+                       break;
+               case LFUN_ROMAN:
+                       flag.setOnOff(font.family() == LyXFont::ROMAN_FAMILY);
+                       break;
+               case LFUN_CODE:
+                       flag.setOnOff(font.family() == LyXFont::TYPEWRITER_FAMILY);
+                       break;
+               default:
+                       break;
+               }
+       }
+       else {
+               MathTextCodes tc = mathcursor->getLastCode();
+               switch (action) {
+               case LFUN_BOLD:
+                       flag.setOnOff(tc == LM_TC_BF);
+                       break;
+               case LFUN_SANS:
+                       flag.setOnOff(tc == LM_TC_SF);
+                       break;
+               case LFUN_EMPH:
+                       flag.setOnOff(tc == LM_TC_CAL);
+                       break;
+               case LFUN_ROMAN:
+                       flag.setOnOff(tc == LM_TC_RM);
+                       break;
+               case LFUN_CODE:
+                       flag.setOnOff(tc == LM_TC_TT);
+                       break;
+               case LFUN_NOUN:
+                       flag.setOnOff(tc == LM_TC_BB);
+                       break;
+               case LFUN_DEFAULT:
+                       flag.setOnOff(tc == LM_TC_VAR);
+                       break;
+               default:
+                       break;
+               }
+       }
+       
        return flag;
 }
 
@@ -742,48 +746,110 @@ func_status::value_type LyXFunc::getStatus(int ac,
 // temporary dispatch method
 void LyXFunc::miniDispatch(string const & s) 
 {
-       if (!s.empty()) {
-               dispatch(s);
+       string s2(frontStrip(strip(s))); 
+       if (!s2.empty()) {
+               verboseDispatch(s2, true);
        }
 }
 
 
-string const LyXFunc::dispatch(string const & s
+void const LyXFunc::verboseDispatch(string const & s, bool show_sc
 {
        // Split command string into command and argument
        string cmd;
        string line = frontStrip(s);
        string const arg = strip(frontStrip(split(line, cmd, ' ')));
 
-       return dispatch(lyxaction.LookupFunc(cmd), arg);
+       int action = lyxaction.LookupFunc(cmd);
+       if (action == LFUN_UNKNOWN_ACTION) {
+               string const msg = string(_("Unknown function ("))
+                       + cmd + ")";
+               owner->message(msg);
+       } else {
+               verboseDispatch(action, show_sc);
+       } 
 }
 
 
-string const LyXFunc::dispatch(int ac,
-                              string const & do_not_use_this_arg)
+void const LyXFunc::verboseDispatch(int ac, bool show_sc) 
 {
-       lyxerr[Debug::ACTION] << "LyXFunc::Dispatch: action[" << ac
-                             <<"] arg[" << do_not_use_this_arg << "]" << endl;
-       
        string argument;
        kb_action action;
         
-        // we have not done anything wrong yet.
-        errorstat = false;
-       dispatch_buffer.erase();
+       // get the real action and argument
+       action = lyxaction.retrieveActionArg(ac, argument);
+
+       verboseDispatch(action, argument, show_sc);
+}
+
+
+
+void const LyXFunc::verboseDispatch(kb_action action,
+                                   string const & argument, bool show_sc)
+{
+       string res = dispatch(action, argument);
+
+       commandshortcut.erase();
        
-       // if action is a pseudo-action, we need the real action
-       if (lyxaction.isPseudoAction(ac)) {
-               string tmparg;
-               action = static_cast<kb_action>
-                       (lyxaction.retrieveActionArg(ac, tmparg));
-               if (!tmparg.empty())
-                       argument = tmparg;
+       if (lyxrc.display_shortcuts && show_sc) {
+               if (action != LFUN_SELFINSERT) {
+                       // Put name of command and list of shortcuts
+                       // for it in minibuffer
+                       string comname = lyxaction.getActionName(action);
+                       
+                       int pseudoaction = action;
+                       bool argsadded = false;
+                       
+                       if (!argument.empty()) {
+                               // the pseudoaction is useful for the bindings
+                               pseudoaction = 
+                                       lyxaction.searchActionArg(action,
+                                                                 argument);
+
+                               if (pseudoaction == LFUN_UNKNOWN_ACTION) {
+                                       pseudoaction = action;
+                               } else {
+                                       comname += " " + argument;
+                                       argsadded = true;
+                               }
+                       }
+
+                       string const shortcuts =
+                               toplevel_keymap->findbinding(pseudoaction);
+
+                       if (!shortcuts.empty()) {
+                               comname += ": " + shortcuts;
+                       } else if (!argsadded && !argument.empty()) {
+                               comname += " " + argument;
+                       }
+
+                       if (!comname.empty()) {
+                               comname = strip(comname);
+                               commandshortcut = "(" + comname + ')';
+                       }
+               }
+        }
+
+       if (res.empty()) {
+               if (!commandshortcut.empty()) {
+                       owner->getMiniBuffer()->addSet(commandshortcut);
+               }
        } else {
-               action = static_cast<kb_action>(ac);
-               if (!do_not_use_this_arg.empty())
-                       argument = do_not_use_this_arg; // except here
+               owner->getMiniBuffer()->addSet(' ' + commandshortcut);
        }
+}
+
+
+string const LyXFunc::dispatch(kb_action action, string argument)
+{
+       lyxerr[Debug::ACTION] << "LyXFunc::Dispatch: action[" << action
+                             <<"] arg[" << argument << "]" << endl;
+
+        // we have not done anything wrong yet.
+        errorstat = false;
+       dispatch_buffer.erase();
 
 #ifdef NEW_DISPATCHER
        // We try do call the most specific dispatcher first:
@@ -791,79 +857,72 @@ string const LyXFunc::dispatch(int ac,
        //  2. the bufferview's dispatch
        //  3. the lyxview's dispatch
 #endif
-       
+
        selection_possible = false;
-       
+
        if (owner->view()->available())
                owner->view()->hideCursor();
 
        // We cannot use this function here
-       if (getStatus(ac, do_not_use_this_arg) & func_status::Disabled)
+       if (getStatus(action, argument).disabled()) {
+               lyxerr[Debug::ACTION] << "LyXFunc::Dispatch: "
+                      << lyxaction.getActionName(action)
+                      << " [" << action << "] is disabled at this location"
+                      << endl;
                goto exit_with_message;
-
+       }
 
        if (owner->view()->available() && owner->view()->theLockingInset()) {
                UpdatableInset::RESULT result;
                if ((action > 1) || ((action == LFUN_UNKNOWN_ACTION) &&
-                                    (keyseq.length >= -1)))
+                                    (!keyseq.deleted())))
                {
-                       if ((action==LFUN_UNKNOWN_ACTION) && argument.empty()){
+                       if ((action == LFUN_UNKNOWN_ACTION)
+                           && argument.empty()){
                                argument = keyseq.getiso();
                        }
                        // Undo/Redo is a bit tricky for insets.
                        if (action == LFUN_UNDO) {
-#ifdef RETHINK_THIS_FOR_NOW_WE_LEAVE_ALL_UNLOCKED
-                               int slx;
-                               int sly;
-                               UpdatableInset * inset = 
-                                       owner->view()->theLockingInset()->getLockingInset();
-                               int inset_id = inset->id();
-                               inset->getCursorPos(owner->view(), slx, sly);
-                               owner->view()->unlockInset(inset);
-#else
-                               owner->view()->unlockInset(owner->view()->theLockingInset());
-#endif
                                owner->view()->menuUndo();
-#ifdef RETHINK_THIS_FOR_NOW_WE_LEAVE_ALL_UNLOCKED
-#if 0
-                               if (TEXT()->cursor.par()->
-                                   isInset(TEXT()->cursor.pos())) {
-                                       inset = static_cast<UpdatableInset*>(
-                                               TEXT()->cursor.par()->
-                                               getInset(TEXT()->
-                                                        cursor.pos()));
-                               } else {
-                                       inset = 0;
-                               }
-#else
-                               inset = static_cast<UpdatableInset *>(owner->view()->buffer()->getInsetFromID(inset_id));
-#endif
-                               if (inset)
-                                       inset->edit(owner->view(),slx,sly,0);
-#endif
                                return string();
                        } else if (action == LFUN_REDO) {
-                               int slx;
-                               int sly;
-                               UpdatableInset * inset = owner->view()->
-                                       theLockingInset();
-                               inset->getCursorPos(owner->view(), slx, sly);
-                               owner->view()->unlockInset(inset);
                                owner->view()->menuRedo();
-                               inset = static_cast<UpdatableInset*>(
-                                       TEXT()->cursor.par()->
-                                       getInset(TEXT()->
-                                                cursor.pos()));
-                               if (inset)
-                                       inset->edit(owner->view(),slx,sly,0); 
                                return string();
                        } else if (((result=owner->view()->theLockingInset()->
-                                  localDispatch(owner->view(), action,
-                                                argument)) ==
-                                  UpdatableInset::DISPATCHED) ||
-                                  (result == UpdatableInset::DISPATCHED_NOUPDATE))
+                                    localDispatch(owner->view(), action, argument)) ==
+                                   UpdatableInset::DISPATCHED) ||
+                                  (result == UpdatableInset::DISPATCHED_NOUPDATE))
                                return string();
-                       else {
+                       else if (result == UpdatableInset::FINISHED) {
+                                       if (TEXT()->cursor.par()->isRightToLeftPar(owner->buffer()->params)) {
+                                               TEXT()->cursorRight(owner->view());
+                                               moveCursorUpdate(true, false);
+                                               owner->showState();
+                                       }
+                                       return string();
+                       } else if (result == UpdatableInset::FINISHED_RIGHT) {
+                               if (!TEXT()->cursor.par()->isRightToLeftPar(owner->buffer()->params)) {
+                                       TEXT()->cursorRight(owner->view());
+                                       moveCursorUpdate(true, false);
+                                       owner->showState();
+                               }
+                               return string();
+                       } else if (result == UpdatableInset::FINISHED_UP) {
+                               if (TEXT()->cursor.row()->previous()) {
+                                       TEXT()->cursorUp(owner->view());
+                                       moveCursorUpdate(true, false);
+                                       owner->showState();
+                               }
+                               return string();
+                       } else if (result == UpdatableInset::FINISHED_DOWN) {
+                               if (TEXT()->cursor.row()->next())
+                                       TEXT()->cursorDown(owner->view());
+                               else
+                                       TEXT()->cursorRight(owner->view());
+                               moveCursorUpdate(true, false);
+                               owner->showState();
+                               return string();
+                       } else {
                                //setMessage(N_("Text mode"));
                                switch (action) {
                                case LFUN_UNKNOWN_ACTION:
@@ -902,14 +961,11 @@ string const LyXFunc::dispatch(int ac,
                }
        }
 
-       lyx::Assert(action != LFUN_SELECT_FILE_SYNC);
-
        switch (action) {
                
        case LFUN_ESCAPE:
        {
                if (!owner->view()->available()) break;
-               
                // this function should be used always [asierra060396]
                UpdatableInset * tli =
                        owner->view()->theLockingInset();
@@ -926,6 +982,7 @@ string const LyXFunc::dispatch(int ac,
                                                        lock,
                                                        true);
                        }
+                       finishUndo();
                }
        }
        break;
@@ -956,9 +1013,7 @@ string const LyXFunc::dispatch(int ac,
                        owner->view()->update(TEXT(),
                                              BufferView::SELECT|BufferView::FITCUR);
                }
-               string buf;
-               keyseq.print(buf, true);
-               owner->message(buf);
+               owner->message(keyseq.print());
        }
        break;
 
@@ -986,9 +1041,7 @@ string const LyXFunc::dispatch(int ac,
        case LFUN_META_FAKE:                                 // RVDK_PATCH_5
        {
                meta_fake_bit = Mod1Mask;
-               string buf;
-               keyseq.print(buf, true);
-               setMessage(buf); // RVDK_PATCH_5
+               setMessage(keyseq.print());
        }
        break;  
 
@@ -1124,7 +1177,7 @@ string const LyXFunc::dispatch(int ac,
        case LFUN_REMOVEERRORS:
                if (owner->view()->removeAutoInsets()) {
                        owner->view()->redraw();
-                       owner->view()->fitCursor(TEXT());
+                       owner->view()->fitCursor();
                }
                break;
 
@@ -1144,14 +1197,6 @@ string const LyXFunc::dispatch(int ac,
                owner->getDialogs()->setUserFreeFont();
                break;
 
-#ifndef NO_LATEX
-       case LFUN_TEX:
-               Tex(owner->view());
-               owner->view()->setState();
-               owner->showState();
-               break;
-#endif
-               
        case LFUN_RECONFIGURE:
                Reconfigure(owner->view());
                break;
@@ -1169,16 +1214,16 @@ string const LyXFunc::dispatch(int ac,
                break;
 #else
 #ifdef WITH_WARNINGS
-#warning Find another implementation here (or another lyxfunc)!
+//#warning Find another implementation here (or another lyxfunc)!
 #endif
 #endif
        case LFUN_HELP_ABOUTLYX:
                owner->getDialogs()->showAboutlyx();
                break;
 
-       case LFUN_HELP_COPYRIGHT:
-       case LFUN_HELP_CREDITS:
-               
+       case LFUN_HELP_TEXINFO:
+               owner->getDialogs()->showTexinfo();
+               break;
 
         case LFUN_HELP_OPEN:
        {
@@ -1204,8 +1249,6 @@ string const LyXFunc::dispatch(int ac,
                break;
         }
 
-       case LFUN_HELP_VERSION:
-       
                // --- version control -------------------------------
        case LFUN_VC_REGISTER:
        {
@@ -1271,26 +1314,6 @@ string const LyXFunc::dispatch(int ac,
                owner->getDialogs()->showLogFile();
                break;
                
-       case LFUN_LAYOUTNO:
-       {
-               lyxerr[Debug::INFO] << "LFUN_LAYOUTNO: (arg) " << argument << endl;
-               int sel = strToInt(argument);
-               lyxerr[Debug::INFO] << "LFUN_LAYOUTNO: (sel) "<< sel << endl;
-               
-               // Should this give a setMessage instead?
-               if (sel == 0) 
-                       return string(); // illegal argument
-
-               --sel; // sel 1..., but layout 0...
-
-               // Pretend we got the name instead.
-               dispatch(int(LFUN_LAYOUT), 
-                        textclasslist.NameOfLayout(owner->view()
-                                                   ->buffer()->params.textclass,
-                                                   sel));
-               return string();
-       }
-               
        case LFUN_LAYOUT_DOCUMENT:
                owner->getDialogs()->showDocument();
                break;
@@ -1322,10 +1345,6 @@ string const LyXFunc::dispatch(int ac,
                owner->getDialogs()->showPreamble();
                break;
                
-       case LFUN_LAYOUT_SAVE_DEFAULT:
-               MenuLayoutSave(owner->view());
-               break;
-               
        case LFUN_DROP_LAYOUTS_CHOICE:
                owner->getToolbar()->openLayoutList();
                break;
@@ -1351,25 +1370,17 @@ string const LyXFunc::dispatch(int ac,
                
        case LFUN_NOTIFY:
        {
-               string buf;
-               keyseq.print(buf);
-               dispatch_buffer = buf;
+               dispatch_buffer = keyseq.print();
                lyxserver->notifyClient(dispatch_buffer);
        }
        break;
 
        case LFUN_GOTOFILEROW:
        {
-#if 0
-               char file_name[100];
-               int row;
-               ::sscanf(argument.c_str(), " %s %d", file_name, &row);
-#else
                string file_name;
                int row;
                istringstream istr(argument.c_str());
                istr >> file_name >> row;
-#endif
                // Must replace extension of the file to be .lyx and get full path
                string const s(ChangeExtension(file_name, ".lyx"));
 
@@ -1399,10 +1410,13 @@ string const LyXFunc::dispatch(int ac,
                        lyxerr[Debug::INFO] << "No matching paragraph found! ["
                                            << id << "]" << std::endl;
                        break;
+               } else {
+                       lyxerr << "Paragraph " << par->id()
+                              << " found." << endl;
                }
 
                // Set the cursor
-               TEXT()->setCursor(owner->view(), par, 0);
+               owner->view()->text->setCursor(owner->view(), par, 0);
                owner->view()->setState();
                owner->showState();
 
@@ -1452,30 +1466,6 @@ string const LyXFunc::dispatch(int ac,
        // --- insert characters ----------------------------------------
 
        // ---  Mathed stuff. If we are here, there is no locked inset yet.
-       
-       // Greek mode     
-       case LFUN_GREEK:
-       {
-               if (!greek_kb_flag) {
-                       greek_kb_flag = 1;
-                       setMessage(N_("Math greek mode on"));
-               } else
-                       greek_kb_flag = 0;
-       }  
-       break;
-      
-       // Greek keyboard      
-       case LFUN_GREEK_TOGGLE:
-       {
-               greek_kb_flag = greek_kb_flag ? 0 : 2;
-               if (greek_kb_flag) {
-                       setMessage(N_("Math greek keyboard on"));
-               } else {
-                       setMessage(N_("Math greek keyboard off"));
-               }
-       }
-       break;
-       
        case LFUN_MATH_EXTERN:
        case LFUN_MATH_NUMBER:
        case LFUN_MATH_NONUMBER:
@@ -1485,6 +1475,14 @@ string const LyXFunc::dispatch(int ac,
        }
        break;
 
+       // passthrough hat and underscore outside mathed:
+       case LFUN_SUBSCRIPT:
+               dispatch(LFUN_SELFINSERT, "_");
+               break;
+       case LFUN_SUPERSCRIPT:
+               dispatch(LFUN_SELFINSERT, "^");
+               break;
+
        case LFUN_MATH_PANEL:
                owner->getDialogs()->showMathPanel();
                break;
@@ -1551,7 +1549,7 @@ string const LyXFunc::dispatch(int ac,
                while (argument.find(';') != string::npos) {
                        string first;
                        argument = split(argument, first, ';');
-                       dispatch(first);
+                       verboseDispatch(first, false);
                }
        }
        break;
@@ -1630,65 +1628,10 @@ string const LyXFunc::dispatch(int ac,
 
 exit_with_message:
 
-       commandshortcut.erase();
-       
-       if (lyxrc.display_shortcuts && show_sc) {
-               if (action != LFUN_SELFINSERT) {
-                       // Put name of command and list of shortcuts
-                       // for it in minibuffer
-                       string comname = lyxaction.getActionName(action);
-
-                       int pseudoaction = action;
-                       bool argsadded = false;
-
-                       if (!argument.empty()) {
-                               // If we have the command with argument, 
-                               // this is better
-                               pseudoaction = 
-                                       lyxaction.searchActionArg(action,
-                                                                 argument);
-
-                               if (pseudoaction == -1) {
-                                       pseudoaction = action;
-                               } else {
-                                       comname += " " + argument;
-                                       argsadded = true;
-                               }
-                       }
-
-                       string const shortcuts =
-                               toplevel_keymap->findbinding(pseudoaction);
-
-                       if (!shortcuts.empty()) {
-                               comname += ": " + shortcuts;
-                       } else if (!argsadded) {
-                               comname += " " + argument;
-                       }
-
-                       if (!comname.empty()) {
-                               comname = strip(comname);
-                               commandshortcut = "(" + comname + ')';
-
-                               // Here we could even add a small pause,
-                               // to annoy the user and make him learn
-                               // the shortcuts.
-                               // No! That will just annoy, not teach
-                               // anything. The user will read the messages
-                               // if they are interested. (Asger)
-                       }
-               }
-        }
-
        string const res = getMessage();
 
-       if (res.empty()) {
-               if (!commandshortcut.empty()) {
-                       owner->getMiniBuffer()->addSet(commandshortcut);
-               }
-       } else {
-               string const msg(_(res) + ' ' + commandshortcut);
-               owner->message(msg);
-       }
+       if (!res.empty())
+               owner->message(_(res));
 
        return res;
 }
@@ -1708,7 +1651,7 @@ void LyXFunc::menuNew(bool fromTemplate)
        if (owner->view()->available()) {
                string const trypath = owner->buffer()->filepath;
                // If directory is writeable, use this as default.
-               if (IsDirWriteable(trypath) == 1)
+               if (IsDirWriteable(trypath))
                        initpath = trypath;
        }
 
@@ -1742,7 +1685,7 @@ void LyXFunc::menuNew(bool fromTemplate)
 
                // Check if the document already is open
                if (bufferlist.exists(s)) {
-                       switch (AskConfirmation(_("Document is already open:"),
+                       switch (Alert::askConfirmation(_("Document is already open:"),
                                                MakeDisplayPath(s, 50),
                                                _("Do you want to close that document now?\n"
                                                  "('No' will just switch to the open version)")))
@@ -1763,7 +1706,7 @@ void LyXFunc::menuNew(bool fromTemplate)
                // Check whether the file already exists
                FileInfo fi(s);
                if (fi.readable() &&
-                   AskQuestion(_("File already exists:"), 
+                   Alert::askQuestion(_("File already exists:"), 
                                MakeDisplayPath(s, 50),
                                _("Do you want to open the document?"))) {
                                // loads document
@@ -1834,7 +1777,7 @@ void LyXFunc::open(string const & fname)
        if (owner->view()->available()) {
                string const trypath = owner->buffer()->filepath;
                // If directory is writeable, use this as default.
-               if (IsDirWriteable(trypath) == 1)
+               if (IsDirWriteable(trypath))
                        initpath = trypath;
        }
 
@@ -1867,8 +1810,14 @@ void LyXFunc::open(string const & fname)
 
        // get absolute path of file and add ".lyx" to the filename if
        // necessary
-       filename = FileSearch(string(), filename, "lyx");
+       string const fullpath = FileSearch(string(), filename, "lyx");
+       if (fullpath.empty()) {
+               Alert::alert(_("Error"), _("Could not find file"), filename);
+               return;
+       }
 
+       filename = fullpath;
        // loads document
        string const disp_fn(MakeDisplayPath(filename));
 
@@ -1885,7 +1834,7 @@ void LyXFunc::open(string const & fname)
                owner->message(str.str().c_str());
        } else {
                ostringstream str;
-               str << _("Could not open docuent") << ' ' << disp_fn;
+               str << _("Could not open document") << ' ' << disp_fn;
                owner->message(str.str().c_str());
        }
 }
@@ -1905,11 +1854,11 @@ void LyXFunc::doImport(string const & argument)
                if (owner->view()->available()) {
                        string const trypath = owner->buffer()->filepath;
                        // If directory is writeable, use this as default.
-                       if (IsDirWriteable(trypath) == 1)
+                       if (IsDirWriteable(trypath))
                                initpath = trypath;
                }
 
-               string const text = _("Select ") + formats.PrettyName(format)
+               string const text = _("Select ") + formats.prettyName(format)
                        + _(" file to import");
 
                FileDialog fileDlg(owner, text, 
@@ -1919,9 +1868,9 @@ void LyXFunc::doImport(string const & argument)
                        make_pair(string(_("Examples")),
                                  string(AddPath(system_lyxdir, "examples"))));
                        
-               string const extension = "*." + formats.Extension(format)
-                       + "| " + formats.PrettyName(format)
-                       + " (*." + formats.Extension(format) + ")";
+               string const extension = "*." + formats.extension(format)
+                       + "| " + formats.prettyName(format)
+                       + " (*." + formats.extension(format) + ")";
 
                FileDialog::Result result = fileDlg.Select(initpath,
                                                           extension);
@@ -1947,7 +1896,7 @@ void LyXFunc::doImport(string const & argument)
 
        // Check if the document already is open
        if (bufferlist.exists(lyxfile)) {
-               switch (AskConfirmation(_("Document is already open:"), 
+               switch (Alert::askConfirmation(_("Document is already open:"), 
                                        MakeDisplayPath(lyxfile, 50),
                                        _("Do you want to close that document now?\n"
                                          "('No' will just switch to the open version)")))
@@ -1968,7 +1917,7 @@ void LyXFunc::doImport(string const & argument)
 
        // Check if a LyX document by the same root exists in filesystem
        FileInfo const f(lyxfile, true);
-       if (f.exist() && !AskQuestion(_("A document by the name"), 
+       if (f.exist() && !Alert::askQuestion(_("A document by the name"), 
                                      MakeDisplayPath(lyxfile),
                                      _("already exists. Overwrite?"))) {
                owner->message(_("Canceled"));
@@ -2028,14 +1977,15 @@ void LyXFunc::initMiniBuffer()
        
        // When meta-fake key is pressed, show the key sequence so far + "M-".
        if (wasMetaKey()) {
-               keyseqStr();
+               text = keyseq.print();
                text += "M-";
        }
 
        // Else, when a non-complete key sequence is pressed,
        // show the available options.
-       else if (keyseqUncomplete()) 
-               text = keyseqOptions();
+       if (keyseq.length() > 0 && !keyseq.deleted()) {
+               text = keyseq.printOptions();
+       }
    
        // Else, show the buffer state.
        else if (owner->view()->available()) {
@@ -2049,7 +1999,7 @@ void LyXFunc::initMiniBuffer()
                text += nicename;
                if (tmpbuf->lyxvc.inUse()) {
                        text += " [";
-                       text += tmpbuf->lyxvc.version();
+                       text += tmpbuf->lyxvc.versionString();
                        text += ' ';
                        text += tmpbuf->lyxvc.locker();
                        if (tmpbuf->isReadonly())