]> git.lyx.org Git - lyx.git/blob - src/frontends/xforms/Menubar_pimpl.C
1f1a198a795b3fa7d110db6f76fc2206dff83256
[lyx.git] / src / frontends / xforms / Menubar_pimpl.C
1 /* This file is part of
2 * ======================================================
3
4 *           LyX, The Document Processor
5 *        
6 *           Copyright (C) 1999 The LyX Team.
7 *
8 *======================================================*/
9
10 #ifdef __GNUG__
11 #pragma implementation
12 #endif
13
14 #include <config.h>
15
16 #include <algorithm>
17 #include "support/lstrings.h"
18 #include "support/LAssert.h"
19 #include "debug.h"
20 #include "LyXAction.h"
21 #include "lyxfunc.h"
22 #include "kbmap.h"
23 #include "buffer.h"
24 #include "LyXView.h"
25 #include "MenuBackend.h"
26 #include "Menubar_pimpl.h"
27
28 using std::endl;
29 using std::vector;
30 using std::max;
31 using std::min;
32
33 typedef vector<int>::size_type size_type;
34
35 extern kb_keymap * toplevel_keymap;
36 extern LyXAction lyxaction;
37
38 // Some constants
39 static const int MENU_LABEL_SIZE = FL_NORMAL_SIZE;
40 static const int mheight = 30;
41 static const int mbheight= 22;
42 // where to place the menubar?
43 static const int yloc = (mheight - mbheight)/2; //air + bw;
44 static const int mbadd = 20; // menu button add (to width)
45 // Some space between buttons on the menubar 
46 static const int air = 2;
47 static char const * menu_tabstop = "aa";
48 static char const * default_tabstop = "aaaaaaaa";
49 // We do not want to mix position values in a menu (like the index of
50 // a submenu) with the action numbers which convey actual information.
51 // Therefore we offset all the action values by an arbitrary large
52 // constant. 
53 static const int action_offset = 1000;
54
55
56 //Defined later, used in makeMenubar().
57 extern "C"
58 void C_Menubar_Pimpl_MenuCallback(FL_OBJECT * ob, long button);
59
60 // This is used a few times below.
61 static inline
62 int string_width(string const & str) 
63 {
64         return fl_get_string_widthTAB(FL_NORMAL_STYLE, MENU_LABEL_SIZE,
65                                       str.c_str(),
66                                       static_cast<int>(str.length()));
67 }
68
69
70 Menubar::Pimpl::Pimpl(LyXView * view, MenuBackend const & mb) 
71         : owner_(view), menubackend_(&mb), current_group_(0)
72 {
73         for (MenuBackend::const_iterator menu = menubackend_->begin();
74             menu != menubackend_->end() ; ++menu) {
75                 if (menu->menubar()) {
76                         FL_OBJECT * group = fl_bgn_group();
77                         makeMenubar(*menu);
78                         fl_end_group();
79                         fl_hide_object(group);
80                         lyxerr[Debug::GUI]
81                                 << "Menubar::Pimpl::Pimpl: "
82                                 << "creating and hiding group " << group
83                                 << " for menubar " << menu->name() << endl;
84                         menubarmap_[menu->name()] = group;
85                 }
86         }
87 }
88
89
90 Menubar::Pimpl::~Pimpl() 
91 {
92         // Should we do something here?
93 }
94
95
96 void Menubar::Pimpl::makeMenubar(Menu const &menu)
97 {
98         FL_FORM * form = owner_->getForm(); 
99         int moffset = 0;
100
101         // Create menu frame if there is non yet.
102         FL_OBJECT * frame = fl_add_frame(FL_UP_FRAME, 0, 0,
103                                          form->w, mheight, "");
104         fl_set_object_resize(frame, FL_RESIZE_ALL);
105         fl_set_object_gravity(frame, NorthWestGravity, 
106                               NorthEastGravity);
107
108         for (Menu::const_iterator i = menu.begin(); 
109              i != menu.end(); ++i) {
110                 FL_OBJECT * obj;
111                 if (i->kind() != MenuItem::Submenu) {
112                         lyxerr << "ERROR: Menubar::Pimpl::createMenubar:"
113                                 " only submenus can appear in a menubar";
114                         break;
115                 }
116                 string label = i->label();
117                 string shortcut = "#" + i->shortcut();
118                 int width = string_width(label);
119                 obj = fl_add_button(FL_MENU_BUTTON,
120                                     air + moffset, yloc,
121                                     width + mbadd,
122                                     mbheight, 
123                                     label.c_str());
124                 fl_set_object_boxtype(obj, FL_FLAT_BOX);
125                 fl_set_object_color(obj, FL_MCOL, FL_MCOL);
126                 fl_set_object_lsize(obj, MENU_LABEL_SIZE);
127                 fl_set_object_lstyle(obj, FL_NORMAL_STYLE);
128                 fl_set_object_resize(obj, FL_RESIZE_ALL);
129                 fl_set_object_gravity(obj, NorthWestGravity, 
130                                       NorthWestGravity);
131                 moffset += obj->w + air;
132                 fl_set_object_shortcut(obj, shortcut.c_str(), 1);
133                 fl_set_object_callback(obj, C_Menubar_Pimpl_MenuCallback, 1);
134
135                 ItemInfo * iteminfo = new ItemInfo(this, 
136                                                    new MenuItem(*i), obj);
137                 buttonlist_.push_back(iteminfo);
138                 obj->u_vdata = iteminfo;
139         }
140
141 }
142
143 void Menubar::Pimpl::set(string const & menu_name) 
144 {
145         lyxerr[Debug::GUI] << "Entering Menubar::Pimpl::set " 
146                            << "for menu `" << menu_name << "'" << endl;
147
148         if (menu_name != current_menu_name_) {
149                 MenubarMap::iterator mbit = menubarmap_.find(menu_name);
150
151                 if (mbit == menubarmap_.end()) {
152                         lyxerr << "ERROR:set: Unknown menu `" << menu_name
153                                << "'" << endl;
154                         return;
155                 }
156
157                 if (current_group_) {
158                         lyxerr[Debug::GUI] << "  hiding group "
159                                            << current_group_ << endl;
160                         fl_hide_object(current_group_);
161                 }
162                 
163                 lyxerr[Debug::GUI] << "  showing group "
164                                    << mbit->second << endl;
165                 fl_show_object(mbit->second);
166                 current_menu_name_ = menu_name;
167                 current_group_ = mbit->second;
168                 lyxerr[Debug::GUI] << "Menubar::Pimpl::set: Menubar set."
169                                    << endl;
170         }
171         else
172                 lyxerr [Debug::GUI] << "Menubar::Pimpl::set: Nothing to do."
173                                     << endl;
174
175
176 void Menubar::Pimpl::openByName(string const & name)
177 {
178         if (menubackend_->getMenu(current_menu_name_).hasSubmenu(name)) {
179                 for (ButtonList::const_iterator cit = buttonlist_.begin();
180                      cit != buttonlist_.end(); ++cit) {
181                         if ((*cit)->item_->submenu() == name) {
182                                 MenuCallback((*cit)->obj_, 1);
183                                 return;
184                         }
185                 }
186         }
187         lyxerr << "Menubar::Pimpl::openByName: menu "
188                << name << " not found" << endl;
189 }
190
191
192 static inline
193 string const limit_string_length(string const & str)
194 {
195         string::size_type const max_item_length = 45;
196
197         if (str.size() > max_item_length)
198                 return str.substr(0, max_item_length - 3) + "...";
199         else
200                 return str;
201 }
202
203
204 static
205 int get_new_submenu(vector<int> & smn, Window win)
206 {
207         static size_type max_number_of_menus = 32;
208         if (smn.size() >= max_number_of_menus)
209                 max_number_of_menus =
210                     fl_setpup_maxpup(static_cast<int>(2*smn.size()));
211         int menu = fl_newpup(win);
212         smn.push_back(menu);
213         return menu;
214 }
215
216
217 size_type const max_number_of_items = 25;
218
219 static inline
220 string const fixlabel(string const & str)
221 {
222 #if FL_REVISION < 89
223         return subst(str, '%', '?');
224 #else
225         return subst(str, "%", "%%");
226 #endif
227 }
228
229
230 void add_toc2(int menu, string const & extra_label,
231               vector<int> & smn, Window win,
232               vector<Buffer::TocItem> const & toc_list,
233               size_type from, size_type to, int depth)
234 {
235         if (to - from <= max_number_of_items) {
236                 for (size_type i = from; i < to; ++i) {
237                         int const action = lyxaction.
238                                 getPseudoAction(LFUN_GOTO_PARAGRAPH,
239                                                 tostr(toc_list[i].par->id()));
240                         string label(4 * max(0, toc_list[i].depth - depth),' ');
241                         label += fixlabel(toc_list[i].str);
242                         label = limit_string_length(label);
243                         label += "%x" + tostr(action + action_offset);
244                         if (i == to - 1 && depth == 0)
245                                 label += extra_label;
246                         fl_addtopup(menu, label.c_str());
247                 }
248         } else {
249                 size_type pos = from;
250                 size_type count = 0;
251                 while (pos < to) {
252                         ++count;
253                         if (count > max_number_of_items) {
254                                 int menu2 = get_new_submenu(smn, win);
255                                 add_toc2(menu2, extra_label, smn, win,
256                                          toc_list, pos, to, depth);
257                                 string label = _("More");
258                                 label += "...%m";
259                                 if (depth == 0)
260                                         label += extra_label;
261                                 fl_addtopup(menu, label.c_str(), menu2);
262                                 break;
263                         }
264                         size_type new_pos = pos+1;
265                         while (new_pos < to &&
266                                toc_list[new_pos].depth > depth)
267                                 ++new_pos;
268
269                         int const action = lyxaction.
270                                 getPseudoAction(LFUN_GOTO_PARAGRAPH,
271                                                 tostr(toc_list[pos].par->id()));
272                         string label(4 * max(0, toc_list[pos].depth - depth), ' ');
273                         label += fixlabel(toc_list[pos].str);
274                         label = limit_string_length(label);
275                         if (new_pos == to && depth == 0)
276                                 label += extra_label;
277
278                         if (new_pos == pos + 1) {
279                                 label += "%x" + tostr(action + action_offset);
280                                 fl_addtopup(menu, label.c_str());
281                         } else {
282                                 int menu2 = get_new_submenu(smn, win);
283                                 add_toc2(menu2, extra_label, smn, win,
284                                          toc_list, pos, new_pos, depth+1);
285                                 label += "%m";
286                                 fl_addtopup(menu, label.c_str(), menu2);
287                         }
288                         pos = new_pos;
289                 }
290         }
291 }
292
293
294 void Menubar::Pimpl::add_toc(int menu, string const & extra_label,
295                              vector<int> & smn, Window win)
296 {
297         //xgettext:no-c-format
298         static char const * MenuNames[3] = { N_("List of Figures%m"),
299         //xgettext:no-c-format
300                                              N_("List of Tables%m"),
301         //xgettext:no-c-format
302                                              N_("List of Algorithms%m") };
303
304         vector<vector<Buffer::TocItem> > toc_list =
305                 owner_->buffer()->getTocList();
306
307         // Handle LOF/LOT/LOA
308         int max_nonempty = 0;
309         for (int i = 1; i <= 3; ++i)
310                 if (!toc_list[i].empty())
311                         max_nonempty = i;
312
313         for (int j = 1; j <= 3; ++j)
314                 if (!toc_list[j].empty()) {
315                         int menu2 = get_new_submenu(smn, win);
316                         for (size_type i = 0; i < toc_list[j].size(); ++i) {
317                                 if (i > max_number_of_items) {
318                                         fl_addtopup(menu2, ". . .%d");
319                                         break;
320                                 }
321                                 int const action = lyxaction.
322                                         getPseudoAction(LFUN_GOTO_PARAGRAPH,
323                                                         tostr(toc_list[j][i].par->id()));
324                                 string label = fixlabel(toc_list[j][i].str);
325                                 label = limit_string_length(label);
326                                 label += "%x" + tostr(action + action_offset);
327                                 fl_addtopup(menu2, label.c_str());
328                         }
329                         if (j == max_nonempty) {
330                                 string label = _(MenuNames[j - 1]);
331                                 label += "%l";
332                                 fl_addtopup(menu, label.c_str(), menu2);
333                         } else
334                                 fl_addtopup(menu, _(MenuNames[j - 1]), menu2);
335                 }
336
337         // Handle normal TOC
338         if (max_nonempty == 0 && toc_list[0].empty()) {
339                 fl_addtopup(menu,_("No Table of Contents%i"));
340                 return;
341         }
342
343         add_toc2(menu, extra_label, smn, win,
344                  toc_list[0], 0, toc_list[0].size(), 0);
345
346 }
347
348 void add_references2(int menu, vector<int> & smn, Window win,
349                      vector<string> const & label_list, string const & type)
350 {
351         size_type const max_number_of_items = 25;
352         size_type const max_number_of_items2 = 20;
353         string::size_type const max_item_length = 40;
354         string::size_type const max_item_length2 = 20;
355
356         if (label_list.size() <= max_number_of_items)
357                 for (size_type i = 0; i < label_list.size(); ++i) {
358                         int const action = (type == "goto")
359                                 ? lyxaction.getPseudoAction(LFUN_REF_GOTO, 
360                                                             label_list[i])
361                                 : lyxaction.getPseudoAction(LFUN_REF_INSERT,
362                                                             type + "|++||++|"
363                                                             + label_list[i]);
364                         string label = label_list[i];
365                         if (label.size() > max_item_length)
366                                 label = label.substr(0, max_item_length-1) + "$";
367                         label += "%x" + tostr(action + action_offset);
368                         fl_addtopup(menu, label.c_str());
369                 }
370         else {
371                 size_type count = 0;
372                 for (size_type i = 0; i < label_list.size();
373                      i += max_number_of_items2) {
374                         ++count;
375                         if (count > max_number_of_items) {
376                                 fl_addtopup(menu, ". . .%d");
377                                 break;
378                         }
379                         size_type j = min(label_list.size(),
380                                           i+max_number_of_items2);
381
382                         string label;
383                         label += (label_list[i].size() > max_item_length2)
384                                 ? label_list[i].substr(0, max_item_length2-1) + "$"
385                                 : label_list[i];
386                         label += "..";
387                         label += (label_list[j-1].size() > max_item_length2)
388                                 ? label_list[j-1].substr(0, max_item_length2-1) + "$"
389                                 : label += label_list[j-1];
390
391                         int menu2 = get_new_submenu(smn, win);
392                         for (size_type k = i;  k < j; ++k) {
393                                 int const action = (type == "goto")
394                                         ? lyxaction.getPseudoAction(LFUN_REF_GOTO, 
395                                                                     label_list[k])
396                                         : lyxaction.getPseudoAction(LFUN_REF_INSERT,
397                                                                     type + "|++||++|"
398                                                                     + label_list[k]);
399                                 string label2 = label_list[k];
400                                 if (label2.size() > max_item_length)
401                                         label2 = label2.substr(0, max_item_length-1) + "$";
402                                 label2 += "%x" + tostr(action + action_offset);
403                                 fl_addtopup(menu2, label2.c_str());
404                         }
405                         label += "%m";
406                         fl_addtopup(menu, label.c_str(), menu2);
407                 }
408         }
409 }
410
411
412 void Menubar::Pimpl::add_references(int menu, string const & extra_label,
413                                     vector<int> & smn, Window win)
414 {
415         //xgettext:no-c-format
416         static char const * MenuNames[6] = { N_("Insert Reference%m"),
417         //xgettext:no-c-format
418                                              N_("Insert Page Number%m"),
419         //xgettext:no-c-format
420                                              N_("Insert vref%m"),
421         //xgettext:no-c-format
422                                              N_("Insert vpageref%m"),
423         //xgettext:no-c-format
424                                              N_("Insert Pretty Ref%m"),
425         //xgettext:no-c-format
426                                              N_("Goto Reference%m") };
427
428         int const EMPTY = 1;
429         int const SGML = 2;
430         int const READONLY = 4;
431
432         static int MenuFlags[6] = {
433                 EMPTY | READONLY,
434                 EMPTY | READONLY,
435                 EMPTY | READONLY | SGML,
436                 EMPTY | READONLY | SGML,
437                 EMPTY | READONLY | SGML,
438                 EMPTY };
439
440         static string const MenuTypes[6] = {
441                 "ref", "pageref", "vref", "vpageref", "prettyref", "goto" };
442
443         vector<string> label_list = owner_->buffer()->getLabelList();
444
445         int flag = 0;
446         if (label_list.empty())
447                 flag += EMPTY;
448         if (owner_->buffer()->isSGML())
449                 flag += SGML;
450         if (owner_->buffer()->isReadonly())
451                 flag += READONLY;
452
453         int max_nonempty = -1;
454         for (int i = 0; i < 6; ++i)
455                 if ((MenuFlags[i] & flag) == 0)
456                         max_nonempty = i;
457
458         for (int i = 0; i < 6; ++i) {
459                 if ((MenuFlags[i] & flag) == 0) {
460                         string label = _(MenuNames[i]);
461                         if (i == max_nonempty)
462                                 label += extra_label;
463                         int menu2 = get_new_submenu(smn, win);
464                         add_references2(menu2, smn, win, label_list,
465                                         MenuTypes[i]);
466                         fl_addtopup(menu, label.c_str(), menu2);
467                 }
468         }
469 }
470
471
472 int Menubar::Pimpl::create_submenu(Window win, LyXView * view, 
473                                    string const & menu_name, 
474                                    vector<int> & smn) 
475 {
476         if (!menubackend_->hasMenu(menu_name)){ 
477                 lyxerr << "ERROR:create_submenu: Unknown menu `" 
478                        << menu_name << "'" << endl;
479                 return -1;
480         }
481         Menu md = Menu();
482         menubackend_->getMenu(menu_name).expand(md, owner_->buffer());
483
484         int menu = get_new_submenu(smn, win);
485         fl_setpup_softedge(menu, true);
486         fl_setpup_bw(menu, -1);
487         lyxerr[Debug::GUI] << "Adding menu " << menu 
488                            << " in deletion list" << endl;
489
490         // Compute the size of the largest label (because xforms is
491         // not able to support shortcuts correctly...)
492         int max_width = 0;
493         string widest_label;
494         Menu::const_iterator end = md.end();
495         for (Menu::const_iterator i = md.begin(); i != end; ++i) {
496                 MenuItem const & item = (*i);
497                 if (item.kind() == MenuItem::Command) {
498                         string label = item.label() + '\t';
499                         int width = string_width(label);
500                         if (width > max_width) {
501                                 max_width = width;
502                                 widest_label = label;
503                         }
504                 }
505         }
506         lyxerr[Debug::GUI] << "max_width=" << max_width 
507                            << ", widest_label=`" << widest_label 
508                            << "'" << endl;
509
510         // Compute where to put separators
511         vector<string> extra_labels(md.size());
512         vector<string>::iterator it = extra_labels.begin();
513         vector<string>::iterator last = it;
514         for (Menu::const_iterator i = md.begin(); i != end; ++i, ++it)
515                 if (i->kind() == MenuItem::Separator)
516                         *last = "%l";
517                 else if (!i->optional() ||
518                          !(view->getLyXFunc()->getStatus(i->action()) & LyXFunc::Disabled))
519                         last = it;
520
521         it = extra_labels.begin();
522         for (Menu::const_iterator i = md.begin(); i != end; ++i, ++it) {
523                 MenuItem const & item = (*i);
524                 string & extra_label = *it;
525
526                 switch (item.kind()) {
527                 case MenuItem::Command: {
528                         LyXFunc::func_status flag = 
529                                 view->getLyXFunc()->getStatus(item.action()); 
530
531                         // handle optional entries.
532                         if (item.optional() && (flag & LyXFunc::Disabled)) {
533                                 lyxerr[Debug::GUI] 
534                                         << "Skipping optional item " 
535                                         << item.label() << endl; 
536                                 break;
537                         }
538
539                         // Get the keys bound to this action, but keep only the
540                         // first one later
541                         string const accel = toplevel_keymap->findbinding(item.action());
542                         // Build the menu label from all the info
543                         string label = item.label();
544
545                         if (!accel.empty()) {
546                                 // Try to be clever and add  just enough
547                                 // tabs to align shortcuts.
548                                 do 
549                                         label += '\t';
550                                 while (string_width(label) < max_width);
551                                 label += accel.substr(1,accel.find(']') - 1);
552                         }
553                         label += "%x" + tostr(item.action() + action_offset)
554                                 + extra_label;
555                         
556                         // Modify the entry using the function status
557                         string pupmode;
558                         if (flag & (LyXFunc::Disabled | LyXFunc::Unknown))
559                                 pupmode += "%i";
560                         if (flag & LyXFunc::ToggleOn)
561                                 pupmode += "%B";
562                         if (flag & LyXFunc::ToggleOff)
563                                 pupmode += "%b";
564                         label += pupmode;
565
566                         // Finally the menu shortcut
567                         string shortcut = item.shortcut();
568
569                         if (!shortcut.empty()) {
570                                 shortcut += lowercase(shortcut[0]);
571                                 label += "%h";
572                                 fl_addtopup(menu, label.c_str(), 
573                                             shortcut.c_str());
574                         } else
575                                 fl_addtopup(menu, label.c_str());
576                         
577                         lyxerr[Debug::GUI] << "Command: \""  
578                                            << lyxaction.getActionName(item.action())
579                                            << "\", binding \"" << accel
580                                            << "\", shortcut \"" << shortcut 
581                                            << "\"" << endl;
582                         break;
583                 }
584
585                 case MenuItem::Submenu: {
586                         int submenu = create_submenu(win, view, 
587                                                      item.submenu(), smn);
588                         if (submenu == -1)
589                                 return -1;
590                         string label = item.label();
591                         label += extra_label + "%m";
592                         string shortcut = item.shortcut();
593                         if (!shortcut.empty()) {
594                                 shortcut += lowercase(shortcut[0]);
595                                 label += "%h";
596                                 fl_addtopup(menu, label.c_str(),
597                                             submenu, shortcut.c_str());
598                         }
599                         else {
600                                 fl_addtopup(menu, label.c_str(), submenu);
601                         }
602                         break;
603                 }
604
605                 case MenuItem::Separator:
606                         // already done, and if it was the first one,
607                         // we just ignore it.
608                         break;
609
610                 case MenuItem::Toc:
611                         add_toc(menu, extra_label, smn, win);
612                         break;
613
614                 case MenuItem::References:
615                         add_references(menu, extra_label, smn, win);
616                         break;
617
618                 case MenuItem::Documents: 
619                 case MenuItem::Lastfiles: 
620                 case MenuItem::ViewFormats:
621                 case MenuItem::UpdateFormats:
622                 case MenuItem::ExportFormats:
623                 case MenuItem::ImportFormats:
624                         lyxerr << "Menubar::Pimpl::create_submenu: "
625                                 "this should not happen" << endl;
626                         break;
627
628                 }
629         }
630         return menu;
631 }
632
633
634 extern "C"
635 void C_Menubar_Pimpl_MenuCallback(FL_OBJECT * ob, long button)
636 {
637         Menubar::Pimpl::MenuCallback(ob, button);
638 }
639
640
641 void Menubar::Pimpl::MenuCallback(FL_OBJECT * ob, long button)
642 {
643         ItemInfo * iteminfo = static_cast<ItemInfo *>(ob->u_vdata);
644 //      lyxerr << "MenuCallback: ItemInfo address=" << iteminfo
645 //             << "Val=(pimpl_=" << iteminfo->pimpl_
646 //             << ", item_=" << iteminfo->item_
647 //             << ", obj_=" << iteminfo->obj_ << ")" <<endl;
648
649         LyXView * view = iteminfo->pimpl_->owner_;
650         MenuItem const * item = iteminfo->item_;
651
652         if (button == 1) {
653                 // set the pseudo menu-button
654                 fl_set_object_boxtype(ob, FL_DOWN_BOX);
655                 fl_set_button(ob, 0);
656                 fl_redraw_object(ob);
657         }
658
659         // Paranoia check
660         Assert(item->kind() == MenuItem::Submenu);
661         
662         // set tabstop length
663         fl_set_tabstop(menu_tabstop);
664         vector<int> submenus;
665         int menu = iteminfo->pimpl_->
666                 create_submenu(FL_ObjWin(ob), view, 
667                                item->submenu(), submenus);
668         if (menu != -1) {
669                 // place popup
670                 fl_setpup_position(view->getForm()->x + ob->x,
671                                    view->getForm()->y + ob->y + ob->h + 10);   
672                 int choice = fl_dopup(menu);
673                 
674                 if (button == 1) {
675                                 // set the pseudo menu-button back
676                         fl_set_object_boxtype(ob, FL_FLAT_BOX);
677                         fl_redraw_object(ob);
678                 }
679
680                 // If the action value is too low, then it is not a
681                 // valid action, but something else.
682                 if (choice >= action_offset + 1) {
683                         view->getLyXFunc()->Dispatch(choice - action_offset);
684                 }
685                 else {
686                         lyxerr[Debug::GUI]
687                                 << "MenuCallback: ignoring bogus action "
688                                 << choice << endl;
689                 }
690         }
691         else 
692                 lyxerr << "Error in MenuCallback" << endl;
693         
694         std::for_each(submenus.begin(), submenus.end(), fl_freepup);
695         // restore tabstop length
696         fl_set_tabstop(default_tabstop);
697
698 }