X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Ffrontends%2Fqt4%2FGuiToolbar.cpp;h=6ff3e9636596977e683264dbda48d617ac5b8085;hb=f1cba8ff64b369792fd49f5ddf90e8126ab476ac;hp=9eb127c7d311d33e04b0f177403f64ab31d24794;hpb=feb96b01aaf238b1bc93313ebd5a66b97e225ee8;p=features.git diff --git a/src/frontends/qt4/GuiToolbar.cpp b/src/frontends/qt4/GuiToolbar.cpp index 9eb127c7d3..6ff3e96365 100644 --- a/src/frontends/qt4/GuiToolbar.cpp +++ b/src/frontends/qt4/GuiToolbar.cpp @@ -3,7 +3,7 @@ * This file is part of LyX, the document processor. * Licence details can be found in the file COPYING. * - * \author Lars Gullik Bjønnes + * \author Lars Gullik Bjønnes * \author John Levon * \author Jean-Marc Lasgouttes * \author Angus Leeming @@ -15,13 +15,16 @@ #include -#include "GuiView.h" -#include "GuiCommandBuffer.h" #include "GuiToolbar.h" -#include "LyXAction.h" + #include "Action.h" -#include "qt_helpers.h" +#include "GuiApplication.h" +#include "GuiCommandBuffer.h" +#include "GuiView.h" +#include "IconPalette.h" #include "InsertTableWidget.h" +#include "qt_helpers.h" +#include "Toolbars.h" #include "Buffer.h" #include "BufferParams.h" @@ -29,19 +32,16 @@ #include "Cursor.h" #include "FuncRequest.h" #include "FuncStatus.h" -#include "IconPalette.h" #include "Layout.h" #include "LyXFunc.h" #include "LyXRC.h" #include "Paragraph.h" #include "TextClass.h" -#include "ToolbarBackend.h" #include "support/debug.h" #include "support/filetools.h" #include "support/gettext.h" #include "support/lstrings.h" -#include "support/lyxalgo.h" // sorted #include #include @@ -49,252 +49,92 @@ #include #include #include +#include #include #include #include #include #include +#include #include #include #include +#include #include +#include #include #include #include -#include - -#include -#include +#include "support/lassert.h" using namespace std; using namespace lyx::support; -static void initializeResources() -{ - static bool initialized = false; - if (!initialized) { - Q_INIT_RESOURCE(Resources); - initialized = true; - } -} - - namespace lyx { namespace frontend { -namespace { - -struct PngMap { - QString key; - QString value; -}; - - -bool operator<(PngMap const & lhs, PngMap const & rhs) -{ - return lhs.key < rhs.key; -} - - -class CompareKey { -public: - CompareKey(QString const & name) : name_(name) {} - bool operator()(PngMap const & other) const { return other.key == name_; } -private: - QString const name_; -}; - - -PngMap sorted_png_map[] = { - { "Bumpeq", "bumpeq2" }, - { "Cap", "cap2" }, - { "Cup", "cup2" }, - { "Delta", "delta2" }, - { "Downarrow", "downarrow2" }, - { "Gamma", "gamma2" }, - { "Lambda", "lambda2" }, - { "Leftarrow", "leftarrow2" }, - { "Leftrightarrow", "leftrightarrow2" }, - { "Longleftarrow", "longleftarrow2" }, - { "Longleftrightarrow", "longleftrightarrow2" }, - { "Longrightarrow", "longrightarrow2" }, - { "Omega", "omega2" }, - { "Phi", "phi2" }, - { "Pi", "pi2" }, - { "Psi", "psi2" }, - { "Rightarrow", "rightarrow2" }, - { "Sigma", "sigma2" }, - { "Subset", "subset2" }, - { "Supset", "supset2" }, - { "Theta", "theta2" }, - { "Uparrow", "uparrow2" }, - { "Updownarrow", "updownarrow2" }, - { "Upsilon", "upsilon2" }, - { "Vdash", "vdash3" }, - { "Xi", "xi2" }, - { "nLeftarrow", "nleftarrow2" }, - { "nLeftrightarrow", "nleftrightarrow2" }, - { "nRightarrow", "nrightarrow2" }, - { "nVDash", "nvdash3" }, - { "nvDash", "nvdash2" }, - { "textrm \\AA", "textrm_AA"}, - { "textrm \\O", "textrm_O"}, - { "vDash", "vdash2" } -}; - -size_t const nr_sorted_png_map = sizeof(sorted_png_map) / sizeof(PngMap); - - -QString findPng(QString const & name) -{ - PngMap const * const begin = sorted_png_map; - PngMap const * const end = begin + nr_sorted_png_map; - BOOST_ASSERT(sorted(begin, end)); - - PngMap const * const it = find_if(begin, end, CompareKey(name)); - - QString png_name; - if (it != end) { - png_name = it->value; - } else { - png_name = name; - png_name.replace('_', "underscore"); - png_name.replace(' ', '_'); - - // This way we can have "math-delim { }" on the toolbar. - png_name.replace('(', "lparen"); - png_name.replace(')', "rparen"); - png_name.replace('[', "lbracket"); - png_name.replace(']', "rbracket"); - png_name.replace('{', "lbrace"); - png_name.replace('}', "rbrace"); - png_name.replace('|', "bars"); - png_name.replace(',', "thinspace"); - png_name.replace(':', "mediumspace"); - png_name.replace(';', "thickspace"); - png_name.replace('!', "negthinspace"); - } - - LYXERR(Debug::GUI, "findPng(" << fromqstr(name) << ")\n" - << "Looking for math PNG called \"" << fromqstr(png_name) << '"'); - return png_name; -} - -} // namespace anon - - -/// return a icon for the given action -static QIcon getIcon(FuncRequest const & f, bool unknown) -{ - initializeResources(); - QPixmap pm; - QString name1; - QString name2; - QString path; - switch (f.action) { - case LFUN_MATH_INSERT: - if (!f.argument().empty()) { - path = "math/"; - name1 = findPng(toqstr(f.argument()).mid(1)); - } - break; - case LFUN_MATH_DELIM: - case LFUN_MATH_BIGDELIM: - path = "math/"; - name1 = findPng(toqstr(f.argument())); - break; - case LFUN_CALL: - path = "commands/"; - name1 = toqstr(f.argument()); - break; - default: - name2 = toqstr(lyxaction.getActionName(f.action)); - name1 = name2; - - if (!f.argument().empty()) { - name1 = name2 + ' ' + toqstr(f.argument()); - name1.replace(' ', '_'); - } - } - - string fullname = libFileSearch("images/" + path, name1, "png").absFilename(); - if (pm.load(toqstr(fullname))) - return pm; - - fullname = libFileSearch("images/" + path, name2, "png").absFilename(); - if (pm.load(toqstr(fullname))) - return pm; - - if (pm.load(":/images/" + path + name1 + ".png")) - return pm; - - if (pm.load(":/images/" + path + name2 + ".png")) - return pm; - - LYXERR(Debug::GUI, "Cannot find icon for command \"" - << lyxaction.getActionName(f.action) - << '(' << to_utf8(f.argument()) << ")\""); - if (unknown) - pm.load(":/images/unknown.png"); - - return pm; -} - - ///////////////////////////////////////////////////////////////////// // // GuiLayoutBox // ///////////////////////////////////////////////////////////////////// -class LayoutItemDelegate : public QAbstractItemDelegate { +class LayoutItemDelegate : public QItemDelegate { public: /// explicit LayoutItemDelegate(QObject * parent = 0) - : QAbstractItemDelegate(parent) + : QItemDelegate(parent) {} /// void paint(QPainter * painter, QStyleOptionViewItem const & option, QModelIndex const & index) const { - QComboBox * combo = static_cast(parent()); - QSortFilterProxyModel const * model - = static_cast(index.model()); - QStyleOptionMenuItem opt = getStyleOption(option, index); - - painter->eraseRect(opt.rect); + QStyleOptionViewItem opt = option; - QString text = underlineFilter(opt.text); - opt.text = QString(); + // default background + painter->fillRect(opt.rect, opt.palette.color(QPalette::Base)); // category header? if (lyxrc.group_layouts) { + QSortFilterProxyModel const * model + = static_cast(index.model()); + QString stdCat = category(*model->sourceModel(), 0); QString cat = category(*index.model(), index.row()); // not the standard layout and not the same as in the previous line? if (stdCat != cat && (index.row() == 0 || cat != category(*index.model(), index.row() - 1))) { + painter->save(); + + // draw unselected background + QStyle::State state = opt.state; + opt.state = opt.state & ~QStyle::State_Selected; + drawBackground(painter, opt, index); + opt.state = state; + // draw category header - paintBackground(painter, opt); - paintCategoryHeader(painter, opt, + drawCategoryHeader(painter, opt, category(*index.model(), index.row())); - QFontMetrics fm(opt.font); + // move rect down below header opt.rect.setTop(opt.rect.top() + headerHeight(opt)); - opt.menuRect = opt.rect; + + painter->restore(); } } - // Draw using the menu item style (this is how QComboBox does it). - // But for the rich text drawing below we will call it with an - // empty string, and later then draw over it the real string. - painter->save(); - combo->style()->drawControl(QStyle::CE_MenuItem, &opt, painter, combo->view()); - painter->restore(); - + QItemDelegate::paint(painter, opt, index); + } + + /// + void drawDisplay(QPainter * painter, QStyleOptionViewItem const & opt, + const QRect & /*rect*/, const QString & text ) const + { + QString utext = underlineFilter(text); + // Draw the rich text. painter->save(); QColor col = opt.palette.text().color(); @@ -305,23 +145,27 @@ public: QTextDocument doc; doc.setDefaultFont(opt.font); - doc.setHtml(text); - painter->translate(opt.rect.x() + 5, opt.rect.y()); + doc.setHtml(utext); + + QTextFrameFormat fmt = doc.rootFrame()->frameFormat(); + fmt.setMargin(0); + doc.rootFrame()->setFrameFormat(fmt); + + painter->translate(opt.rect.x() + 5, + opt.rect.y() + (opt.rect.height() - opt.fontMetrics.height()) / 2); doc.documentLayout()->draw(painter, context); painter->restore(); } /// - QSize sizeHint(QStyleOptionViewItem const & option, + QSize sizeHint(QStyleOptionViewItem const & opt, QModelIndex const & index) const { GuiLayoutBox * combo = static_cast(parent()); QSortFilterProxyModel const * model - = static_cast(index.model()); - QStyleOptionMenuItem opt = getStyleOption(option, index); - QSize size = combo->style()->sizeFromContents( - QStyle::CT_MenuItem, &opt, option.rect.size(), combo); - + = static_cast(index.model()); + QSize size = QItemDelegate::sizeHint(opt, index); + /// QComboBox uses the first row height to estimate the /// complete popup height during QComboBox::showPopup(). /// To avoid scrolling we have to sneak in space for the headers. @@ -359,38 +203,16 @@ private: { return model.data(model.index(row, 2), Qt::DisplayRole).toString(); } - - /// - void paintBackground(QPainter * painter, QStyleOptionMenuItem const & opt) const - { - QComboBox * combo = static_cast(parent()); - - // we only want to paint a background using the style, so - // disable every thing else - QStyleOptionMenuItem sopt = opt; - sopt.menuRect = sopt.rect; - sopt.state = QStyle::State_Active | QStyle::State_Enabled; - sopt.checked = false; - sopt.text = QString(); - painter->save(); - combo->style()->drawControl(QStyle::CE_MenuItem, &sopt, - painter, combo->view()); - painter->restore(); - } - /// - int headerHeight(QStyleOptionMenuItem const & opt) const + int headerHeight(QStyleOptionViewItem const & opt) const { - QFontMetrics fm(opt.font); - return fm.height() * 8 / 10; + return opt.fontMetrics.height() * 8 / 10; } /// - void paintCategoryHeader(QPainter * painter, QStyleOptionMenuItem const & opt, + void drawCategoryHeader(QPainter * painter, QStyleOptionViewItem const & opt, QString const & category) const { - painter->save(); - // slightly blended color QColor lcol = opt.palette.text().color(); lcol.setAlpha(127); @@ -421,8 +243,6 @@ private: painter->drawLine(right + 1, ymid, opt.rect.right(), ymid); } else painter->drawLine(opt.rect.x(), ymid, opt.rect.right(), ymid); - - painter->restore(); } @@ -441,7 +261,7 @@ private: p->filter(); for (int i = 0; i < f.length(); ++i) { int p = s.indexOf(f[i], lastp + 1, Qt::CaseInsensitive); - BOOST_ASSERT(p != -1); + LASSERT(p != -1, /**/); if (lastp == p - 1 && lastp != -1) { // remove ")" and append "x)" r = r.left(r.length() - 4) + s[p] + ""; @@ -455,31 +275,6 @@ private: r += s.mid(lastp + 1); return r; } - - /// - QStyleOptionMenuItem getStyleOption(QStyleOptionViewItem const & option, - QModelIndex const & index) const - { - QComboBox * combo = static_cast(parent()); - - // create the options for a menu item - QStyleOptionMenuItem menuOption; - menuOption.palette = QApplication::palette("QMenu"); - menuOption.state = QStyle::State_Active | QStyle::State_Enabled; - menuOption.menuRect = option.rect; - menuOption.rect = option.rect; - menuOption.font = combo->font(); - menuOption.fontMetrics = QFontMetrics(menuOption.font); - menuOption.tabWidth = 0; - menuOption.text = index.model()->data(index, Qt::DisplayRole).toString() - .replace(QLatin1Char('&'), QLatin1String("&&")); - menuOption.menuItemType = QStyleOptionMenuItem::Normal; - if (option.state & QStyle::State_Selected) - menuOption.state |= QStyle::State_Selected; - menuOption.checkType = QStyleOptionMenuItem::NotCheckable; - menuOption.checked = false; - return menuOption; - } }; @@ -499,8 +294,9 @@ public: }; -GuiLayoutBox::GuiLayoutBox(GuiView & owner) - : owner_(owner), lastSel_(-1), layoutItemDelegate_(new LayoutItemDelegate(this)), +GuiLayoutBox::GuiLayoutBox(GuiToolbar * bar, GuiView & owner) + : owner_(owner), bar_(bar), lastSel_(-1), + layoutItemDelegate_(new LayoutItemDelegate(this)), visibleCategories_(0), inShowPopup_(false) { setSizeAdjustPolicy(QComboBox::AdjustToContents); @@ -521,7 +317,10 @@ GuiLayoutBox::GuiLayoutBox(GuiView & owner) view()->setItemDelegateForColumn(0, layoutItemDelegate_); QObject::connect(this, SIGNAL(activated(int)), - this, SLOT(selected(int))); + this, SLOT(selected(int))); + QObject::connect(bar_, SIGNAL(iconSizeChanged(QSize)), + this, SLOT(setIconSize(QSize))); + owner_.setLayoutDialog(this); updateContents(true); } @@ -529,12 +328,6 @@ GuiLayoutBox::GuiLayoutBox(GuiView & owner) void GuiLayoutBox::setFilter(QString const & s) { - if (!s.isEmpty()) - owner_.message(_("Filtering layouts with \"" + fromqstr(s) + "\". " - "Press ESC to remove filter.")); - else - owner_.message(_("Enter characters to filter the layout list.")); - bool enabled = view()->updatesEnabled(); view()->setUpdatesEnabled(false); @@ -563,13 +356,20 @@ void GuiLayoutBox::setFilter(QString const & s) // We do not call our implementation of showPopup because that // would reset the filter again. This is only needed if the user clicks // on the QComboBox. - BOOST_ASSERT(!inShowPopup_); + LASSERT(!inShowPopup_, /**/); inShowPopup_ = true; QComboBox::showPopup(); inShowPopup_ = false; // The item delegate hack is off again. So trigger a relayout of the popup. filterModel_->triggerLayoutChange(); + + if (!s.isEmpty()) + owner_.message(bformat(_("Filtering layouts with \"%1$s\". " + "Press ESC to remove filter."), + qstring_to_ucs4(s))); + else + owner_.message(_("Enter characters to filter the layout list.")); } view()->setUpdatesEnabled(enabled); @@ -627,7 +427,7 @@ void GuiLayoutBox::showPopup() // call QComboBox::showPopup. But set the inShowPopup_ flag to switch on // the hack in the item delegate to make space for the headers. - BOOST_ASSERT(!inShowPopup_); + LASSERT(!inShowPopup_, /**/); inShowPopup_ = true; QComboBox::showPopup(); inShowPopup_ = false; @@ -687,6 +487,18 @@ bool GuiLayoutBox::eventFilter(QObject * o, QEvent * e) return QComboBox::eventFilter(o, e); } + +void GuiLayoutBox::setIconSize(QSize size) +{ +#ifdef Q_WS_MACX + bool small = size.height() < 20; + setAttribute(Qt::WA_MacSmallSize, small); + setAttribute(Qt::WA_MacNormalSize, !small); +#else + (void)size; // suppress warning +#endif +} + void GuiLayoutBox::set(docstring const & layout) { @@ -695,14 +507,21 @@ void GuiLayoutBox::set(docstring const & layout) if (!text_class_) return; - QString const & name = toqstr((*text_class_)[layout].name()); - if (name == currentText()) + Layout const & lay = (*text_class_)[layout]; + QString const newLayout = toqstr(lay.name()); + + int const curItem = currentIndex(); + QModelIndex const mindex = + filterModel_->mapToSource(filterModel_->index(curItem, 1)); + QString const & currentLayout = model_->itemFromIndex(mindex)->text(); + if (newLayout == currentLayout) { + LYXERR(Debug::GUI, "Already had " << newLayout << " selected."); return; + } - QList r = model_->findItems(name, Qt::MatchExactly, 1); + QList r = model_->findItems(newLayout, Qt::MatchExactly, 1); if (r.empty()) { - lyxerr << "Trying to select non existent layout type " - << fromqstr(name) << endl; + LYXERR0("Trying to select non existent layout type " << newLayout); return; } @@ -711,10 +530,12 @@ void GuiLayoutBox::set(docstring const & layout) void GuiLayoutBox::addItemSort(docstring const & item, docstring const & category, - bool sorted, bool sortedByCat) + bool sorted, bool sortedByCat, bool unknown) { QString qitem = toqstr(item); - QString titem = toqstr(translateIfPossible(item)); + // FIXME This is wrong for RTL, I'd suppose. + QString titem = toqstr(translateIfPossible(item) + + (unknown ? _(" (unknown)") : from_ascii(""))); QString qcat = toqstr(translateIfPossible(category)); QList row; @@ -756,7 +577,7 @@ void GuiLayoutBox::addItemSort(docstring const & item, docstring const & categor if (i < end) { // find alphabetic position while (i != end - && model_->item(i, 0)->text().compare(titem) < 0 + && model_->item(i, 0)->text().localeAwareCompare(titem) < 0 && (!sortedByCat || model_->item(i, 2)->text() == qcat)) ++i; } @@ -782,7 +603,7 @@ void GuiLayoutBox::updateContents(bool reset) // or we've moved from one inset to another DocumentClass const * text_class = &buffer->params().documentClass(); Inset const * inset = - owner_.view()->cursor().innerParagraph().inInset(); + &(owner_.view()->cursor().innerParagraph().inInset()); if (!reset && text_class_ == text_class && inset_ == inset) { set(owner_.view()->cursor().innerParagraph().layout().name()); return; @@ -794,18 +615,22 @@ void GuiLayoutBox::updateContents(bool reset) model_->clear(); DocumentClass::const_iterator lit = text_class_->begin(); DocumentClass::const_iterator len = text_class_->end(); + for (; lit != len; ++lit) { docstring const & name = lit->name(); + bool const useEmpty = inset_->forcePlainLayout() || inset_->usePlainLayout(); // if this inset requires the empty layout, we skip the default // layout - if (name == text_class_->defaultLayoutName() && inset && - (inset->forceEmptyLayout() || inset->useEmptyLayout())) + if (name == text_class_->defaultLayoutName() && inset_ && useEmpty) continue; // if it doesn't require the empty layout, we skip it - if (name == text_class_->emptyLayoutName() && inset && - !inset->forceEmptyLayout() && !inset->useEmptyLayout()) + if (name == text_class_->plainLayoutName() && inset_ && !useEmpty) + continue; + // obsoleted layouts are skipped as well + if (!lit->obsoleted_by().empty()) continue; - addItemSort(name, lit->category(), lyxrc.sort_layouts, lyxrc.group_layouts); + addItemSort(name, lit->category(), lyxrc.sort_layouts, + lyxrc.group_layouts, lit->isUnknown()); } set(owner_.view()->cursor().innerParagraph().layout().name()); @@ -814,7 +639,8 @@ void GuiLayoutBox::updateContents(bool reset) // needed to recalculate size hint hide(); setMinimumWidth(sizeHint().width()); - setEnabled(!buffer->isReadonly()); + setEnabled(!buffer->isReadonly() && + lyx::getStatus(FuncRequest(LFUN_LAYOUT)).enabled()); show(); } @@ -823,9 +649,7 @@ void GuiLayoutBox::selected(int index) { // get selection QModelIndex mindex = filterModel_->mapToSource(filterModel_->index(index, 1)); - docstring const layoutName = - qstring_to_ucs4(model_->itemFromIndex(mindex)->text()); - + docstring layoutName = qstring_to_ucs4(model_->itemFromIndex(mindex)->text()); owner_.setFocus(); if (!text_class_) { @@ -843,7 +667,7 @@ void GuiLayoutBox::selected(int index) resetFilter(); return; } - lyxerr << "ERROR (layoutSelected): layout not found!" << endl; + LYXERR0("ERROR (layoutSelected): layout " << layoutName << " not found!"); } @@ -856,27 +680,53 @@ void GuiLayoutBox::selected(int index) GuiToolbar::GuiToolbar(ToolbarInfo const & tbinfo, GuiView & owner) - : QToolBar(qt_(tbinfo.gui_name), &owner), owner_(owner), - layout_(0), command_buffer_(0) + : QToolBar(toqstr(tbinfo.gui_name), &owner), visibility_(0), + allowauto_(false), owner_(owner), layout_(0), command_buffer_(0), + tbinfo_(tbinfo), filled_(false) { - // give visual separation between adjacent toolbars - addSeparator(); + setIconSize(owner.iconSize()); + connect(&owner, SIGNAL(iconSizeChanged(QSize)), this, + SLOT(setIconSize(QSize))); - // TODO: save toolbar position + // Toolbar dragging is allowed. setMovable(true); + // This is used by QMainWindow::restoreState for proper main window state + // restauration. + setObjectName(toqstr(tbinfo.name)); + restoreSession(); +} - ToolbarInfo::item_iterator it = tbinfo.items.begin(); - ToolbarInfo::item_iterator end = tbinfo.items.end(); + +void GuiToolbar::fill() +{ + if (filled_) + return; + ToolbarInfo::item_iterator it = tbinfo_.items.begin(); + ToolbarInfo::item_iterator end = tbinfo_.items.end(); for (; it != end; ++it) - add(*it); + add(*it); + filled_ = true; +} + + +void GuiToolbar::showEvent(QShowEvent * ev) +{ + fill(); + ev->accept(); +} + + +void GuiToolbar::setVisibility(int visibility) +{ + visibility_ = visibility; + allowauto_ = visibility_ >= Toolbars::MATH; } Action * GuiToolbar::addItem(ToolbarItem const & item) { - Action * act = new Action(owner_, - getIcon(item.func_, false), - toqstr(item.label_), item.func_, toqstr(item.label_)); + Action * act = new Action(&owner_, getIcon(item.func_, false), + toqstr(item.label_), item.func_, toqstr(item.label_), this); actions_.append(act); return act; } @@ -900,8 +750,7 @@ public: connect(bar_, SIGNAL(iconSizeChanged(QSize)), this, SLOT(setIconSize(QSize))); setCheckable(true); - ToolbarInfo const * tbinfo = - toolbarbackend.getDefinedToolbarInfo(tbitem_.name_); + ToolbarInfo const * tbinfo = guiApp->toolbars().info(tbitem_.name_); if (tbinfo) // use the icon of first action for the toolbar button setIcon(getIcon(tbinfo->items.begin()->func_, true)); @@ -916,10 +765,9 @@ public: initialized_ = true; - ToolbarInfo const * tbinfo = - toolbarbackend.getDefinedToolbarInfo(tbitem_.name_); + ToolbarInfo const * tbinfo = guiApp->toolbars().info(tbitem_.name_); if (!tbinfo) { - lyxerr << "Unknown toolbar " << tbitem_.name_ << endl; + LYXERR0("Unknown toolbar " << tbitem_.name_); return; } IconPalette * panel = new IconPalette(this); @@ -971,10 +819,9 @@ public: m->setWindowTitle(label); m->setTearOffEnabled(true); connect(bar_, SIGNAL(updated()), m, SLOT(updateParent())); - ToolbarInfo const * tbinfo = - toolbarbackend.getDefinedToolbarInfo(tbitem_.name_); + ToolbarInfo const * tbinfo = guiApp->toolbars().info(tbitem_.name_); if (!tbinfo) { - lyxerr << "Unknown toolbar " << tbitem_.name_ << endl; + LYXERR0("Unknown toolbar " << tbitem_.name_); return; } ToolbarInfo::item_iterator it = tbinfo->items.begin(); @@ -998,7 +845,7 @@ void GuiToolbar::add(ToolbarItem const & item) addSeparator(); break; case ToolbarItem::LAYOUTS: - layout_ = new GuiLayoutBox(owner_); + layout_ = new GuiLayoutBox(this, owner_); addWidget(layout_); break; case ToolbarItem::MINIBUFFER: @@ -1041,42 +888,21 @@ void GuiToolbar::add(ToolbarItem const & item) } -void GuiToolbar::saveInfo(ToolbarSection::ToolbarInfo & tbinfo) +void GuiToolbar::update(bool in_math, bool in_table, bool in_review, + bool in_mathmacrotemplate) { - // if tbinfo.state == auto *do not* set on/off - if (tbinfo.state != ToolbarSection::ToolbarInfo::AUTO) { - if (GuiToolbar::isVisible()) - tbinfo.state = ToolbarSection::ToolbarInfo::ON; - else - tbinfo.state = ToolbarSection::ToolbarInfo::OFF; + if (visibility_ & Toolbars::AUTO) { + bool show_it = in_math && (visibility_ & Toolbars::MATH) + || in_table && (visibility_ & Toolbars::TABLE) + || in_review && (visibility_ & Toolbars::REVIEW) + || in_mathmacrotemplate && (visibility_ & Toolbars::MATHMACROTEMPLATE); + setVisible(show_it); } - // - // no need to save it here. - Qt::ToolBarArea loc = owner_.toolBarArea(this); - - if (loc == Qt::TopToolBarArea) - tbinfo.location = ToolbarSection::ToolbarInfo::TOP; - else if (loc == Qt::BottomToolBarArea) - tbinfo.location = ToolbarSection::ToolbarInfo::BOTTOM; - else if (loc == Qt::RightToolBarArea) - tbinfo.location = ToolbarSection::ToolbarInfo::RIGHT; - else if (loc == Qt::LeftToolBarArea) - tbinfo.location = ToolbarSection::ToolbarInfo::LEFT; - else - tbinfo.location = ToolbarSection::ToolbarInfo::NOTSET; - - // save toolbar position. They are not used to restore toolbar position - // now because move(x,y) does not work for toolbar. - tbinfo.posx = pos().x(); - tbinfo.posy = pos().y(); -} - -void GuiToolbar::updateContents() -{ // update visible toolbars only if (!isVisible()) return; + // This is a speed bottleneck because this is called on every keypress // and update calls getStatus, which copies the cursor at least two times for (int i = 0; i < actions_.size(); ++i) @@ -1090,7 +916,59 @@ void GuiToolbar::updateContents() } +QString GuiToolbar::sessionKey() const +{ + return "views/" + QString::number(owner_.id()) + "/" + objectName(); +} + + +void GuiToolbar::saveSession() const +{ + QSettings settings; + settings.setValue(sessionKey() + "/visibility", visibility_); +} + + +void GuiToolbar::restoreSession() +{ + QSettings settings; + setVisibility(settings.value(sessionKey() + "/visibility").toInt()); +} + + +void GuiToolbar::toggle() +{ + docstring state; + if (allowauto_) { + if (!(visibility_ & Toolbars::AUTO)) { + visibility_ |= Toolbars::AUTO; + hide(); + state = _("auto"); + } else { + visibility_ &= ~Toolbars::AUTO; + if (isVisible()) { + hide(); + state = _("off"); + } else { + show(); + state = _("on"); + } + } + } else { + if (isVisible()) { + hide(); + state = _("off"); + } else { + show(); + state = _("on"); + } + } + + owner_.message(bformat(_("Toolbar \"%1$s\" state set to %2$s"), + qstring_to_ucs4(windowTitle()), state)); +} + } // namespace frontend } // namespace lyx -#include "GuiToolbar_moc.cpp" +#include "moc_GuiToolbar.cpp"