]> git.lyx.org Git - lyx.git/blobdiff - src/lyxfunc.C
John's Layout Tabular UI improvements and Martins fixes to clearing the
[lyx.git] / src / lyxfunc.C
index 8253912947af2f37060692733803c597a3da0b41..0456ef8bd14433fc78f9389885c5c0a5e0c7eacd 100644 (file)
@@ -348,7 +348,6 @@ func_status::value_type LyXFunc::getStatus(int ac) const
        return getStatus(ac, string());
 }
 
-
 func_status::value_type LyXFunc::getStatus(int ac,
                                           string const & not_to_use_arg) const
 {
@@ -506,10 +505,7 @@ func_status::value_type LyXFunc::getStatus(int ac,
                                disable = true;
                                break;
                        }
-                       if (argument[0] == align) 
-                               flag |= func_status::ToggleOn;
-                       else
-                               flag |= func_status::ToggleOff;
+                       func_status::toggle(flag, argument[0] == align);
                } else
                        disable = true;
                break;
@@ -532,10 +528,7 @@ func_status::value_type LyXFunc::getStatus(int ac,
                                disable = true;
                                break;
                        }
-                       if (argument[0] == align) 
-                               flag |= func_status::ToggleOn;
-                       else
-                               flag |= func_status::ToggleOff;
+                       func_status::toggle(flag, argument[0] == align);
                } else
                        disable = true;
                break;
@@ -544,24 +537,17 @@ func_status::value_type LyXFunc::getStatus(int ac,
                Inset * tli = owner->view()->theLockingInset();
                if (tli && (tli->lyxCode() == Inset::MATH_CODE)) {
                        MathInsetTypes type = mathcursor->formula()->getType();
-                       func_status::value_type box = func_status::ToggleOff;
                        if (argument == "inline") {
-                               if (type == LM_OT_SIMPLE)
-                                       box = func_status::ToggleOn;
+                               func_status::toggle(flag, type == LM_OT_SIMPLE);
                        } else if (argument == "display") {
-                               if (type == LM_OT_EQUATION)
-                                       box = func_status::ToggleOn;
+                               func_status::toggle(flag, type == LM_OT_EQUATION);
                        } else if (argument == "eqnarray") {
-                               if (type == LM_OT_EQNARRAY)
-                                       box = func_status::ToggleOn;
+                               func_status::toggle(flag, type == LM_OT_EQNARRAY);
                        } else if (argument == "align") {
-                               if (type == LM_OT_ALIGN)
-                                       box = func_status::ToggleOn;
+                               func_status::toggle(flag, type == LM_OT_ALIGN);
                        } else {
-                               box = func_status::OK;
                                disable = true;
                        }
-                       flag |= box;
                } else
                        disable = true;
                break;
@@ -573,8 +559,6 @@ func_status::value_type LyXFunc::getStatus(int ac,
        case LFUN_MATH_LIMITS: 
        case LFUN_MATH_NONUMBER: 
        case LFUN_MATH_NUMBER:
-       case LFUN_SUBSCRIPT:
-       case LFUN_SUPERSCRIPT:
                disable = !mathcursor;
                break;
 
@@ -701,37 +685,75 @@ func_status::value_type LyXFunc::getStatus(int ac,
        if (disable)
                flag |= func_status::Disabled;
        
-       // the font related functions (and a few others)
-       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;
        case LFUN_READ_ONLY_TOGGLE:
-               if (buf->isReadonly())
-                       box = func_status::ToggleOn;
+               func_status::toggle(flag, buf->isReadonly());
                break;
        case LFUN_APPENDIX:
-               if (TEXT(false)->cursor.par()->params().startOfAppendix())
-                       box = func_status::ToggleOn;
+               func_status::toggle(flag, 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:
+                       func_status::toggle(flag, font.emph() == LyXFont::ON);
+                       break;
+               case LFUN_NOUN:
+                       func_status::toggle(flag, font.noun() == LyXFont::ON);
+                       break;
+               case LFUN_BOLD:
+                       func_status::toggle(flag, font.series() == LyXFont::BOLD_SERIES);
+                       break;
+               case LFUN_SANS:
+                       func_status::toggle(flag, font.family() == LyXFont::SANS_FAMILY);
+                       break;
+               case LFUN_ROMAN:
+                       func_status::toggle(flag, font.family() == LyXFont::ROMAN_FAMILY);
+                       break;
+               case LFUN_CODE:
+                       func_status::toggle(flag, font.family() == LyXFont::TYPEWRITER_FAMILY);
+                       break;
+               default:
+                       break;
+               }
+       }
+#if 0
+       else {
+               MathTextCodes tc = mathcursor->getLastCode();
+               switch (action) {
+               case LFUN_BOLD:
+                       func_status::toggle(flag, tc == LM_TC_BF);
+                       break;
+               case LFUN_SANS:
+                       func_status::toggle(flag, tc == LM_TC_SF);
+                       break;
+               case LFUN_EMPH:
+                       func_status::toggle(flag, tc == LM_TC_CAL);
+                       break;
+               case LFUN_ROMAN:
+                       func_status::toggle(flag, tc == LM_TC_RM);
+                       break;
+               case LFUN_CODE:
+                       func_status::toggle(flag, tc == LM_TC_TT);
+                       break;
+               case LFUN_NOUN:
+                       func_status::toggle(flag, tc == LM_TC_BB);
+                       break;
+               case LFUN_DEFAULT:
+                       func_status::toggle(flag, tc == LM_TC_VAR);
+                       break;
+               default:
+                       break;
+               }
+       }
+       #endif
+       
        return flag;
 }
 
@@ -739,8 +761,10 @@ 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()) {
+               dispatch(s2);
        }
 }
 
@@ -752,7 +776,16 @@ string const LyXFunc::dispatch(string const & s)
        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);
+               return string();
+       } else {
+               return dispatch(action, arg);
+       } 
 }
 
 
@@ -782,6 +815,7 @@ string const LyXFunc::dispatch(int ac,
                        argument = do_not_use_this_arg; // except here
        }
 
 #ifdef NEW_DISPATCHER
        // We try do call the most specific dispatcher first:
        //  1. the lockinginset's dispatch
@@ -896,7 +930,6 @@ string const LyXFunc::dispatch(int ac,
        case LFUN_ESCAPE:
        {
                if (!owner->view()->available()) break;
-               
                // this function should be used always [asierra060396]
                UpdatableInset * tli =
                        owner->view()->theLockingInset();
@@ -913,6 +946,7 @@ string const LyXFunc::dispatch(int ac,
                                                        lock,
                                                        true);
                        }
+                       finishUndo();
                }
        }
        break;
@@ -1400,13 +1434,19 @@ string const LyXFunc::dispatch(int ac,
        case LFUN_MATH_NUMBER:
        case LFUN_MATH_NONUMBER:
        case LFUN_MATH_LIMITS:
-       case LFUN_SUBSCRIPT:
-       case LFUN_SUPERSCRIPT:
        {
                setErrorMessage(N_("This is only allowed in math mode!"));
        }
        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;
@@ -1583,7 +1623,7 @@ exit_with_message:
 
                        if (!shortcuts.empty()) {
                                comname += ": " + shortcuts;
-                       } else if (!argsadded) {
+                       } else if (!argsadded && !argument.empty()) {
                                comname += " " + argument;
                        }