]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/qt4/GuiPrefs.cpp
Compil fix.
[lyx.git] / src / frontends / qt4 / GuiPrefs.cpp
index 103003be7acb02005445d859da3f06054aa2cdee..40b855a27d25619d44b844b4608083d6fd3c1822 100644 (file)
 
 #include "GuiPrefs.h"
 
-#include "qt_helpers.h"
+#include "ColorCache.h"
+#include "FileDialog.h"
 #include "GuiApplication.h"
+#include "GuiFontExample.h"
 #include "GuiFontLoader.h"
+#include "GuiKeySymbol.h"
+#include "qt_helpers.h"
 
 #include "BufferList.h"
 #include "Color.h"
 #include "ConverterCache.h"
-#include "FileDialog.h"
+#include "FontEnums.h"
 #include "FuncRequest.h"
-#include "GuiFontExample.h"
-#include "GuiKeySymbol.h"
 #include "KeyMap.h"
 #include "KeySequence.h"
+#include "Language.h"
 #include "LyXAction.h"
 #include "PanelStack.h"
 #include "paper.h"
@@ -44,7 +47,9 @@
 
 #include "frontends/alert.h"
 #include "frontends/Application.h"
+#include "frontends/FontLoader.h"
 
+#include <QAbstractItemModel>
 #include <QCheckBox>
 #include <QColorDialog>
 #include <QFontDatabase>
@@ -217,15 +222,6 @@ string const catLanguage = N_("Language Settings");
 string const catOutput = N_("Output");
 string const catFiles = N_("File Handling");
 
-static int findPos_helper(QStringList const & vec, QString const & val)
-{
-       for (int i = 0; i != vec.size(); ++i)
-               if (vec[i] == val)
-                       return i;
-       return 0;
-}
-
-
 static void parseFontName(QString const & mangled0,
        string & name, string & foundry)
 {
@@ -329,8 +325,8 @@ static void setComboxFont(QComboBox * cb, string const & family,
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefPlaintext::PrefPlaintext(QWidget * parent)
-       : PrefModule(qt_(catOutput), qt_("Plain text"), 0, parent)
+PrefPlaintext::PrefPlaintext(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("Plain text"), form)
 {
        setupUi(this);
        connect(plaintextLinelengthSB, SIGNAL(valueChanged(int)),
@@ -360,8 +356,8 @@ void PrefPlaintext::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefDate::PrefDate(QWidget * parent)
-       : PrefModule(qt_(catOutput), qt_("Date format"), 0, parent)
+PrefDate::PrefDate(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("Date format"), form)
 {
        setupUi(this);
        connect(DateED, SIGNAL(textChanged(QString)),
@@ -387,8 +383,8 @@ void PrefDate::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefInput::PrefInput(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catEditing), qt_("Keyboard/Mouse"), form, parent)
+PrefInput::PrefInput(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Keyboard/Mouse"), form)
 {
        setupUi(this);
 
@@ -423,9 +419,9 @@ void PrefInput::update(LyXRC const & rc)
 }
 
 
-QString PrefInput::testKeymap(QString keymap)
+QString PrefInput::testKeymap(QString const & keymap)
 {
-       return form_->browsekbmap(toqstr(internal_path(fromqstr(keymap))));
+       return form_->browsekbmap(internalPath(keymap));
 }
 
 
@@ -462,8 +458,8 @@ void PrefInput::on_keymapCB_toggled(bool keymap)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefCompletion::PrefCompletion(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catEditing), qt_("Input Completion"), form, parent)
+PrefCompletion::PrefCompletion(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Input Completion"), form)
 {
        setupUi(this);
 
@@ -524,8 +520,8 @@ void PrefCompletion::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefLatex::PrefLatex(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catOutput), qt_("LaTeX"), form, parent)
+PrefLatex::PrefLatex(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("LaTeX"), form)
 {
        setupUi(this);
        connect(latexEncodingED, SIGNAL(textChanged(QString)),
@@ -591,8 +587,8 @@ void PrefLatex::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefScreenFonts::PrefScreenFonts(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catLookAndFeel), qt_("Screen fonts"), form, parent)
+PrefScreenFonts::PrefScreenFonts(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("Screen fonts"), form)
 {
        setupUi(this);
 
@@ -765,8 +761,8 @@ struct ColorSorter
 
 } // namespace anon
 
-PrefColors::PrefColors(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catLookAndFeel), qt_("Colors"), form, parent)
+PrefColors::PrefColors(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("Colors"), form)
 {
        setupUi(this);
 
@@ -814,7 +810,7 @@ void PrefColors::apply(LyXRC & /*rc*/) const
 {
        for (unsigned int i = 0; i < lcolors_.size(); ++i)
                if (curcolors_[i] != newcolors_[i])
-                       form_->setColor(lcolors_[i], fromqstr(newcolors_[i]));
+                       form_->setColor(lcolors_[i], newcolors_[i]);
 }
 
 
@@ -864,8 +860,8 @@ void PrefColors::change_lyxObjects_selection()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefDisplay::PrefDisplay(QWidget * parent)
-       : PrefModule(qt_(catLookAndFeel), qt_("Graphics"), 0, parent)
+PrefDisplay::PrefDisplay(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("Graphics"), form)
 {
        setupUi(this);
        connect(instantPreviewCO, SIGNAL(activated(int)),
@@ -935,8 +931,8 @@ void PrefDisplay::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefPaths::PrefPaths(GuiPreferences * form, QWidget * parent)
-       : PrefModule(QString(), qt_("Paths"), form, parent)
+PrefPaths::PrefPaths(GuiPreferences * form)
+       : PrefModule(QString(), qt_("Paths"), form)
 {
        setupUi(this);
        connect(exampleDirPB, SIGNAL(clicked()), this, SLOT(select_exampledir()));
@@ -1048,8 +1044,8 @@ void PrefPaths::select_lyxpipe()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefSpellchecker::PrefSpellchecker(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catLanguage), qt_("Spellchecker"), form, parent)
+PrefSpellchecker::PrefSpellchecker(GuiPreferences * form)
+       : PrefModule(qt_(catLanguage), qt_("Spellchecker"), form)
 {
        setupUi(this);
 
@@ -1157,8 +1153,8 @@ void PrefSpellchecker::select_dict()
 /////////////////////////////////////////////////////////////////////
 
 
-PrefConverters::PrefConverters(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catFiles), qt_("Converters"), form, parent)
+PrefConverters::PrefConverters(GuiPreferences * form)
+       : PrefModule(qt_(catFiles), qt_("Converters"), form)
 {
        setupUi(this);
 
@@ -1223,8 +1219,8 @@ void PrefConverters::updateGui()
        Formats::const_iterator cit = form_->formats().begin();
        Formats::const_iterator end = form_->formats().end();
        for (; cit != end; ++cit) {
-               converterFromCO->addItem(toqstr(cit->prettyname()));
-               converterToCO->addItem(toqstr(cit->prettyname()));
+               converterFromCO->addItem(qt_(cit->prettyname()));
+               converterToCO->addItem(qt_(cit->prettyname()));
        }
 
        // currentRowChanged(int) is also triggered when updating the listwidget
@@ -1235,10 +1231,10 @@ void PrefConverters::updateGui()
        Converters::const_iterator ccit = form_->converters().begin();
        Converters::const_iterator cend = form_->converters().end();
        for (; ccit != cend; ++ccit) {
-               string const name =
-                       ccit->From->prettyname() + " -> " + ccit->To->prettyname();
+               QString const name =
+                       qt_(ccit->From->prettyname()) + " -> " + qt_(ccit->To->prettyname());
                int type = form_->converters().getNumber(ccit->From->name(), ccit->To->name());
-               new QListWidgetItem(toqstr(name), convertersLW, type);
+               new QListWidgetItem(name, convertersLW, type);
        }
        convertersLW->sortItems(Qt::AscendingOrder);
        convertersLW->blockSignals(false);
@@ -1288,13 +1284,13 @@ void PrefConverters::updateButtons()
                || from.name() == to.name());
 
        int const cnr = convertersLW->currentItem()->type();
-       Converter const & c(form_->converters().get(cnr));
+       Converter const & c = form_->converters().get(cnr);
        string const old_command = c.command;
        string const old_flag = c.flags;
-       string const new_command(fromqstr(converterED->text()));
-       string const new_flag(fromqstr(converterFlagED->text()));
+       string const new_command = fromqstr(converterED->text());
+       string const new_flag = fromqstr(converterFlagED->text());
 
-       bool modified = (old_command != new_command) || (old_flag != new_flag);
+       bool modified = (old_command != new_command || old_flag != new_flag);
 
        converterModifyPB->setEnabled(valid && known && modified);
        converterNewPB->setEnabled(valid && !known);
@@ -1354,10 +1350,23 @@ void PrefConverters::on_cacheCB_stateChanged(int state)
 
 /////////////////////////////////////////////////////////////////////
 //
-// PrefFileformats
+// FormatValidator
 //
 /////////////////////////////////////////////////////////////////////
-//
+
+class FormatValidator : public QValidator
+{
+public:
+       FormatValidator(QWidget *, Formats const & f);
+       void fixup(QString & input) const;
+       QValidator::State validate(QString & input, int & pos) const;
+private:
+       virtual QString toString(Format const & format) const = 0;
+       int nr() const;
+       Formats const & formats_;
+};
+
+
 FormatValidator::FormatValidator(QWidget * parent, Formats const & f) 
        : QValidator(parent), formats_(f)
 {
@@ -1369,11 +1378,10 @@ void FormatValidator::fixup(QString & input) const
        Formats::const_iterator cit = formats_.begin();
        Formats::const_iterator end = formats_.end();
        for (; cit != end; ++cit) {
-               string const name = str(cit);
+               QString const name = toString(*cit);
                if (distance(formats_.begin(), cit) == nr()) {
-                       input = toqstr(name);
+                       input = name;
                        return;
-
                }
        }
 }
@@ -1385,9 +1393,9 @@ QValidator::State FormatValidator::validate(QString & input, int & /*pos*/) cons
        Formats::const_iterator end = formats_.end();
        bool unknown = true;
        for (; unknown && cit != end; ++cit) {
-               string const name = str(cit);
+               QString const name = toString(*cit);
                if (distance(formats_.begin(), cit) != nr())
-                       unknown = toqstr(name) != input;
+                       unknown = name != input;
        }
 
        if (unknown && !input.isEmpty()) 
@@ -1404,31 +1412,54 @@ int FormatValidator::nr() const
 }
 
 
-FormatNameValidator::FormatNameValidator(QWidget * parent, Formats const & f) 
-       : FormatValidator(parent, f)
-{
-}
+/////////////////////////////////////////////////////////////////////
+//
+// FormatNameValidator
+//
+/////////////////////////////////////////////////////////////////////
 
-string FormatNameValidator::str(Formats::const_iterator it) const
+class FormatNameValidator : public FormatValidator
 {
-       return it->name();
-}
-
+public:
+       FormatNameValidator(QWidget * parent, Formats const & f)
+               : FormatValidator(parent, f)
+       {}
+private:
+       QString toString(Format const & format) const
+       {
+               return toqstr(format.name());
+       }
+};
 
-FormatPrettynameValidator::FormatPrettynameValidator(QWidget * parent, Formats const & f) 
-       : FormatValidator(parent, f)
-{
-}
 
+/////////////////////////////////////////////////////////////////////
+//
+// FormatPrettynameValidator
+//
+/////////////////////////////////////////////////////////////////////
 
-string FormatPrettynameValidator::str(Formats::const_iterator it) const
+class FormatPrettynameValidator : public FormatValidator
 {
-       return it->prettyname();
-}
+public:
+       FormatPrettynameValidator(QWidget * parent, Formats const & f)
+               : FormatValidator(parent, f)
+       {}
+private:
+       QString toString(Format const & format) const
+       {
+               return qt_(format.prettyname());
+       }
+};
 
 
-PrefFileformats::PrefFileformats(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catFiles), qt_("File formats"), form, parent)
+/////////////////////////////////////////////////////////////////////
+//
+// PrefFileformats
+//
+/////////////////////////////////////////////////////////////////////
+
+PrefFileformats::PrefFileformats(GuiPreferences * form)
+       : PrefModule(qt_(catFiles), qt_("File formats"), form)
 {
        setupUi(this);
        formatED->setValidator(new FormatNameValidator(formatsCB, form_->formats()));
@@ -1445,6 +1476,21 @@ PrefFileformats::PrefFileformats(GuiPreferences * form, QWidget * parent)
 }
 
 
+namespace {
+
+string const l10n_shortcut(string const prettyname, string const shortcut)
+{
+       if (shortcut.empty())
+               return string();
+
+       string l10n_format =
+               to_utf8(_(prettyname + '|' + shortcut));
+       return split(l10n_format, '|');
+}
+
+}; // namespace anon
+
+
 void PrefFileformats::apply(LyXRC & /*rc*/) const
 {
 }
@@ -1467,8 +1513,8 @@ void PrefFileformats::updateView()
        Formats::const_iterator cit = form_->formats().begin();
        Formats::const_iterator end = form_->formats().end();
        for (; cit != end; ++cit)
-               formatsCB->addItem(toqstr(cit->prettyname()),
-                                                       QVariant(form_->formats().getNumber(cit->name())) );
+               formatsCB->addItem(qt_(cit->prettyname()),
+                                  QVariant(form_->formats().getNumber(cit->name())));
 
        // restore selection
        int const item = formatsCB->findText(current, Qt::MatchExactly);
@@ -1486,7 +1532,8 @@ void PrefFileformats::on_formatsCB_currentIndexChanged(int i)
        formatED->setText(toqstr(f.name()));
        copierED->setText(toqstr(form_->movers().command(f.name())));
        extensionED->setText(toqstr(f.extension()));
-       shortcutED->setText(toqstr(f.shortcut()));
+       shortcutED->setText(
+               toqstr(l10n_shortcut(f.prettyname(), f.shortcut())));
        viewerED->setText(toqstr(f.viewer()));
        editorED->setText(toqstr(f.editor()));
        documentCB->setChecked((f.documentFormat()));
@@ -1536,7 +1583,11 @@ void PrefFileformats::on_editorED_textEdited(const QString & s)
 
 void PrefFileformats::on_shortcutED_textEdited(const QString & s)
 {
-       currentFormat().setShortcut(fromqstr(s));
+       string const new_shortcut = fromqstr(s);
+       if (new_shortcut == l10n_shortcut(currentFormat().prettyname(),
+                                         currentFormat().shortcut()))
+               return;
+       currentFormat().setShortcut(new_shortcut);
        changed();
 }
 
@@ -1572,11 +1623,11 @@ void PrefFileformats::on_formatsCB_editTextChanged(const QString &)
 
 void PrefFileformats::updatePrettyname()
 {
-       string const newname = fromqstr(formatsCB->currentText());
-       if (newname == currentFormat().prettyname())
+       QString const newname = formatsCB->currentText();
+       if (newname == qt_(currentFormat().prettyname()))
                return;
 
-       currentFormat().setPrettyname(newname);
+       currentFormat().setPrettyname(fromqstr(newname));
        formatsChanged();
        updateView();
        changed();
@@ -1626,8 +1677,8 @@ void PrefFileformats::on_formatRemovePB_clicked()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefLanguage::PrefLanguage(QWidget * parent)
-       : PrefModule(qt_(catLanguage), qt_("Language"), 0, parent)
+PrefLanguage::PrefLanguage(GuiPreferences * form)
+       : PrefModule(qt_(catLanguage), qt_("Language"), form)
 {
        setupUi(this);
 
@@ -1658,13 +1709,10 @@ PrefLanguage::PrefLanguage(QWidget * parent)
 
        defaultLanguageCO->clear();
 
-       // store the lang identifiers for later
-       //foreach (LanguagePair const & lpair, languageData(false)) {
-       QList<LanguagePair> l = languageData(false);
-       foreach (LanguagePair const & lpair, l) {
-               defaultLanguageCO->addItem(lpair.first);
-               lang_.append(lpair.second);
-       }
+       QAbstractItemModel * language_model = guiApp->languageModel();
+       // FIXME: it would be nice if sorting was enabled/disabled via a checkbox.
+       language_model->sort(0);
+       defaultLanguageCO->setModel(language_model);
 }
 
 
@@ -1681,7 +1729,8 @@ void PrefLanguage::apply(LyXRC & rc) const
        rc.language_package = fromqstr(languagePackageED->text());
        rc.language_command_begin = fromqstr(startCommandED->text());
        rc.language_command_end = fromqstr(endCommandED->text());
-       rc.default_language = fromqstr(lang_[defaultLanguageCO->currentIndex()]);
+       rc.default_language = fromqstr(
+               defaultLanguageCO->itemData(defaultLanguageCO->currentIndex()).toString());
 }
 
 
@@ -1702,7 +1751,7 @@ void PrefLanguage::update(LyXRC const & rc)
        startCommandED->setText(toqstr(rc.language_command_begin));
        endCommandED->setText(toqstr(rc.language_command_end));
 
-       int const pos = findPos_helper(lang_, toqstr(rc.default_language));
+       int const pos = defaultLanguageCO->findData(toqstr(rc.default_language));
        defaultLanguageCO->setCurrentIndex(pos);
 }
 
@@ -1713,8 +1762,8 @@ void PrefLanguage::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefPrinter::PrefPrinter(QWidget * parent)
-       : PrefModule(qt_(catOutput), qt_("Printer"), 0, parent)
+PrefPrinter::PrefPrinter(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("Printer"), form)
 {
        setupUi(this);
 
@@ -1811,8 +1860,8 @@ void PrefPrinter::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefUserInterface::PrefUserInterface(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catLookAndFeel), qt_("User interface"), form, parent)
+PrefUserInterface::PrefUserInterface(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("User interface"), form)
 {
        setupUi(this);
 
@@ -1883,14 +1932,15 @@ void PrefUserInterface::select_ui()
                uiFileED->setText(file);
 }
 
+
 /////////////////////////////////////////////////////////////////////
 //
 // PrefEdit
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefEdit::PrefEdit(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catEditing), qt_("Control"), form, parent)
+PrefEdit::PrefEdit(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Control"), form)
 {
        setupUi(this);
 
@@ -1961,8 +2011,8 @@ GuiShortcutDialog::GuiShortcutDialog(QWidget * parent) : QDialog(parent)
 }
 
 
-PrefShortcuts::PrefShortcuts(GuiPreferences * form, QWidget * parent)
-       : PrefModule(qt_(catEditing), qt_("Shortcuts"), form, parent)
+PrefShortcuts::PrefShortcuts(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Shortcuts"), form)
 {
        setupUi(this);
 
@@ -1972,7 +2022,6 @@ PrefShortcuts::PrefShortcuts(GuiPreferences * form, QWidget * parent)
        shortcutsTW->setSortingEnabled(true);
        // Multi-selection can be annoying.
        // shortcutsTW->setSelectionMode(QAbstractItemView::MultiSelection);
-       shortcutsTW->header()->resizeSection(0, 200);
 
        connect(bindFilePB, SIGNAL(clicked()),
                this, SLOT(select_bind()));
@@ -2053,7 +2102,7 @@ void PrefShortcuts::updateShortcutsTW()
        mathItem_->setFlags(mathItem_->flags() & ~Qt::ItemIsSelectable);
        
        bufferItem_ = new QTreeWidgetItem(shortcutsTW);
-       bufferItem_->setText(0, qt_("Buffer and Window"));
+       bufferItem_->setText(0, qt_("Document and Window"));
        bufferItem_->setFlags(bufferItem_->flags() & ~Qt::ItemIsSelectable);
        
        layoutItem_ = new QTreeWidgetItem(shortcutsTW);
@@ -2080,15 +2129,18 @@ void PrefShortcuts::updateShortcutsTW()
        KeyMap::BindingList::const_iterator it = bindinglist.begin();
        KeyMap::BindingList::const_iterator it_end = bindinglist.end();
        for (; it != it_end; ++it)
-               insertShortcutItem(it->request, it->sequence, item_type(it->tag));
+               insertShortcutItem(it->request, it->sequence, ItemType(it->tag));
 
        shortcutsTW->sortItems(0, Qt::AscendingOrder);
        QList<QTreeWidgetItem*> items = shortcutsTW->selectedItems();
        removePB->setEnabled(!items.isEmpty() && !items[0]->text(1).isEmpty());
+       modifyPB->setEnabled(!items.isEmpty());
+
+       shortcutsTW->resizeColumnToContents(0);
 }
 
 
-void PrefShortcuts::setItemType(QTreeWidgetItem * item, item_type tag)
+void PrefShortcuts::setItemType(QTreeWidgetItem * item, ItemType tag)
 {
        item->setData(0, Qt::UserRole, QVariant(tag));
        QFont font;
@@ -2113,14 +2165,14 @@ void PrefShortcuts::setItemType(QTreeWidgetItem * item, item_type tag)
 
 
 QTreeWidgetItem * PrefShortcuts::insertShortcutItem(FuncRequest const & lfun,
-               KeySequence const & seq, item_type tag)
+               KeySequence const & seq, ItemType tag)
 {
        FuncCode action = lfun.action;
        string const action_name = lyxaction.getActionName(action);
        QString const lfun_name = toqstr(from_utf8(action_name) 
                        + ' ' + lfun.argument());
        QString const shortcut = toqstr(seq.print(KeySequence::ForGui));
-       item_type item_tag = tag;
+       ItemType item_tag = tag;
 
        QTreeWidgetItem * newItem = 0;
        // for unbind items, try to find an existing item in the system bind list
@@ -2178,10 +2230,11 @@ void PrefShortcuts::on_shortcutsTW_itemSelectionChanged()
 {
        QList<QTreeWidgetItem*> items = shortcutsTW->selectedItems();
        removePB->setEnabled(!items.isEmpty() && !items[0]->text(1).isEmpty());
+       modifyPB->setEnabled(!items.isEmpty());
        if (items.isEmpty())
                return;
        
-       item_type tag = static_cast<item_type>(items[0]->data(0, Qt::UserRole).toInt());
+       ItemType tag = static_cast<ItemType>(items[0]->data(0, Qt::UserRole).toInt());
        if (tag == UserUnbind)
                removePB->setText(qt_("Res&tore"));
        else
@@ -2190,13 +2243,17 @@ void PrefShortcuts::on_shortcutsTW_itemSelectionChanged()
 
 
 void PrefShortcuts::on_shortcutsTW_itemDoubleClicked()
+{
+       modifyShortcut();
+}
+
+
+void PrefShortcuts::modifyShortcut()
 {
        QTreeWidgetItem * item = shortcutsTW->currentItem();
        if (item->flags() & Qt::ItemIsSelectable) {
                shortcut_->lfunLE->setText(item->text(0));
-               // clear the shortcut because I assume that a user will enter
-               // a new shortcut.
-               shortcut_->shortcutLE->reset();
+               shortcut_->shortcutLE->setText(item->text(1));
                shortcut_->shortcutLE->setFocus();
                shortcut_->exec();
        }
@@ -2215,6 +2272,12 @@ void PrefShortcuts::select_bind()
 }
 
 
+void PrefShortcuts::on_modifyPB_pressed()
+{
+       modifyShortcut();
+}
+
+
 void PrefShortcuts::on_newPB_pressed()
 {
        shortcut_->lfunLE->clear();
@@ -2232,7 +2295,7 @@ void PrefShortcuts::on_removePB_pressed()
                string shortcut = fromqstr(items[i]->data(1, Qt::UserRole).toString());
                string lfun = fromqstr(items[i]->text(0));
                FuncRequest func = lyxaction.lookupFunc(lfun);
-               item_type tag = static_cast<item_type>(items[i]->data(0, Qt::UserRole).toInt());
+               ItemType tag = static_cast<ItemType>(items[i]->data(0, Qt::UserRole).toInt());
                
                switch (tag) {
                case System: {
@@ -2354,8 +2417,8 @@ void PrefShortcuts::shortcut_clearPB_pressed()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefIdentity::PrefIdentity(QWidget * parent)
-       : PrefModule(QString(), qt_("Identity"), 0, parent)
+PrefIdentity::PrefIdentity(GuiPreferences * form)
+       : PrefModule(QString(), qt_("Identity"), form)
 {
        setupUi(this);
 
@@ -2399,33 +2462,33 @@ GuiPreferences::GuiPreferences(GuiView & lv)
        connect(closePB, SIGNAL(clicked()), this, SLOT(slotClose()));
        connect(restorePB, SIGNAL(clicked()), this, SLOT(slotRestore()));
 
-       add(new PrefUserInterface(this));
-       add(new PrefEdit(this));
-       add(new PrefShortcuts(this));
-       add(new PrefScreenFonts(this));
-       add(new PrefColors(this));
-       add(new PrefDisplay);
-       add(new PrefInput(this));
-       add(new PrefCompletion(this));
+       addModule(new PrefUserInterface(this));
+       addModule(new PrefEdit(this));
+       addModule(new PrefShortcuts(this));
+       addModule(new PrefScreenFonts(this));
+       addModule(new PrefColors(this));
+       addModule(new PrefDisplay(this));
+       addModule(new PrefInput(this));
+       addModule(new PrefCompletion(this));
 
-       add(new PrefPaths(this));
+       addModule(new PrefPaths(this));
 
-       add(new PrefIdentity);
+       addModule(new PrefIdentity(this));
 
-       add(new PrefLanguage);
-       add(new PrefSpellchecker(this));
+       addModule(new PrefLanguage(this));
+       addModule(new PrefSpellchecker(this));
 
-       add(new PrefPrinter);
-       add(new PrefDate);
-       add(new PrefPlaintext);
-       add(new PrefLatex(this));
+       addModule(new PrefPrinter(this));
+       addModule(new PrefDate(this));
+       addModule(new PrefPlaintext(this));
+       addModule(new PrefLatex(this));
 
        PrefConverters * converters = new PrefConverters(this);
        PrefFileformats * formats = new PrefFileformats(this);
        connect(formats, SIGNAL(formatsChanged()),
                        converters, SLOT(updateGui()));
-       add(converters);
-       add(formats);
+       addModule(converters);
+       addModule(formats);
 
        prefsPS->setCurrentPanel(qt_("User interface"));
 // FIXME: hack to work around resizing bug in Qt >= 4.2
@@ -2442,7 +2505,7 @@ GuiPreferences::GuiPreferences(GuiView & lv)
 }
 
 
-void GuiPreferences::add(PrefModule * module)
+void GuiPreferences::addModule(PrefModule * module)
 {
        LASSERT(module, /**/);
        if (module->category().isEmpty())
@@ -2536,9 +2599,9 @@ void GuiPreferences::dispatchParams()
 }
 
 
-void GuiPreferences::setColor(ColorCode col, string const & hex)
+void GuiPreferences::setColor(ColorCode col, QString const & hex)
 {
-       colors_.push_back(lcolor.getLyXName(col) + ' ' + hex);
+       colors_.push_back(lcolor.getLyXName(col) + ' ' + fromqstr(hex));
 }