X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Ffrontends%2Fxforms%2FMenubar_pimpl.C;h=a9efd984aa618af23194f1ca37a2a336b74d985e;hb=3e39bef2c13125023f3b72532d90575bbe307335;hp=7e486ad3fe98366328e2e37caca5dc4971fd688b;hpb=b8072fd6f77ebd592ecd514dadc04839516b8a8b;p=lyx.git diff --git a/src/frontends/xforms/Menubar_pimpl.C b/src/frontends/xforms/Menubar_pimpl.C index 7e486ad3fe..a9efd984aa 100644 --- a/src/frontends/xforms/Menubar_pimpl.C +++ b/src/frontends/xforms/Menubar_pimpl.C @@ -1,387 +1,516 @@ -/* This file is part of -* ====================================================== -* -* LyX, The Document Processor -* -* Copyright (C) 1999 The LyX Team. -* -*======================================================*/ +/** + * \file Menubar_pimpl.C + * Copyright 1999-2001 The LyX Team. + * See the file COPYING. + * + * \author Lars Gullik Bjønnes, larsbj@lyx.org + */ + +#include + +#include #ifdef __GNUG__ #pragma implementation #endif -#include - -#include -#include +#include "Menubar_pimpl.h" +#include "MenuBackend.h" +#include "LyXAction.h" +#include "kbmap.h" +#include "buffer.h" +#include "Dialogs.h" +#include "XFormsView.h" +#include "lyxfunc.h" +#include "FloatList.h" #include "support/lstrings.h" -#include "support/filetools.h" -#include "support/StrPool.h" #include "support/LAssert.h" +#include "gettext.h" #include "debug.h" -#include "LyXAction.h" -#include "lyxfunc.h" -#include "kbmap.h" -#include "bufferlist.h" -#include "lastfiles.h" -#include "LyXView.h" -#include "MenuBackend.h" -#include "Menubar_pimpl.h" using std::endl; +using std::vector; +using std::max; +using std::min; +using std::for_each; -extern kb_keymap * toplevel_keymap; +typedef vector::size_type size_type; + +extern boost::scoped_ptr toplevel_keymap; extern LyXAction lyxaction; -extern BufferList bufferlist; -extern LastFiles * lastfiles; + +namespace { // Some constants -const int MENU_LABEL_SIZE = FL_NORMAL_SIZE; -const int mheight = 30; -const int mbheight= 22; +int const MENU_LABEL_SIZE = FL_NORMAL_SIZE; +int const MENU_LABEL_STYLE = FL_NORMAL_STYLE; +int const mheight = 30; +int const mbheight= 22; // where to place the menubar? -const int yloc = (mheight - mbheight)/2; //air + bw; -const int mbadd = 20; // menu button add (to width) -// Some space between buttons on the menubar -const int air = 2; +int const yloc = (mheight - mbheight)/2; //air + bw; +int const mbadd = 20; // menu button add (to width) +// Some space between buttons on the menubar +int const air = 2; char const * menu_tabstop = "aa"; char const * default_tabstop = "aaaaaaaa"; - - -Menubar::Pimpl::Pimpl(LyXView * view, MenuBackend const & mb) - : frame_(0), owner_(view), menubackend_(&mb) -{ - // Should we do something here? -} - -Menubar::Pimpl::~Pimpl() -{ - // Should we do something here? -} +// We do not want to mix position values in a menu (like the index of +// a submenu) with the action numbers which convey actual information. +// Therefore we offset all the action values by an arbitrary large +// constant. +int const action_offset = 1000; // This is used a few times below. inline -int string_width(string const & str) +int string_width(string const & str) { - return fl_get_string_widthTAB(FL_NORMAL_STYLE, MENU_LABEL_SIZE, - str.c_str(), str.length()); + return fl_get_string_widthTAB(MENU_LABEL_STYLE, MENU_LABEL_SIZE, + str.c_str(), + static_cast(str.length())); } -//Defined later, used in set(). -extern "C" -void C_Menubar_Pimpl_MenuCallback(FL_OBJECT * ob, long button); +} // namespace anon -void Menubar::Pimpl::set(string const & menu_name) -{ - lyxerr[Debug::GUI] << "Entering Menubar::Pimpl::set " - << "for menu `" << menu_name << "'" << endl; - if (menu_name == current_menu) { - lyxerr[Debug::GUI] << "Nothing to do." << endl; - return; - } +extern "C" { - // If the backend has not been initialized yet, we use a - // default instead. - if (menubackend_->empty()) { - lyxerr << "Menubar::Pimpl::set: menubackend is empty! " - "using default values." << endl; - MenuBackend * mb = new MenuBackend(); - mb->defaults(); - menubackend_ = mb; + //Defined later, used in makeMenubar(). + static + void C_Menubar_Pimpl_MenuCallback(FL_OBJECT * ob, long button) + { + Menubar::Pimpl::MenuCallback(ob, button); } - if (!menubackend_->hasMenu(menu_name)){ - lyxerr << "ERROR:set: Unknown menu `" << menu_name - << "'" << endl; - return; - } +} - Menu menu = menubackend_->getMenu(menu_name); - if (!menu.menubar()) { - lyxerr << "Only a menubar-type object can go in a " - "toplevel menu" << endl; - return; +Menubar::Pimpl::Pimpl(LyXView * view, MenuBackend const & mb) + : owner_(view), menubackend_(&mb), current_group_(0) +{ + for (MenuBackend::const_iterator menu = menubackend_->begin(); + menu != menubackend_->end() ; ++menu) { + if (menu->menubar()) { + FL_OBJECT * group = fl_bgn_group(); + makeMenubar(*menu); + fl_end_group(); + fl_hide_object(group); + lyxerr[Debug::GUI] + << "Menubar::Pimpl::Pimpl: " + << "creating and hiding group " << group + << " for menubar " << menu->name() << endl; + menubarmap_[menu->name()] = group; + } } +} - current_menu = menu_name; - FL_FORM * form = owner_->getForm(); - int moffset = 0; - bool form_was_open, form_was_frozen; - - if (fl_current_form == form) - form_was_open = true; - else if (fl_current_form == 0) { - form_was_open = false; - fl_addto_form(form); - } - else { - lyxerr << "Something is wrong: unknown form " - << fl_current_form << " is already open" - << "(main form is " << form << ")" << endl; - return; - } - if (form->frozen) - form_was_frozen = true; - else { - form_was_frozen = false; - fl_freeze_form(form); - } - // Delete old buttons if there are some. - for(ButtonList::const_iterator cit = buttonlist_.begin(); - cit != buttonlist_.end(); ++cit) { - if ((*cit)->obj_) { - fl_delete_object((*cit)->obj_); - fl_free_object((*cit)->obj_); - } - delete (*cit); - } - buttonlist_.clear(); +void Menubar::Pimpl::makeMenubar(Menu const & menu) +{ + FL_FORM * form = static_cast(owner_)->getForm(); + int moffset = 0; // Create menu frame if there is non yet. - if (!frame_) { - frame_ = fl_add_frame(FL_UP_FRAME, 0, 0, form->w, mheight, ""); - fl_set_object_resize(frame_, FL_RESIZE_ALL); - fl_set_object_gravity(frame_, NorthWestGravity, - NorthEastGravity); - } - - for (Menu::const_iterator i = menu.begin(); - i != menu.end(); ++i) { + FL_OBJECT * frame = fl_add_frame(FL_UP_FRAME, 0, 0, + form->w, mheight, ""); + fl_set_object_resize(frame, FL_RESIZE_ALL); + fl_set_object_gravity(frame, NorthWestGravity, + NorthEastGravity); + + Menu::const_iterator i = menu.begin(); + Menu::const_iterator end = menu.end(); + for (; i != end; ++i) { FL_OBJECT * obj; if (i->kind() != MenuItem::Submenu) { - lyxerr << "ERROR: Menubar::Pimpl::Pimpl:" - " only submenus can appear in a menubar"; - break; + lyxerr << "ERROR: Menubar::Pimpl::createMenubar:" + " only submenus can appear in a menubar" + << endl; + continue; } - string label = i->label(); - string shortcut = i->shortcut(); - int width = string_width(label); - obj = fl_add_button(FL_TOUCH_BUTTON, + string const label = i->label(); + string const shortcut = "#" + i->shortcut(); + int const width = string_width(label); + obj = fl_add_button(FL_MENU_BUTTON, air + moffset, yloc, width + mbadd, - mbheight, + mbheight, label.c_str()); fl_set_object_boxtype(obj, FL_FLAT_BOX); fl_set_object_color(obj, FL_MCOL, FL_MCOL); fl_set_object_lsize(obj, MENU_LABEL_SIZE); - fl_set_object_lstyle(obj, FL_NORMAL_STYLE); + fl_set_object_lstyle(obj, MENU_LABEL_STYLE); fl_set_object_resize(obj, FL_RESIZE_ALL); - fl_set_object_gravity(obj, NorthWestGravity, + fl_set_object_gravity(obj, NorthWestGravity, NorthWestGravity); moffset += obj->w + air; fl_set_object_shortcut(obj, shortcut.c_str(), 1); fl_set_object_callback(obj, C_Menubar_Pimpl_MenuCallback, 1); - ItemInfo * iteminfo = new ItemInfo(this, - new MenuItem(*i), obj); + + boost::shared_ptr + iteminfo(new ItemInfo(this, new MenuItem(*i), obj)); buttonlist_.push_back(iteminfo); - obj->u_vdata = iteminfo; -// lyxerr << "MenuCallback: ItemInfo address=" << iteminfo -// << " Val=(pimpl_=" << iteminfo->pimpl_ -// << ", item_=" << iteminfo->item_ -// << ", obj_=" << iteminfo->obj_ << ")" <u_vdata = iteminfo.get(); } - if (!form_was_frozen) { - fl_unfreeze_form(form); - } - if (!form_was_open) - fl_end_form(); +} - lyxerr[Debug::GUI] << "Menubar set." << endl; -} -void Menubar::Pimpl::openByName(string const & name) +void Menubar::Pimpl::set(string const & menu_name) { - for(ButtonList::const_iterator cit = buttonlist_.begin(); - cit != buttonlist_.end(); ++cit) { - if ((*cit)->item_->submenu() == name) { - MenuCallback((*cit)->obj_, 1); + lyxerr[Debug::GUI] << "Entering Menubar::Pimpl::set " + << "for menu `" << menu_name << "'" << endl; + + if (menu_name != current_menu_name_) { + MenubarMap::iterator mbit = menubarmap_.find(menu_name); + + if (mbit == menubarmap_.end()) { + lyxerr << "ERROR:set: Unknown menu `" << menu_name + << "'" << endl; return; } + + if (current_group_) { + lyxerr[Debug::GUI] << " hiding group " + << current_group_ << endl; + fl_hide_object(current_group_); + } + + lyxerr[Debug::GUI] << " showing group " + << mbit->second << endl; + fl_show_object(mbit->second); + current_menu_name_ = menu_name; + current_group_ = mbit->second; + lyxerr[Debug::GUI] << "Menubar::Pimpl::set: Menubar set." + << endl; + } else + lyxerr [Debug::GUI] << "Menubar::Pimpl::set: Nothing to do." + << endl; +} + + +void Menubar::Pimpl::openByName(string const & name) +{ + if (menubackend_->getMenu(current_menu_name_).hasSubmenu(name)) { + for (ButtonList::const_iterator cit = buttonlist_.begin(); + cit != buttonlist_.end(); ++cit) { + if ((*cit)->item_->submenu() == name) { + MenuCallback((*cit)->obj_, 1); + return; + } + } } lyxerr << "Menubar::Pimpl::openByName: menu " << name << " not found" << endl; } -void Menubar::Pimpl::add_lastfiles(int menu, string const & extra_label, - std::vector & /*smn*/, - StrPool & strpool) +namespace { + +inline +string const limit_string_length(string const & str) { - int ii = 1; - for (LastFiles::const_iterator cit = lastfiles->begin(); - cit != lastfiles->end() && ii < 10; ++cit, ++ii) { - - int action = - lyxaction.getPseudoAction(LFUN_FILE_OPEN, (*cit)); - string label = tostr(ii) + ". " - + MakeDisplayPath((*cit),30) - + "%x" + tostr(action); - if ((cit + 1) == lastfiles->end()) - label += extra_label; - string shortcut = tostr(ii) + "#" + tostr(ii); - lyxerr[Debug::GUI] << "shortcut is " << shortcut << - endl; - - int n = fl_addtopup(menu, strpool.add(label)); - fl_setpup_shortcut(menu, n, strpool.add(shortcut)); - } + string::size_type const max_item_length = 45; + if (str.size() > max_item_length) + return str.substr(0, max_item_length - 3) + "..."; + else + return str; } -void Menubar::Pimpl::add_documents(int menu, string const & extra_label, - std::vector & /*smn*/, - StrPool & strpool) + +int get_new_submenu(vector & smn, Window win) { - std::vector names = bufferlist.getFileNames(); + static size_type max_number_of_menus = 32; + if (smn.size() >= max_number_of_menus) + max_number_of_menus = + fl_setpup_maxpup(static_cast(2*smn.size())); + int menu = fl_newpup(win); + smn.push_back(menu); + return menu; +} - if (names.empty()) { - fl_addtopup(menu,_("No Documents Open!%i")); - return; + +size_type const max_number_of_items = 25; + +inline +string const fixlabel(string const & str) +{ +#if FL_REVISION < 89 + return subst(str, '%', '?'); +#else + return subst(str, "%", "%%"); +#endif +} + + + +void add_toc2(int menu, string const & extra_label, + vector & smn, Window win, + vector const & toc_list, + size_type from, size_type to, int depth) +{ + int shortcut_count = 0; + if (to - from <= max_number_of_items) { + for (size_type i = from; i < to; ++i) { + int const action = lyxaction. + getPseudoAction(LFUN_GOTO_PARAGRAPH, + tostr(toc_list[i].par->id())); + string label(4 * max(0, toc_list[i].depth - depth),' '); + label += fixlabel(toc_list[i].str); + label = limit_string_length(label); + label += "%x" + tostr(action + action_offset); + if (i == to - 1 && depth == 0) + label += extra_label; + if (toc_list[i].depth == depth + && ++shortcut_count <= 9) { + label += "%h"; + fl_addtopup(menu, label.c_str(), + tostr(shortcut_count).c_str()); + } else + fl_addtopup(menu, label.c_str()); + } + } else { + size_type pos = from; + size_type count = 0; + while (pos < to) { + ++count; + if (count > max_number_of_items) { + int menu2 = get_new_submenu(smn, win); + add_toc2(menu2, extra_label, smn, win, + toc_list, pos, to, depth); + string label = _("More"); + label += "...%m"; + if (depth == 0) + label += extra_label; + fl_addtopup(menu, label.c_str(), menu2); + break; + } + size_type new_pos = pos+1; + while (new_pos < to && + toc_list[new_pos].depth > depth) + ++new_pos; + + int const action = lyxaction. + getPseudoAction(LFUN_GOTO_PARAGRAPH, + tostr(toc_list[pos].par->id())); + string label(4 * max(0, toc_list[pos].depth - depth), ' '); + label += fixlabel(toc_list[pos].str); + label = limit_string_length(label); + if (new_pos == to && depth == 0) + label += extra_label; + string shortcut; + if (toc_list[pos].depth == depth && + ++shortcut_count <= 9) + shortcut = tostr(shortcut_count); + + if (new_pos == pos + 1) { + label += "%x" + tostr(action + action_offset); + if (!shortcut.empty()) { + label += "%h"; + fl_addtopup(menu, label.c_str(), + shortcut.c_str()); + } else + fl_addtopup(menu, label.c_str()); + } else { + int menu2 = get_new_submenu(smn, win); + add_toc2(menu2, extra_label, smn, win, + toc_list, pos, new_pos, depth+1); + label += "%m"; + if (!shortcut.empty()) { + label += "%h"; + fl_addtopup(menu, label.c_str(), menu2, + shortcut.c_str()); + } else + fl_addtopup(menu, label.c_str(), menu2); + } + pos = new_pos; + } } +} + +} // namespace anon + - for (std::vector::const_iterator cit = names.begin(); - cit != names.end() ; ++cit) { - int action = - lyxaction.getPseudoAction(LFUN_SWITCHBUFFER, *cit); - string label = MakeDisplayPath(*cit, 30) - + "%x" + tostr(action); - if ((cit + 1) == names.end()) - label += extra_label; - - fl_addtopup(menu, strpool.add(label)); +void Menubar::Pimpl::add_toc(int menu, string const & extra_label, + vector & smn, Window win) +{ + Buffer::Lists toc_list = owner_->buffer()->getLists(); + Buffer::Lists::const_iterator cit = toc_list.begin(); + Buffer::Lists::const_iterator end = toc_list.end(); + for (; cit != end; ++cit) { + // Handle this elsewhere + if (cit->first == "TOC") continue; + + // All the rest is for floats + int menu_first_sub = get_new_submenu(smn, win); + int menu_current = menu_first_sub; + Buffer::SingleList::const_iterator ccit = cit->second.begin(); + Buffer::SingleList::const_iterator eend = cit->second.end(); + size_type count = 0; + for (; ccit != eend; ++ccit) { + ++count; + if (count > max_number_of_items) { + int menu_tmp = get_new_submenu(smn, win); + string label = _("More"); + label += "...%m"; + fl_addtopup(menu_current, label.c_str(), menu_tmp); + count = 1; + menu_current = menu_tmp; + } + int const action = + lyxaction + .getPseudoAction(LFUN_GOTO_PARAGRAPH, + tostr(ccit->par->id())); + string label = fixlabel(ccit->str); + label = limit_string_length(label); + label += "%x" + tostr(action + action_offset); + fl_addtopup(menu_current, label.c_str()); + } + string const m = floatList[cit->first]->second.name() + "%m"; + fl_addtopup(menu, m.c_str(), menu_first_sub); } + + // Handle normal TOC + cit = toc_list.find("TOC"); + if (cit == end) { + string const tmp = _("No Table of contents%i") + extra_label; + fl_addtopup(menu, tmp.c_str()); + return; + } else { + add_toc2(menu, extra_label, smn, win, + cit->second, 0, cit->second.size(), 0); + } } -int Menubar::Pimpl::create_submenu(Window win, LyXView * view, - string const & menu_name, - std::vector & smn, StrPool & strpool) +int Menubar::Pimpl::create_submenu(Window win, LyXView * view, + string const & menu_name, + vector & smn) { - if (!menubackend_->hasMenu(menu_name)){ - lyxerr << "ERROR:create_submenu: Unknown menu `" + if (!menubackend_->hasMenu(menu_name)) { + lyxerr << "ERROR:create_submenu: Unknown menu `" << menu_name << "'" << endl; return -1; } - Menu md = menubackend_->getMenu(menu_name); + Menu md; + menubackend_->getMenu(menu_name).expand(md, owner_->buffer()); - int menu = fl_newpup(win); + int const menu = get_new_submenu(smn, win); fl_setpup_softedge(menu, true); fl_setpup_bw(menu, -1); - lyxerr[Debug::GUI] << "Adding menu " << menu + lyxerr[Debug::GUI] << "Adding menu " << menu << " in deletion list" << endl; - smn.push_back(menu); // Compute the size of the largest label (because xforms is // not able to support shortcuts correctly...) int max_width = 0; string widest_label; - for (Menu::const_iterator i = md.begin(); i != md.end(); ++i) { - MenuItem item = (*i); + Menu::const_iterator end = md.end(); + for (Menu::const_iterator i = md.begin(); i != end; ++i) { + MenuItem const & item = (*i); if (item.kind() == MenuItem::Command) { - string label = item.label() + '\t'; - int width = string_width(label); + string const label = item.label() + '\t'; + int const width = string_width(label); if (width > max_width) { max_width = width; widest_label = label; } } } - lyxerr[Debug::GUI] << "max_width=" << max_width - << ", widest_label=`" << widest_label + lyxerr[Debug::GUI] << "max_width=" << max_width + << ", widest_label=`" << widest_label << "'" << endl; - for (Menu::const_iterator i = md.begin(); i != md.end(); ++i) { - MenuItem item = (*i); - // Is there a separator after this item? - string extra_label; - if ((i+1) != md.end() - && (i+1)->kind() == MenuItem::Separator) - extra_label = "%l"; - - switch(item.kind()) { + // Compute where to put separators + vector extra_labels(md.size()); + vector::iterator it = extra_labels.begin(); + vector::iterator last = it; + for (Menu::const_iterator i = md.begin(); i != end; ++i, ++it) + if (i->kind() == MenuItem::Separator) + *last = "%l"; + else if (!i->optional() || + !(view->getLyXFunc()->getStatus(i->action()).disabled())) + last = it; + + it = extra_labels.begin(); + for (Menu::const_iterator i = md.begin(); i != end; ++i, ++it) { + MenuItem const & item = (*i); + string & extra_label = *it; + + switch (item.kind()) { case MenuItem::Command: { - LyXFunc::func_status flag = - view->getLyXFunc()->getStatus(item.action()); - + FuncStatus const flag = + view->getLyXFunc()->getStatus(item.action()); // handle optional entries. - if (item.optional() && (flag & LyXFunc::Disabled)) { - lyxerr[Debug::GUI] - << "Skipping optional item " - << item.label() << endl; + if (item.optional() + && (flag.disabled())) { + lyxerr[Debug::GUI] + << "Skipping optional item " + << item.label() << endl; break; } // Get the keys bound to this action, but keep only the // first one later - string accel = toplevel_keymap->findbinding(item.action()); + string const accel = toplevel_keymap->findbinding(kb_action(item.action())); // Build the menu label from all the info string label = item.label(); if (!accel.empty()) { // Try to be clever and add just enough // tabs to align shortcuts. - do + do label += '\t'; - while (string_width(label) < max_width); + while (string_width(label) < max_width + 5); label += accel.substr(1,accel.find(']') - 1); } - label += "%x" + tostr(item.action()) + extra_label; - + label += "%x" + tostr(item.action() + action_offset) + + extra_label; + // Modify the entry using the function status string pupmode; - if (flag & (LyXFunc::Disabled | LyXFunc::Unknown)) - pupmode += "%i"; - if (flag & LyXFunc::ToggleOn) + if (flag.onoff(true)) pupmode += "%B"; - if (flag & LyXFunc::ToggleOff) + if (flag.onoff(false)) pupmode += "%b"; + if (flag.disabled() || flag.unknown()) + pupmode += "%i"; label += pupmode; // Finally the menu shortcut string shortcut = item.shortcut(); - string xfshortcut; + if (!shortcut.empty()) { - xfshortcut += uppercase(shortcut[0]); - xfshortcut += '#'; - xfshortcut += uppercase(shortcut[0]); - xfshortcut += lowercase(shortcut[0]); - xfshortcut += '#'; - xfshortcut += lowercase(shortcut[0]); - lyxerr[Debug::GUI] << "shortcut is " - << xfshortcut << endl; + shortcut += lowercase(shortcut[0]); label += "%h"; - fl_addtopup(menu, strpool.add(label), - strpool.add(xfshortcut)); + fl_addtopup(menu, label.c_str(), + shortcut.c_str()); } else - fl_addtopup(menu, strpool.add(label)); - - lyxerr[Debug::GUI] << "Command: \"" - << lyxaction.getActionName(item.action()) - << "\", Binding " << accel - << ", shortcut " << xfshortcut - << endl; - + fl_addtopup(menu, label.c_str()); + lyxerr[Debug::GUI] << "Command: \"" + << lyxaction.getActionName(item.action()) + << "\", binding \"" << accel + << "\", shortcut \"" << shortcut + << "\"" << endl; break; } case MenuItem::Submenu: { - int submenu = create_submenu(win, view, - item.submenu(), - smn, strpool); + int submenu = create_submenu(win, view, + item.submenu(), smn); if (submenu == -1) return -1; string label = item.label(); label += extra_label + "%m"; string shortcut = item.shortcut(); - int n = fl_addtopup(menu, strpool.add(label), submenu); - fl_setpup_shortcut(menu, n, strpool.add(shortcut)); + if (!shortcut.empty()) { + shortcut += lowercase(shortcut[0]); + label += "%h"; + fl_addtopup(menu, label.c_str(), + submenu, shortcut.c_str()); + } else { + fl_addtopup(menu, label.c_str(), submenu); + } break; } @@ -390,13 +519,20 @@ int Menubar::Pimpl::create_submenu(Window win, LyXView * view, // we just ignore it. break; - case MenuItem::Documents: - add_documents(menu, extra_label, smn, strpool); + case MenuItem::Toc: + add_toc(menu, extra_label, smn, win); break; - - case MenuItem::Lastfiles: - add_lastfiles(menu, extra_label, smn, strpool); + case MenuItem::Documents: + case MenuItem::Lastfiles: + case MenuItem::ViewFormats: + case MenuItem::UpdateFormats: + case MenuItem::ExportFormats: + case MenuItem::ImportFormats: + case MenuItem::FloatListInsert: + case MenuItem::FloatInsert: + lyxerr << "Menubar::Pimpl::create_submenu: " + "this should not happen" << endl; break; } @@ -404,23 +540,17 @@ int Menubar::Pimpl::create_submenu(Window win, LyXView * view, return menu; } -extern "C" -void C_Menubar_Pimpl_MenuCallback(FL_OBJECT * ob, long button) -{ - Menubar::Pimpl::MenuCallback(ob, button); -} - void Menubar::Pimpl::MenuCallback(FL_OBJECT * ob, long button) { ItemInfo * iteminfo = static_cast(ob->u_vdata); -// lyxerr << "MenuCallback: ItemInfo address=" << iteminfo -// << "Val=(pimpl_=" << iteminfo->pimpl_ -// << ", item_=" << iteminfo->item_ -// << ", obj_=" << iteminfo->obj_ << ")" <pimpl_ +// << ", item_=" << iteminfo->item_ +// << ", obj_=" << iteminfo->obj_ << ")" <pimpl_->owner_; - MenuItem const * item = iteminfo->item_; + MenuItem const * item = iteminfo->item_.get(); if (button == 1) { // set the pseudo menu-button @@ -430,36 +560,40 @@ void Menubar::Pimpl::MenuCallback(FL_OBJECT * ob, long button) } // Paranoia check - Assert(item->kind() == MenuItem::Submenu); - + lyx::Assert(item->kind() == MenuItem::Submenu); + // set tabstop length fl_set_tabstop(menu_tabstop); - std::vector submenus; - StrPool strpool; + vector submenus; int menu = iteminfo->pimpl_-> - create_submenu(FL_ObjWin(ob), view, - item->submenu(), - submenus, strpool); + create_submenu(FL_ObjWin(ob), view, + item->submenu(), submenus); if (menu != -1) { // place popup - fl_setpup_position(view->getForm()->x + ob->x, - view->getForm()->y + ob->y + ob->h + 10); + fl_setpup_position(static_cast(view)->getForm()->x + ob->x, + static_cast(view)->getForm()->y + ob->y + ob->h + 10); int choice = fl_dopup(menu); - + if (button == 1) { // set the pseudo menu-button back fl_set_object_boxtype(ob, FL_FLAT_BOX); fl_redraw_object(ob); } - - if (choice >= 1) { - view->getLyXFunc()->Dispatch(choice); + + // If the action value is too low, then it is not a + // valid action, but something else. + if (choice >= action_offset + 1) { + view->getLyXFunc()->verboseDispatch(choice - action_offset, true); + } else { + lyxerr[Debug::GUI] + << "MenuCallback: ignoring bogus action " + << choice << endl; } - } - else + } else { lyxerr << "Error in MenuCallback" << endl; - - std::for_each(submenus.begin(), submenus.end(), fl_freepup); + } + + for_each(submenus.begin(), submenus.end(), fl_freepup); // restore tabstop length fl_set_tabstop(default_tabstop);