]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/qt4/GuiApplication.cpp
Amend f441590c
[lyx.git] / src / frontends / qt4 / GuiApplication.cpp
index f03776caca6baf27483a11ac5e76262156c1832c..4b6e769ddf89d1feba72f20a3ec72c700f9eb982 100644 (file)
@@ -249,13 +249,13 @@ vector<string> loadableImageFormats()
 
 namespace {
 
-struct PngMap {
+struct ImgMap {
        QString key;
        QString value;
 };
 
 
-bool operator<(PngMap const & lhs, PngMap const & rhs)
+bool operator<(ImgMap const & lhs, ImgMap const & rhs)
 {
        return lhs.key < rhs.key;
 }
@@ -264,7 +264,7 @@ bool operator<(PngMap const & lhs, PngMap const & rhs)
 class CompareKey {
 public:
        CompareKey(QString const & name) : name_(name) {}
-       bool operator()(PngMap const & other) const { return other.key == name_; }
+       bool operator()(ImgMap const & other) const { return other.key == name_; }
 private:
        QString const name_;
 };
@@ -274,7 +274,7 @@ private:
 // Upper case comes before lower case
 // Please don't change the formatting, this list is parsed by
 // development/tools/generate_symbols_images.py.
-PngMap sorted_png_map[] = {
+ImgMap sorted_img_map[] = {
        { "Arrownot", "arrownot2"},
        { "Arrowvert", "arrowvert2"},
        { "Bowtie", "bowtie2" },
@@ -370,17 +370,16 @@ PngMap sorted_png_map[] = {
 };
 
 
-size_t const nr_sorted_png_map = sizeof(sorted_png_map) / sizeof(PngMap);
+size_t const nr_sorted_img_map = sizeof(sorted_img_map) / sizeof(ImgMap);
 
 // This list specifies which system's theme icon is related to which lyx
 // command. It was based on:
 // http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html
 // this must be sorted alphabetically
 // Upper case comes before lower case
-PngMap sorted_theme_icon_map[] = {
+ImgMap sorted_theme_icon_map[] = {
        { "bookmark-goto 0", "go-jump" },
        { "buffer-new", "document-new" },
-       { "buffer-print", "document-print" },
        { "buffer-write", "document-save" },
        { "buffer-write-as", "document-save-as" },
        { "buffer-zoom-in", "zoom-in" },
@@ -389,7 +388,6 @@ PngMap sorted_theme_icon_map[] = {
        { "cut", "edit-cut" },
        { "depth-decrement", "format-indent-less" },
        { "depth-increment", "format-indent-more" },
-       { "dialog-show print", "document-print" },
        { "dialog-show spellchecker", "tools-check-spelling" },
        { "dialog-show-new-inset graphics", "insert-image" },
        { "dialog-toggle findreplaceadv", "edit-find-replace" },
@@ -406,42 +404,42 @@ PngMap sorted_theme_icon_map[] = {
        { "window-new", "window-new" }
 };
 
-size_t const nr_sorted_theme_icon_map = sizeof(sorted_theme_icon_map) / sizeof(PngMap);
+size_t const nr_sorted_theme_icon_map = sizeof(sorted_theme_icon_map) / sizeof(ImgMap);
 
 
-QString findPng(QString const & name)
+QString findImg(QString const & name)
 {
-       PngMap const * const begin = sorted_png_map;
-       PngMap const * const end = begin + nr_sorted_png_map;
+       ImgMap const * const begin = sorted_img_map;
+       ImgMap const * const end = begin + nr_sorted_img_map;
        LATTEST(sorted(begin, end));
 
-       PngMap const * const it = find_if(begin, end, CompareKey(name));
+       ImgMap const * const it = find_if(begin, end, CompareKey(name));
 
-       QString png_name;
+       QString img_name;
        if (it != end) {
-               png_name = it->value;
+               img_name = it->value;
        } else {
-               png_name = name;
-               png_name.replace('_', "underscore");
-               png_name.replace(' ', '_');
+               img_name = name;
+               img_name.replace('_', "underscore");
+               img_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");
+               img_name.replace('(', "lparen");
+               img_name.replace(')', "rparen");
+               img_name.replace('[', "lbracket");
+               img_name.replace(']', "rbracket");
+               img_name.replace('{', "lbrace");
+               img_name.replace('}', "rbrace");
+               img_name.replace('|', "bars");
+               img_name.replace(',', "thinspace");
+               img_name.replace(':', "mediumspace");
+               img_name.replace(';', "thickspace");
+               img_name.replace('!', "negthinspace");
        }
 
-       LYXERR(Debug::GUI, "findPng(" << name << ")\n"
-               << "Looking for math PNG called \"" << png_name << '"');
-       return png_name;
+       LYXERR(Debug::GUI, "findImg(" << name << ")\n"
+               << "Looking for math icon called \"" << img_name << '"');
+       return img_name;
 }
 
 } // namespace anon
@@ -449,11 +447,11 @@ QString findPng(QString const & name)
 
 QString themeIconName(QString const & action)
 {
-       PngMap const * const begin = sorted_theme_icon_map;
-       PngMap const * const end = begin + nr_sorted_theme_icon_map;
+       ImgMap const * const begin = sorted_theme_icon_map;
+       ImgMap const * const end = begin + nr_sorted_theme_icon_map;
        LASSERT(sorted(begin, end), /**/);
 
-       PngMap const * const it = find_if(begin, end, CompareKey(action));
+       ImgMap const * const it = find_if(begin, end, CompareKey(action));
 
        if (it != end)
                return it->value;
@@ -471,13 +469,13 @@ QString iconName(FuncRequest const & f, bool unknown)
        case LFUN_MATH_INSERT:
                if (!f.argument().empty()) {
                        path = "math/";
-                       name1 = findPng(toqstr(f.argument()).mid(1));
+                       name1 = findImg(toqstr(f.argument()).mid(1));
                }
                break;
        case LFUN_MATH_DELIM:
        case LFUN_MATH_BIGDELIM:
                path = "math/";
-               name1 = findPng(toqstr(f.argument()));
+               name1 = findImg(toqstr(f.argument()));
                break;
        case LFUN_CALL:
                path = "commands/";
@@ -560,28 +558,47 @@ QString iconName(FuncRequest const & f, bool unknown)
                FileName fname = imageLibFileSearch(imagedir, "unknown", "svgz,png", mode);
                if (fname.exists())
                        return toqstr(fname.absFileName());
-               return QString(":/images/unknown.png");
+               return QString(":/images/unknown.svgz");
        }
 
        return QString();
 }
 
+
+bool getPixmap(QPixmap & pixmap, QString const & path)
+{
+       if (pixmap.load(path)) {
+#if QT_VERSION >= 0x050000
+               if (path.endsWith(".svgz") || path.endsWith(".svg") ) {
+                       GuiApplication const * guiApp = theGuiApp();
+                       if (guiApp != 0) {
+                               pixmap.setDevicePixelRatio(guiApp->pixelRatio());
+                       }
+               }
+#endif
+               return true;
+       }
+       return false;
+}
+
+
 QPixmap getPixmap(QString const & path, QString const & name, QString const & ext)
 {
-       QPixmap pixmap;
        QString imagedir = path;
        FileName fname = imageLibFileSearch(imagedir, name, ext, theGuiApp()->imageSearchMode());
        QString fpath = toqstr(fname.absFileName());
+       QPixmap pixmap = QPixmap();
 
-       if (pixmap.load(fpath)) {
+       if (getPixmap(pixmap, fpath)) {
                return pixmap;
-       } else {
-           QStringList exts = ext.split(",");
-           fpath = ":/" + path + name + ".";
-           for (int i = 0; i < exts.size(); ++i) {
-               if (pixmap.load(fpath + exts.at(i)))
+       }
+       
+       QStringList exts = ext.split(",");
+       fpath = ":/" + path + name + ".";
+       for (int i = 0; i < exts.size(); ++i) {
+               if (getPixmap(pixmap, fpath + exts.at(i))) {
                        return pixmap;
-           }
+               }
        }
 
        bool const list = ext.contains(",");
@@ -592,6 +609,7 @@ QPixmap getPixmap(QString const & path, QString const & name, QString const & ex
        return QPixmap();
 }
 
+
 QIcon getIcon(FuncRequest const & f, bool unknown)
 {
 #if (QT_VERSION >= 0x040600)
@@ -610,13 +628,13 @@ QIcon getIcon(FuncRequest const & f, bool unknown)
                return QIcon();
 
        //LYXERR(Debug::GUI, "Found icon: " << icon);
-       QPixmap pm;
-       if (!pm.load(icon)) {
+       QPixmap pixmap = QPixmap();
+       if (!getPixmap(pixmap,icon)) {
                LYXERR0("Cannot load icon " << icon << " please verify resource system!");
                return QIcon();
        }
 
-       return QIcon(pm);
+       return QIcon(pixmap);
 }
 
 
@@ -945,6 +963,9 @@ struct GuiApplication::Private
        ///
        KeyModifier meta_fake_bit;
 
+       /// The result of last dispatch action
+       DispatchResult dispatch_result_;
+
        /// Multiple views container.
        /**
        * Warning: This must not be a smart pointer as the destruction of the
@@ -1095,7 +1116,7 @@ docstring Application::iconName(FuncRequest const & f, bool unknown)
 
 docstring Application::mathIcon(docstring const & c)
 {
-       return qstring_to_ucs4(findPng(toqstr(c)));
+       return qstring_to_ucs4(findImg(toqstr(c)));
 }
 
 
@@ -1352,7 +1373,7 @@ static docstring makeDispatchMessage(docstring const & msg,
 }
 
 
-void GuiApplication::dispatch(FuncRequest const & cmd)
+DispatchResult const & GuiApplication::dispatch(FuncRequest const & cmd)
 {
        Buffer * buffer = 0;
        if (current_view_ && current_view_->currentBufferView()) {
@@ -1372,6 +1393,9 @@ void GuiApplication::dispatch(FuncRequest const & cmd)
        // the buffer may have been closed by one action
        if (theBufferList().isLoaded(buffer))
                buffer->undo().endUndoGroup();
+
+       d->dispatch_result_ = dr;
+       return d->dispatch_result_;
 }
 
 
@@ -1711,6 +1735,8 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                }
                // Make sure we don't keep old colors in cache.
                d->color_cache_.clear();
+               // Update the current view
+               lyx::dispatch(FuncRequest(LFUN_SCREEN_FONT_UPDATE));
                break;
        }
 
@@ -2012,6 +2038,7 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                        if (current_view_ == 0)
                                createView();
                }
+               // fall through
        }
 
        default:
@@ -2133,9 +2160,15 @@ void GuiApplication::processKeySym(KeySymbol const & keysym, KeyModifier state)
                        // must not be inserted (#5704)
                        if (!isPrintable(encoded_last_key)) {
                                LYXERR(Debug::KEY, "Non-printable character! Omitting.");
-                               current_view_->restartCursor();
+                               if (current_view_)
+                                       current_view_->restartCursor();
                                return;
                        }
+                       // The following modifier check is not needed on Mac.
+                       // The keysym is either not text or it is different
+                       // from the non-modifier keysym. See #9875 for the
+                       // broken alt-modifier effect of having this code active.
+#if !defined(Q_OS_MAC)
                        // If a non-Shift Modifier is used we have a non-bound key sequence
                        // (such as Alt+j = j). This should be omitted (#5575).
                        // On Windows, AltModifier and ControlModifier are both
@@ -2146,11 +2179,15 @@ void GuiApplication::processKeySym(KeySymbol const & keysym, KeyModifier state)
 #if defined(Q_OS_WIN) || defined(Q_CYGWIN_WIN)
                            && !(state & AltModifier && state & ControlModifier)
 #endif
-                           ) {
-                               current_view_->message(_("Unknown function."));
-                               current_view_->restartCursor();
+                           )
+                       {
+                               if (current_view_) {
+                                       current_view_->message(_("Unknown function."));
+                                       current_view_->restartCursor();
+                               }
                                return;
                        }
+#endif
                        // Since all checks above were passed, we now really have text that
                        // is to be inserted (e.g., AltGr-bound symbols). Thus change the
                        // func to LFUN_SELF_INSERT and thus cause the text to be inserted
@@ -2423,6 +2460,9 @@ void GuiApplication::execBatchCommands()
 #ifdef Q_OS_MAC
 #if QT_VERSION > 0x040600
        setAttribute(Qt::AA_MacDontSwapCtrlAndMeta,lyxrc.mac_dontswap_ctrl_meta);
+#endif
+#if QT_VERSION > 0x050100
+       setAttribute(Qt::AA_UseHighDpiPixmaps,true);
 #endif
        // Create the global default menubar which is shown for the dialogs
        // and if no GuiView is visible.
@@ -2442,13 +2482,12 @@ QAbstractItemModel * GuiApplication::languageModel()
 
        QStandardItemModel * lang_model = new QStandardItemModel(this);
        lang_model->insertColumns(0, 3);
-       int current_row;
        QIcon speller(getPixmap("images/", "dialog-show_spellchecker", "svgz,png"));
        QIcon saurus(getPixmap("images/", "thesaurus-entry", "svgz,png"));
        Languages::const_iterator it = lyx::languages.begin();
        Languages::const_iterator end = lyx::languages.end();
        for (; it != end; ++it) {
-               current_row = lang_model->rowCount();
+               int current_row = lang_model->rowCount();
                lang_model->insertRows(current_row, 1);
                QModelIndex pl_item = lang_model->index(current_row, 0);
                QModelIndex sp_item = lang_model->index(current_row, 1);
@@ -2712,10 +2751,11 @@ void GuiApplication::commitData(QSessionManager & sm)
 
 void GuiApplication::unregisterView(GuiView * gv)
 {
-       LAPPERR(d->views_[gv->id()] == gv);
-       d->views_.remove(gv->id());
-       if (current_view_ == gv)
-               current_view_ = 0;
+       if(d->views_.contains(gv->id()) && d->views_.value(gv->id()) == gv) {
+               d->views_.remove(gv->id());
+               if (current_view_ == gv)
+                       current_view_ = 0;
+       }
 }
 
 
@@ -3053,7 +3093,7 @@ bool GuiApplication::x11EventFilter(XEvent * xev)
 }
 #elif defined(QPA_XCB)
 bool GuiApplication::nativeEventFilter(const QByteArray & eventType,
-                                      void * message, long *) Q_DECL_OVERRIDE
+                                      void * message, long *)
 {
        if (!current_view_ || eventType != "xcb_generic_event_t")
                return false;