]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/qt4/GuiPrefs.cpp
Compil fix.
[lyx.git] / src / frontends / qt4 / GuiPrefs.cpp
index cce2399110a42f1bc059666dd8d78e5ce4e4e85d..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 "support/debug.h"
+#include "FontEnums.h"
 #include "FuncRequest.h"
-#include "support/gettext.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"
 #include "Session.h"
 
+#include "support/debug.h"
 #include "support/FileName.h"
 #include "support/filetools.h"
+#include "support/foreach.h"
+#include "support/gettext.h"
 #include "support/lstrings.h"
 #include "support/os.h"
 #include "support/Package.h"
-#include "support/FileFilterList.h"
 
 #include "graphics/GraphicsTypes.h"
 
 #include "frontends/alert.h"
 #include "frontends/Application.h"
+#include "frontends/FontLoader.h"
 
+#include <QAbstractItemModel>
 #include <QCheckBox>
 #include <QColorDialog>
 #include <QFontDatabase>
@@ -56,7 +62,6 @@
 #include <QTreeWidget>
 #include <QTreeWidgetItem>
 #include <QValidator>
-#include <QCloseEvent>
 
 #include <iomanip>
 #include <sstream>
@@ -71,25 +76,152 @@ using namespace lyx::support::os;
 namespace lyx {
 namespace frontend {
 
-
 /////////////////////////////////////////////////////////////////////
 //
-// Helpers
+// Browser Helpers
 //
 /////////////////////////////////////////////////////////////////////
 
-template<class A>
-static size_t findPos_helper(vector<A> const & vec, A const & val)
+/** Launch a file dialog and return the chosen file.
+       filename: a suggested filename.
+       title: the title of the dialog.
+       pattern: *.ps etc.
+       dir1 = (name, dir), dir2 = (name, dir): extra buttons on the dialog.
+*/
+QString browseFile(QString const & filename,
+       QString const & title,
+       QStringList const & filters,
+       bool save = false,
+       QString const & label1 = QString(),
+       QString const & dir1 = QString(),
+       QString const & label2 = QString(),
+       QString const & dir2 = QString())
+{
+       QString lastPath = ".";
+       if (!filename.isEmpty())
+               lastPath = onlyPath(filename);
+
+       FileDialog dlg(title, LFUN_SELECT_FILE_SYNC);
+       dlg.setButton2(label1, dir1);
+       dlg.setButton2(label2, dir2);
+
+       FileDialog::Result result;
+
+       if (save)
+               result = dlg.save(lastPath, filters, onlyFilename(filename));
+       else
+               result = dlg.open(lastPath, filters, onlyFilename(filename));
+
+       return result.second;
+}
+
+
+/** Wrapper around browseFile which tries to provide a filename
+*  relative to the user or system directory. The dir, name and ext
+*  parameters have the same meaning as in the
+*  support::LibFileSearch function.
+*/
+QString browseLibFile(QString const & dir,
+       QString const & name,
+       QString const & ext,
+       QString const & title,
+       QStringList const & filters)
 {
-       typedef typename vector<A>::const_iterator Cit;
+       // FIXME UNICODE
+       QString const label1 = qt_("System files|#S#s");
+       QString const dir1 =
+               toqstr(addName(package().system_support().absFilename(), fromqstr(dir)));
+
+       QString const label2 = qt_("User files|#U#u");
+       QString const dir2 =
+               toqstr(addName(package().user_support().absFilename(), fromqstr(dir)));
+
+       QString const result = browseFile(toqstr(
+               libFileSearch(dir, name, ext).absFilename()),
+               title, filters, false, dir1, dir2);
+
+       // remove the extension if it is the default one
+       QString noextresult;
+       if (getExtension(result) == ext)
+               noextresult = removeExtension(result);
+       else
+               noextresult = result;
 
-       Cit it = find(vec.begin(), vec.end(), val);
-       if (it == vec.end())
-               return 0;
-       return distance(vec.begin(), it);
+       // remove the directory, if it is the default one
+       QString const file = onlyFilename(noextresult);
+       if (toqstr(libFileSearch(dir, file, ext).absFilename()) == result)
+               return file;
+       else
+               return noextresult;
+}
+
+
+/** Launch a file dialog and return the chosen directory.
+       pathname: a suggested pathname.
+       title: the title of the dialog.
+       dir1 = (name, dir), dir2 = (name, dir): extra buttons on the dialog.
+*/
+QString browseDir(QString const & pathname,
+       QString const & title,
+       QString const & label1 = QString(),
+       QString const & dir1 = QString(),
+       QString const & label2 = QString(),
+       QString const & dir2 = QString())
+{
+       QString lastPath = ".";
+       if (!pathname.isEmpty())
+               lastPath = onlyPath(pathname);
+
+       FileDialog dlg(title, LFUN_SELECT_FILE_SYNC);
+       dlg.setButton1(label1, dir1);
+       dlg.setButton2(label2, dir2);
+
+       FileDialog::Result const result =
+               dlg.opendir(lastPath, onlyFilename(pathname));
+
+       return result.second;
+}
+
+
+} // namespace frontend
+
+
+QString browseRelFile(QString const & filename, QString const & refpath,
+       QString const & title, QStringList const & filters, bool save,
+       QString const & label1, QString const & dir1,
+       QString const & label2, QString const & dir2)
+{
+       QString const fname = makeAbsPath(filename, refpath);
+
+
+       QString const outname =
+               frontend::browseFile(fname, title, filters, save, label1, dir1, label2, dir2);
+
+       QString const reloutname =
+               toqstr(makeRelPath(qstring_to_ucs4(outname), qstring_to_ucs4(refpath)));
+
+       if (reloutname.startsWith("../"))
+               return outname;
+       else
+               return reloutname;
 }
 
 
+
+/////////////////////////////////////////////////////////////////////
+//
+// Helpers
+//
+/////////////////////////////////////////////////////////////////////
+
+namespace frontend {
+
+string const catLookAndFeel = N_("Look & Feel");
+string const catEditing = N_("Editing");
+string const catLanguage = N_("Language Settings");
+string const catOutput = N_("Output");
+string const catFiles = N_("File Handling");
+
 static void parseFontName(QString const & mangled0,
        string & name, string & foundry)
 {
@@ -112,7 +244,7 @@ static void setComboxFont(QComboBox * cb, string const & family,
        if (!foundry.empty())
                fontname += " [" + toqstr(foundry) + ']';
 
-       for (int i = 0; i < cb->count(); ++i) {
+       for (int i = 0; i != cb->count(); ++i) {
                if (cb->itemText(i) == fontname) {
                        cb->setCurrentIndex(i);
                        return;
@@ -162,18 +294,18 @@ static void setComboxFont(QComboBox * cb, string const & family,
                font.setStyleHint(QFont::TypeWriter);
                font.setFamily(font_family);
        } else {
-               lyxerr << "FAILED to find the default font: '"
-                      << foundry << "', '" << family << '\''<< endl;
+               LYXERR0("FAILED to find the default font: '"
+                      << foundry << "', '" << family << '\'');
                return;
        }
 
        QFontInfo info(font);
        string default_font_name, dummyfoundry;
        parseFontName(info.family(), default_font_name, dummyfoundry);
-       lyxerr << "Apparent font is " << default_font_name << endl;
+       LYXERR0("Apparent font is " << default_font_name);
 
        for (int i = 0; i < cb->count(); ++i) {
-               lyxerr << "Looking at " << fromqstr(cb->itemText(i)) << endl;
+               LYXERR0("Looking at " << cb->itemText(i));
                if (compare_ascii_no_case(fromqstr(cb->itemText(i)),
                                    default_font_name) == 0) {
                        cb->setCurrentIndex(i);
@@ -181,19 +313,20 @@ static void setComboxFont(QComboBox * cb, string const & family,
                }
        }
 
-       lyxerr << "FAILED to find the font: '"
-              << foundry << "', '" << family << '\'' <<endl;
+       LYXERR0("FAILED to find the font: '"
+              << foundry << "', '" << family << '\'');
 }
 
 
+
 /////////////////////////////////////////////////////////////////////
 //
 // PrefPlaintext
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefPlaintext::PrefPlaintext(QWidget * parent)
-       : PrefModule(_("Plain text"), 0, parent)
+PrefPlaintext::PrefPlaintext(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("Plain text"), form)
 {
        setupUi(this);
        connect(plaintextLinelengthSB, SIGNAL(valueChanged(int)),
@@ -223,8 +356,8 @@ void PrefPlaintext::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefDate::PrefDate(QWidget * parent)
-       : PrefModule(_("Date format"), 0, parent)
+PrefDate::PrefDate(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("Date format"), form)
 {
        setupUi(this);
        connect(DateED, SIGNAL(textChanged(QString)),
@@ -250,8 +383,8 @@ void PrefDate::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefInput::PrefInput(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("Keyboard/Mouse"), form, parent)
+PrefInput::PrefInput(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Keyboard/Mouse"), form)
 {
        setupUi(this);
 
@@ -286,9 +419,9 @@ void PrefInput::update(LyXRC const & rc)
 }
 
 
-QString PrefInput::testKeymap(QString keymap)
+QString PrefInput::testKeymap(QString const & keymap)
 {
-       return toqstr(form_->browsekbmap(from_utf8(internal_path(fromqstr(keymap)))));
+       return form_->browsekbmap(internalPath(keymap));
 }
 
 
@@ -319,14 +452,76 @@ void PrefInput::on_keymapCB_toggled(bool keymap)
 }
 
 
+/////////////////////////////////////////////////////////////////////
+//
+// PrefCompletion
+//
+/////////////////////////////////////////////////////////////////////
+
+PrefCompletion::PrefCompletion(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Input Completion"), form)
+{
+       setupUi(this);
+
+       connect(inlineDelaySB, SIGNAL(valueChanged(double)),
+               this, SIGNAL(changed()));
+       connect(inlineMathCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(inlineTextCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(inlineDotsCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(popupDelaySB, SIGNAL(valueChanged(double)),
+               this, SIGNAL(changed()));
+       connect(popupMathCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(popupTextCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(popupAfterCompleteCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(cursorTextCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+}
+
+
+void PrefCompletion::apply(LyXRC & rc) const
+{
+       rc.completion_inline_delay = inlineDelaySB->value();
+       rc.completion_inline_math = inlineMathCB->isChecked();
+       rc.completion_inline_text = inlineTextCB->isChecked();
+       rc.completion_inline_dots = inlineDotsCB->isChecked() ? 13 : -1;
+       rc.completion_popup_delay = popupDelaySB->value();
+       rc.completion_popup_math = popupMathCB->isChecked();
+       rc.completion_popup_text = popupTextCB->isChecked();
+       rc.completion_cursor_text = cursorTextCB->isChecked();
+       rc.completion_popup_after_complete =
+               popupAfterCompleteCB->isChecked();
+}
+
+
+void PrefCompletion::update(LyXRC const & rc)
+{
+       inlineDelaySB->setValue(rc.completion_inline_delay);
+       inlineMathCB->setChecked(rc.completion_inline_math);
+       inlineTextCB->setChecked(rc.completion_inline_text);
+       inlineDotsCB->setChecked(rc.completion_inline_dots != -1);
+       popupDelaySB->setValue(rc.completion_popup_delay);
+       popupMathCB->setChecked(rc.completion_popup_math);
+       popupTextCB->setChecked(rc.completion_popup_text);
+       cursorTextCB->setChecked(rc.completion_cursor_text);
+       popupAfterCompleteCB->setChecked(rc.completion_popup_after_complete);
+}
+
+
+
 /////////////////////////////////////////////////////////////////////
 //
 // PrefLatex
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefLatex::PrefLatex(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("LaTeX"), form, parent)
+PrefLatex::PrefLatex(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("LaTeX"), form)
 {
        setupUi(this);
        connect(latexEncodingED, SIGNAL(textChanged(QString)),
@@ -392,8 +587,8 @@ void PrefLatex::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefScreenFonts::PrefScreenFonts(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("Screen fonts"), form, parent)
+PrefScreenFonts::PrefScreenFonts(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("Screen fonts"), form)
 {
        setupUi(this);
 
@@ -566,8 +761,8 @@ struct ColorSorter
 
 } // namespace anon
 
-PrefColors::PrefColors(GuiPreferences * form, QWidget * parent)
-       : PrefModule( _("Colors"), form, parent)
+PrefColors::PrefColors(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("Colors"), form)
 {
        setupUi(this);
 
@@ -615,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]);
 }
 
 
@@ -631,6 +826,7 @@ void PrefColors::update(LyXRC const & /*rc*/)
        change_lyxObjects_selection();
 }
 
+
 void PrefColors::change_color()
 {
        int const row = lyxObjectsLW->currentRow();
@@ -664,8 +860,8 @@ void PrefColors::change_lyxObjects_selection()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefDisplay::PrefDisplay(QWidget * parent)
-       : PrefModule(_("Graphics"), 0, parent)
+PrefDisplay::PrefDisplay(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("Graphics"), form)
 {
        setupUi(this);
        connect(instantPreviewCO, SIGNAL(activated(int)),
@@ -735,8 +931,8 @@ void PrefDisplay::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefPaths::PrefPaths(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("Paths"), form, parent)
+PrefPaths::PrefPaths(GuiPreferences * form)
+       : PrefModule(QString(), qt_("Paths"), form)
 {
        setupUi(this);
        connect(exampleDirPB, SIGNAL(clicked()), this, SLOT(select_exampledir()));
@@ -790,61 +986,55 @@ void PrefPaths::update(LyXRC const & rc)
 
 void PrefPaths::select_exampledir()
 {
-       docstring file(form_->browsedir(
-               from_utf8(internal_path(fromqstr(exampleDirED->text()))),
-               _("Select directory for example files")));
-       if (!file.empty())
-               exampleDirED->setText(toqstr(file));
+       QString file = browseDir(internalPath(exampleDirED->text()),
+               qt_("Select directory for example files"));
+       if (!file.isEmpty())
+               exampleDirED->setText(file);
 }
 
 
 void PrefPaths::select_templatedir()
 {
-       docstring file(form_->browsedir(
-               from_utf8(internal_path(fromqstr(templateDirED->text()))),
-               _("Select a document templates directory")));
-       if (!file.empty())
-               templateDirED->setText(toqstr(file));
+       QString file = browseDir(internalPath(templateDirED->text()),
+               qt_("Select a document templates directory"));
+       if (!file.isEmpty())
+               templateDirED->setText(file);
 }
 
 
 void PrefPaths::select_tempdir()
 {
-       docstring file(form_->browsedir(
-               from_utf8(internal_path(fromqstr(tempDirED->text()))),
-               _("Select a temporary directory")));
-       if (!file.empty())
-               tempDirED->setText(toqstr(file));
+       QString file = browseDir(internalPath(tempDirED->text()),
+               qt_("Select a temporary directory"));
+       if (!file.isEmpty())
+               tempDirED->setText(file);
 }
 
 
 void PrefPaths::select_backupdir()
 {
-       docstring file(form_->browsedir(
-               from_utf8(internal_path(fromqstr(backupDirED->text()))),
-               _("Select a backups directory")));
-       if (!file.empty())
-               backupDirED->setText(toqstr(file));
+       QString file = browseDir(internalPath(backupDirED->text()),
+               qt_("Select a backups directory"));
+       if (!file.isEmpty())
+               backupDirED->setText(file);
 }
 
 
 void PrefPaths::select_workingdir()
 {
-       docstring file(form_->browsedir(
-               from_utf8(internal_path(fromqstr(workingDirED->text()))),
-               _("Select a document directory")));
-       if (!file.empty())
-               workingDirED->setText(toqstr(file));
+       QString file = browseDir(internalPath(workingDirED->text()),
+               qt_("Select a document directory"));
+       if (!file.isEmpty())
+               workingDirED->setText(file);
 }
 
 
 void PrefPaths::select_lyxpipe()
 {
-       docstring file(form_->browse(
-               from_utf8(internal_path(fromqstr(lyxserverDirED->text()))),
-               _("Give a filename for the LyX server pipe")));
-       if (!file.empty())
-               lyxserverDirED->setText(toqstr(file));
+       QString file = form_->browse(internalPath(lyxserverDirED->text()),
+               qt_("Give a filename for the LyX server pipe"));
+       if (!file.isEmpty())
+               lyxserverDirED->setText(file);
 }
 
 
@@ -854,8 +1044,8 @@ void PrefPaths::select_lyxpipe()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefSpellchecker::PrefSpellchecker(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("Spellchecker"), form, parent)
+PrefSpellchecker::PrefSpellchecker(GuiPreferences * form)
+       : PrefModule(qt_(catLanguage), qt_("Spellchecker"), form)
 {
        setupUi(this);
 
@@ -949,10 +1139,9 @@ void PrefSpellchecker::update(LyXRC const & rc)
 
 void PrefSpellchecker::select_dict()
 {
-       docstring file(form_->browsedict(
-               from_utf8(internal_path(fromqstr(persDictionaryED->text())))));
-       if (!file.empty())
-               persDictionaryED->setText(toqstr(file));
+       QString file = form_->browsedict(internalPath(persDictionaryED->text()));
+       if (!file.isEmpty())
+               persDictionaryED->setText(file);
 }
 
 
@@ -964,8 +1153,8 @@ void PrefSpellchecker::select_dict()
 /////////////////////////////////////////////////////////////////////
 
 
-PrefConverters::PrefConverters(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("Converters"), form, parent)
+PrefConverters::PrefConverters(GuiPreferences * form)
+       : PrefModule(qt_(catFiles), qt_("Converters"), form)
 {
        setupUi(this);
 
@@ -1030,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
@@ -1042,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);
@@ -1095,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);
@@ -1161,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)
 {
@@ -1176,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;
-
                }
        }
 }
@@ -1192,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()) 
@@ -1211,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(_("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()));
@@ -1252,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
 {
 }
@@ -1274,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);
@@ -1293,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()));
@@ -1343,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();
 }
 
@@ -1379,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();
@@ -1433,8 +1677,8 @@ void PrefFileformats::on_formatRemovePB_clicked()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefLanguage::PrefLanguage(QWidget * parent)
-       : PrefModule(_("Language"), 0, parent)
+PrefLanguage::PrefLanguage(GuiPreferences * form)
+       : PrefModule(qt_(catLanguage), qt_("Language"), form)
 {
        setupUi(this);
 
@@ -1465,15 +1709,10 @@ PrefLanguage::PrefLanguage(QWidget * parent)
 
        defaultLanguageCO->clear();
 
-       // store the lang identifiers for later
-       vector<LanguagePair> const langs = getLanguageData(false);
-       vector<LanguagePair>::const_iterator lit  = langs.begin();
-       vector<LanguagePair>::const_iterator lend = langs.end();
-       lang_.clear();
-       for (; lit != lend; ++lit) {
-               defaultLanguageCO->addItem(toqstr(lit->first));
-               lang_.push_back(lit->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);
 }
 
 
@@ -1490,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 = lang_[defaultLanguageCO->currentIndex()];
+       rc.default_language = fromqstr(
+               defaultLanguageCO->itemData(defaultLanguageCO->currentIndex()).toString());
 }
 
 
@@ -1511,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 = int(findPos_helper(lang_, rc.default_language));
+       int const pos = defaultLanguageCO->findData(toqstr(rc.default_language));
        defaultLanguageCO->setCurrentIndex(pos);
 }
 
@@ -1522,8 +1762,8 @@ void PrefLanguage::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefPrinter::PrefPrinter(QWidget * parent)
-       : PrefModule(_("Printer"), 0, parent)
+PrefPrinter::PrefPrinter(GuiPreferences * form)
+       : PrefModule(qt_(catOutput), qt_("Printer"), form)
 {
        setupUi(this);
 
@@ -1620,8 +1860,8 @@ void PrefPrinter::update(LyXRC const & rc)
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefUserInterface::PrefUserInterface(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("User interface"), form, parent)
+PrefUserInterface::PrefUserInterface(GuiPreferences * form)
+       : PrefModule(qt_(catLookAndFeel), qt_("User interface"), form)
 {
        setupUi(this);
 
@@ -1629,6 +1869,8 @@ PrefUserInterface::PrefUserInterface(GuiPreferences * form, QWidget * parent)
                autoSaveSB, SLOT(setEnabled(bool)));
        connect(autoSaveCB, SIGNAL(toggled(bool)),
                TextLabel1, SLOT(setEnabled(bool)));
+       connect(openDocumentsInTabsCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
        connect(uiFilePB, SIGNAL(clicked()),
                this, SLOT(select_ui()));
        connect(uiFileED, SIGNAL(textChanged(QString)),
@@ -1639,12 +1881,6 @@ PrefUserInterface::PrefUserInterface(GuiPreferences * form, QWidget * parent)
                this, SIGNAL(changed()));
        connect(allowGeometrySessionCB, SIGNAL(clicked()),
                this, SIGNAL(changed()));
-       connect(cursorFollowsCB, SIGNAL(clicked()),
-               this, SIGNAL(changed()));
-       connect(sortEnvironmentsCB, SIGNAL(clicked()),
-               this, SIGNAL(changed()));
-       connect(macroEditStyleCO, SIGNAL(activated(int)),
-               this, SIGNAL(changed()));
        connect(autoSaveSB, SIGNAL(valueChanged(int)),
                this, SIGNAL(changed()));
        connect(autoSaveCB, SIGNAL(clicked()),
@@ -1663,17 +1899,11 @@ void PrefUserInterface::apply(LyXRC & rc) const
        rc.use_lastfilepos = restoreCursorCB->isChecked();
        rc.load_session = loadSessionCB->isChecked();
        rc.allow_geometry_session = allowGeometrySessionCB->isChecked();
-       rc.cursor_follows_scrollbar = cursorFollowsCB->isChecked();
-       rc.sort_layouts = sortEnvironmentsCB->isChecked();
-       switch (macroEditStyleCO->currentIndex()) {
-               case 0: rc.macro_edit_style = LyXRC::MACRO_EDIT_INLINE_BOX; break;
-               case 1: rc.macro_edit_style = LyXRC::MACRO_EDIT_INLINE; break;
-               case 2: rc.macro_edit_style = LyXRC::MACRO_EDIT_LIST;   break;
-       }
        rc.autosave = autoSaveSB->value() * 60;
        rc.make_backup = autoSaveCB->isChecked();
        rc.num_lastfiles = lastfilesSB->value();
        rc.use_tooltip = tooltipCB->isChecked();
+       rc.open_buffers_in_tabs = openDocumentsInTabsCB->isChecked();
 }
 
 
@@ -1683,9 +1913,6 @@ void PrefUserInterface::update(LyXRC const & rc)
        restoreCursorCB->setChecked(rc.use_lastfilepos);
        loadSessionCB->setChecked(rc.load_session);
        allowGeometrySessionCB->setChecked(rc.allow_geometry_session);
-       cursorFollowsCB->setChecked(rc.cursor_follows_scrollbar);
-       sortEnvironmentsCB->setChecked(rc.sort_layouts);
-       macroEditStyleCO->setCurrentIndex(rc.macro_edit_style);
        // convert to minutes
        int mins(rc.autosave / 60);
        if (rc.autosave && !mins)
@@ -1694,16 +1921,79 @@ void PrefUserInterface::update(LyXRC const & rc)
        autoSaveCB->setChecked(rc.make_backup);
        lastfilesSB->setValue(rc.num_lastfiles);
        tooltipCB->setChecked(rc.use_tooltip);
+       openDocumentsInTabsCB->setChecked(rc.open_buffers_in_tabs);
 }
 
 
 void PrefUserInterface::select_ui()
 {
-       docstring const name =
-               from_utf8(internal_path(fromqstr(uiFileED->text())));
-       docstring file = form_->browseUI(name);
-       if (!file.empty())
-               uiFileED->setText(toqstr(file));
+       QString file = form_->browseUI(internalPath(uiFileED->text()));
+       if (!file.isEmpty())
+               uiFileED->setText(file);
+}
+
+
+/////////////////////////////////////////////////////////////////////
+//
+// PrefEdit
+//
+/////////////////////////////////////////////////////////////////////
+
+PrefEdit::PrefEdit(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Control"), form)
+{
+       setupUi(this);
+
+       connect(cursorFollowsCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(sortEnvironmentsCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(groupEnvironmentsCB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(macroEditStyleCO, SIGNAL(activated(int)),
+               this, SIGNAL(changed()));
+       connect(fullscreenLimitGB, SIGNAL(clicked()),
+               this, SIGNAL(changed()));
+       connect(fullscreenWidthSB, SIGNAL(valueChanged(int)),
+               this, SIGNAL(changed()));
+       connect(toggleTabbarCB, SIGNAL(toggled(bool)),
+               this, SIGNAL(changed()));
+       connect(toggleScrollbarCB, SIGNAL(toggled(bool)),
+               this, SIGNAL(changed()));
+       connect(toggleToolbarsCB, SIGNAL(toggled(bool)),
+               this, SIGNAL(changed()));
+}
+
+
+void PrefEdit::apply(LyXRC & rc) const
+{
+       rc.cursor_follows_scrollbar = cursorFollowsCB->isChecked();
+       rc.sort_layouts = sortEnvironmentsCB->isChecked();
+       rc.group_layouts = groupEnvironmentsCB->isChecked();
+       switch (macroEditStyleCO->currentIndex()) {
+               case 0: rc.macro_edit_style = LyXRC::MACRO_EDIT_INLINE_BOX; break;
+               case 1: rc.macro_edit_style = LyXRC::MACRO_EDIT_INLINE; break;
+               case 2: rc.macro_edit_style = LyXRC::MACRO_EDIT_LIST;   break;
+       }
+       rc.full_screen_toolbars = toggleToolbarsCB->isChecked();
+       rc.full_screen_scrollbar = toggleScrollbarCB->isChecked();
+       rc.full_screen_tabbar = toggleTabbarCB->isChecked();
+       rc.full_screen_width = fullscreenWidthSB->value();
+       rc.full_screen_limit = fullscreenLimitGB->isChecked();
+}
+
+
+void PrefEdit::update(LyXRC const & rc)
+{
+       cursorFollowsCB->setChecked(rc.cursor_follows_scrollbar);
+       sortEnvironmentsCB->setChecked(rc.sort_layouts);
+       groupEnvironmentsCB->setChecked(rc.group_layouts);
+       macroEditStyleCO->setCurrentIndex(rc.macro_edit_style);
+       toggleScrollbarCB->setChecked(rc.full_screen_scrollbar);
+       toggleToolbarsCB->setChecked(rc.full_screen_toolbars);
+       toggleTabbarCB->setChecked(rc.full_screen_tabbar);
+       fullscreenWidthSB->setValue(rc.full_screen_width);
+       fullscreenLimitGB->setChecked(rc.full_screen_limit);
 }
 
 
@@ -1721,8 +2011,8 @@ GuiShortcutDialog::GuiShortcutDialog(QWidget * parent) : QDialog(parent)
 }
 
 
-PrefShortcuts::PrefShortcuts(GuiPreferences * form, QWidget * parent)
-       : PrefModule(_("Shortcuts"), form, parent)
+PrefShortcuts::PrefShortcuts(GuiPreferences * form)
+       : PrefModule(qt_(catEditing), qt_("Shortcuts"), form)
 {
        setupUi(this);
 
@@ -1732,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()));
@@ -1805,23 +2094,23 @@ void PrefShortcuts::updateShortcutsTW()
        shortcutsTW->clear();
 
        editItem_ = new QTreeWidgetItem(shortcutsTW);
-       editItem_->setText(0, toqstr("Cursor, Mouse and Editing functions"));
+       editItem_->setText(0, qt_("Cursor, Mouse and Editing functions"));
        editItem_->setFlags(editItem_->flags() & ~Qt::ItemIsSelectable);
 
        mathItem_ = new QTreeWidgetItem(shortcutsTW);
-       mathItem_->setText(0, toqstr("Mathematical Symbols"));
+       mathItem_->setText(0, qt_("Mathematical Symbols"));
        mathItem_->setFlags(mathItem_->flags() & ~Qt::ItemIsSelectable);
        
        bufferItem_ = new QTreeWidgetItem(shortcutsTW);
-       bufferItem_->setText(0, toqstr("Buffer and Window"));
+       bufferItem_->setText(0, qt_("Document and Window"));
        bufferItem_->setFlags(bufferItem_->flags() & ~Qt::ItemIsSelectable);
        
        layoutItem_ = new QTreeWidgetItem(shortcutsTW);
-       layoutItem_->setText(0, toqstr("Font, Layouts and Textclasses"));
+       layoutItem_->setText(0, qt_("Font, Layouts and Textclasses"));
        layoutItem_->setFlags(layoutItem_->flags() & ~Qt::ItemIsSelectable);
 
        systemItem_ = new QTreeWidgetItem(shortcutsTW);
-       systemItem_->setText(0, toqstr("System and Miscellaneous"));
+       systemItem_->setText(0, qt_("System and Miscellaneous"));
        systemItem_->setFlags(systemItem_->flags() & ~Qt::ItemIsSelectable);
 
        // listBindings(unbound=true) lists all bound and unbound lfuns
@@ -1840,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;
@@ -1873,16 +2165,16 @@ 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)
 {
-       kb_action action = lfun.action;
+       FuncCode action = lfun.action;
        string const action_name = lyxaction.getActionName(action);
        QString const lfun_name = toqstr(from_utf8(action_name) 
-                       + " " + lfun.argument());
+                       + ' ' + lfun.argument());
        QString const shortcut = toqstr(seq.print(KeySequence::ForGui));
-       item_type item_tag = tag;
+       ItemType item_tag = tag;
 
-       QTreeWidgetItem * newItem = NULL;
+       QTreeWidgetItem * newItem = 0;
        // for unbind items, try to find an existing item in the system bind list
        if (tag == UserUnbind) {
                QList<QTreeWidgetItem*> const items = shortcutsTW->findItems(lfun_name, 
@@ -1897,13 +2189,13 @@ QTreeWidgetItem * PrefShortcuts::insertShortcutItem(FuncRequest const & lfun,
                // unmatched removed?).
                if (!newItem) {
                        item_tag = UserExtraUnbind;
-                       return NULL;
+                       return 0;
                }
        }
        if (!newItem) {
                switch(lyxaction.getActionType(action)) {
                case LyXAction::Hidden:
-                       return NULL;
+                       return 0;
                case LyXAction::Edit:
                        newItem = new QTreeWidgetItem(editItem_);
                        break;
@@ -1938,25 +2230,30 @@ 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(toqstr("Restore"));
+               removePB->setText(qt_("Res&tore"));
        else
-               removePB->setText(toqstr("Remove"));
+               removePB->setText(qt_("Remo&ve"));
 }
 
 
 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();
        }
@@ -1965,18 +2262,22 @@ void PrefShortcuts::on_shortcutsTW_itemDoubleClicked()
 
 void PrefShortcuts::select_bind()
 {
-       docstring const name =
-               from_utf8(internal_path(fromqstr(bindFileED->text())));
-       docstring file = form_->browsebind(name);
-       if (!file.empty()) {
-               bindFileED->setText(toqstr(file));
+       QString file = form_->browsebind(internalPath(bindFileED->text()));
+       if (!file.isEmpty()) {
+               bindFileED->setText(file);
                system_bind_ = KeyMap();
-               system_bind_.read(to_utf8(file));
+               system_bind_.read(fromqstr(file));
                updateShortcutsTW();
        }
 }
 
 
+void PrefShortcuts::on_modifyPB_pressed()
+{
+       modifyShortcut();
+}
+
+
 void PrefShortcuts::on_newPB_pressed()
 {
        shortcut_->lfunLE->clear();
@@ -1994,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: {
@@ -2002,7 +2303,7 @@ void PrefShortcuts::on_removePB_pressed()
                        // but add an user unbind item
                        user_unbind_.bind(shortcut, func);
                        setItemType(items[i], UserUnbind);
-                       removePB->setText(toqstr("Restore"));
+                       removePB->setText(qt_("Res&tore"));
                        break;
                }
                case UserBind: {
@@ -2022,7 +2323,7 @@ void PrefShortcuts::on_removePB_pressed()
                        // become System again.
                        user_unbind_.unbind(shortcut, func);
                        setItemType(items[i], System);
-                       removePB->setText(toqstr("Remove"));
+                       removePB->setText(qt_("Remo&ve"));
                        break;
                }
                case UserExtraUnbind: {
@@ -2116,8 +2417,8 @@ void PrefShortcuts::shortcut_clearPB_pressed()
 //
 /////////////////////////////////////////////////////////////////////
 
-PrefIdentity::PrefIdentity(QWidget * parent)
-       : PrefModule(_("Identity"), 0, parent)
+PrefIdentity::PrefIdentity(GuiPreferences * form)
+       : PrefModule(QString(), qt_("Identity"), form)
 {
        setupUi(this);
 
@@ -2161,33 +2462,35 @@ GuiPreferences::GuiPreferences(GuiView & lv)
        connect(closePB, SIGNAL(clicked()), this, SLOT(slotClose()));
        connect(restorePB, SIGNAL(clicked()), this, SLOT(slotRestore()));
 
-       add(new PrefUserInterface(this));
-       add(new PrefShortcuts(this));
-       add(new PrefScreenFonts(this));
-       add(new PrefColors(this));
-       add(new PrefDisplay);
-       add(new PrefInput(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(_("User interface"));
+       prefsPS->setCurrentPanel(qt_("User interface"));
 // FIXME: hack to work around resizing bug in Qt >= 4.2
 // bug verified with Qt 4.2.{0-3} (JSpitzm)
 #if QT_VERSION >= 0x040200
@@ -2202,10 +2505,13 @@ GuiPreferences::GuiPreferences(GuiView & lv)
 }
 
 
-void GuiPreferences::add(PrefModule * module)
+void GuiPreferences::addModule(PrefModule * module)
 {
-       BOOST_ASSERT(module);
-       prefsPS->addPanel(module, module->title());
+       LASSERT(module, /**/);
+       if (module->category().isEmpty())
+               prefsPS->addPanel(module, module->title());
+       else
+               prefsPS->addPanel(module, module->title(), module->category());
        connect(module, SIGNAL(changed()), this, SLOT(change_adaptor()));
        modules_.push_back(module);
 }
@@ -2293,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));
 }
 
 
@@ -2305,54 +2611,38 @@ void GuiPreferences::updateScreenFonts()
 }
 
 
-docstring const GuiPreferences::browsebind(docstring const & file) const
-{
-       return browseLibFile(from_ascii("bind"), file, from_ascii("bind"),
-                            _("Choose bind file"),
-                            FileFilterList(_("LyX bind files (*.bind)")));
-}
-
-
-docstring const GuiPreferences::browseUI(docstring const & file) const
+QString GuiPreferences::browsebind(QString const & file) const
 {
-       return browseLibFile(from_ascii("ui"), file, from_ascii("ui"),
-                            _("Choose UI file"),
-                            FileFilterList(_("LyX UI files (*.ui)")));
+       return browseLibFile("bind", file, "bind", qt_("Choose bind file"),
+                            QStringList(qt_("LyX bind files (*.bind)")));
 }
 
 
-docstring const GuiPreferences::browsekbmap(docstring const & file) const
+QString GuiPreferences::browseUI(QString const & file) const
 {
-       return browseLibFile(from_ascii("kbd"), file, from_ascii("kmap"),
-                            _("Choose keyboard map"),
-                            FileFilterList(_("LyX keyboard maps (*.kmap)")));
+       return browseLibFile("ui", file, "ui", qt_("Choose UI file"),
+                            QStringList(qt_("LyX UI files (*.ui)")));
 }
 
 
-docstring const GuiPreferences::browsedict(docstring const & file) const
+QString GuiPreferences::browsekbmap(QString const & file) const
 {
-       if (lyxrc.use_spell_lib)
-               return browseFile(file,
-                                 _("Choose personal dictionary"),
-                                 FileFilterList(_("*.pws")));
-       else
-               return browseFile(file,
-                                 _("Choose personal dictionary"),
-                                 FileFilterList(_("*.ispell")));
+       return browseLibFile("kbd", file, "kmap", qt_("Choose keyboard map"),
+                            QStringList(qt_("LyX keyboard maps (*.kmap)")));
 }
 
 
-docstring const GuiPreferences::browse(docstring const & file,
-                                 docstring const & title) const
+QString GuiPreferences::browsedict(QString const & file) const
 {
-       return browseFile(file, title, FileFilterList(), true);
+       return browseFile(file, qt_("Choose personal dictionary"),
+               QStringList(lyxrc.use_spell_lib ? qt_("*.pws") : qt_("*.ispell")));
 }
 
 
-docstring const GuiPreferences::browsedir(docstring const & path,
-                                    docstring const & title) const
+QString GuiPreferences::browse(QString const & file,
+                                 QString const & title) const
 {
-       return browseDir(path, title);
+       return browseFile(file, title, QStringList(), true);
 }