]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/qt4/GuiDocument.cpp
Andre's s/getTextClass/textClass/ cleanup.
[lyx.git] / src / frontends / qt4 / GuiDocument.cpp
index 7063e9e0e2d2c228a6e356280e8d2dc1db4b54e4..d49ad159e64b45094b1e03adb2b5545d05fb6d2e 100644 (file)
@@ -46,6 +46,8 @@
 #include "support/filetools.h"
 #include "support/lstrings.h"
 
+#include "frontends/alert.h"
+
 #include <boost/bind.hpp>
 
 #include <QCloseEvent>
@@ -58,6 +60,7 @@
 using namespace std;
 using namespace lyx::support;
 
+
 ///
 template<class Pair>
 vector<typename Pair::second_type> const
@@ -69,6 +72,7 @@ getSecond(vector<Pair> const & pr)
         return tmp;
 }
 
+
 char const * const tex_graphics[] =
 {
        "default", "dvips", "dvitops", "emtex",
@@ -134,6 +138,288 @@ vector<pair<string, lyx::docstring> > pagestyles;
 namespace lyx {
 namespace frontend {
 
+
+/// 
+QModelIndex getSelectedIndex(QListView * lv)
+{
+       QModelIndex retval = QModelIndex();
+       QModelIndexList selIdx = 
+                       lv->selectionModel()->selectedIndexes();
+       if (!selIdx.empty())
+               retval = selIdx.first();
+       return retval;
+}
+
+
+namespace {
+       vector<string> getRequiredList(string const & modName) 
+       {
+               LyXModule const * const mod = moduleList[modName];
+               if (!mod)
+                       return vector<string>(); //empty such thing
+               return mod->getRequiredModules();
+       }
+
+
+       vector<string> getExcludedList(string const & modName)
+       {
+               LyXModule const * const mod = moduleList[modName];
+               if (!mod)
+                       return vector<string>(); //empty such thing
+               return mod->getExcludedModules();
+       }
+
+
+       docstring getModuleDescription(string const & modName)
+       {
+               LyXModule const * const mod = moduleList[modName];
+               if (!mod)
+                       return _("Module not found!");
+               return _(mod->getDescription());
+       }
+
+
+       vector<string> getPackageList(string const & modName)
+       {
+               LyXModule const * const mod = moduleList[modName];
+               if (!mod)
+                       return vector<string>(); //empty such thing
+               return mod->getPackageList();
+       }
+
+
+       bool isModuleAvailable(string const & modName)
+       {
+               LyXModule * mod = moduleList[modName];
+               if (!mod)
+                       return false;
+               return mod->isAvailable();
+       }
+} //anonymous namespace
+
+
+ModuleSelMan::ModuleSelMan(
+       QListView * availableLV, 
+       QListView * selectedLV,
+       QPushButton * addPB, 
+       QPushButton * delPB, 
+       QPushButton * upPB, 
+       QPushButton * downPB,
+       GuiIdListModel * availableModel,
+       GuiIdListModel * selectedModel) :
+GuiSelectionManager(availableLV, selectedLV, addPB, delPB,
+                    upPB, downPB, availableModel, selectedModel) 
+{}
+       
+
+void ModuleSelMan::updateAddPB() 
+{
+       int const arows = availableModel->rowCount();
+       QModelIndexList const availSels = 
+                       availableLV->selectionModel()->selectedIndexes();
+       if (arows == 0 || availSels.isEmpty()  || isSelected(availSels.first())) {
+               addPB->setEnabled(false);
+               return;
+       }
+       
+       QModelIndex const & idx = availableLV->selectionModel()->currentIndex();
+       string const modName = getAvailableModel()->getIDString(idx.row());
+       vector<string> reqs = getRequiredList(modName);
+       vector<string> excl = getExcludedList(modName);
+       
+       if (reqs.empty() && excl.empty()) {
+               addPB->setEnabled(true);
+               return;
+       }
+
+       int const srows = selectedModel->rowCount();
+       vector<string> selModList;
+       for (int i = 0; i < srows; ++i)
+               selModList.push_back(getSelectedModel()->getIDString(i));
+
+       vector<string>::const_iterator selModStart = selModList.begin();
+       vector<string>::const_iterator selModEnd   = selModList.end();
+       
+       //Check whether some required module is available
+       if (!reqs.empty()) {
+               bool foundOne = false;
+               vector<string>::const_iterator it  = reqs.begin();
+               vector<string>::const_iterator end = reqs.end();
+               for (; it != end; ++it) {
+                       if (find(selModStart, selModEnd, *it) != selModEnd) {
+                               foundOne = true;
+                               break;
+                       }
+               }
+               if (!foundOne) {
+                       addPB->setEnabled(false);
+                       return;
+               }
+       }
+       
+       //Check whether any excluded module is being used
+       if (!excl.empty()) {
+               vector<string>::const_iterator it  = excl.begin();
+               vector<string>::const_iterator end = excl.end();
+               for (; it != end; ++it) {
+                       if (find(selModStart, selModEnd, *it) != selModEnd) {
+                               addPB->setEnabled(false);
+                               return;
+                       }
+               }
+       }
+
+       addPB->setEnabled(true);
+}
+
+
+void ModuleSelMan::updateDownPB()
+{
+       int const srows = selectedModel->rowCount();
+       if (srows == 0) {
+               downPB->setEnabled(false);
+               return;
+       }
+       QModelIndexList const selSels = 
+                       selectedLV->selectionModel()->selectedIndexes();
+       //disable if empty or last item is selected
+       if (selSels.empty() || selSels.first().row() == srows - 1) {
+               downPB->setEnabled(false);
+               return;
+       }
+       //determine whether immediately succeding element requires this one
+       QModelIndex const & curIdx = selectedLV->selectionModel()->currentIndex();
+       int curRow = curIdx.row();
+       if (curRow < 0 || curRow >= srows - 1) { //this shouldn't happen...
+               downPB->setEnabled(false);
+               return;
+       }
+       string const curModName = getSelectedModel()->getIDString(curRow);
+       string const nextModName = getSelectedModel()->getIDString(curRow + 1);
+
+       vector<string> reqs = getRequiredList(nextModName);
+
+       //if it doesn't require anything....
+       if (reqs.empty()) {
+               downPB->setEnabled(true);
+               return;
+       }
+
+       //FIXME This should perhaps be more flexible and check whether, even 
+       //if this one is required, there is also an earlier one that is required.
+       //enable it if this module isn't required
+       downPB->setEnabled(
+                       find(reqs.begin(), reqs.end(), curModName) == reqs.end());
+}
+
+void ModuleSelMan::updateUpPB() 
+{
+       int const srows = selectedModel->rowCount();
+       if (srows == 0) {
+               upPB->setEnabled(false);
+               return;
+       }
+       QModelIndexList const selSels = 
+                       selectedLV->selectionModel()->selectedIndexes();
+       //disable if empty or first item is selected
+       if (selSels.empty() || selSels.first().row() == 0) {
+               upPB->setEnabled(false);
+               return;
+       }
+
+       //determine whether immediately preceding element is required by this one
+       QModelIndex const & curIdx = selectedLV->selectionModel()->currentIndex();
+       int curRow = curIdx.row();
+       if (curRow <= -1 || curRow > srows - 1) { //sanity check
+               downPB->setEnabled(false);
+               return;
+       }
+       string const curModName = getSelectedModel()->getIDString(curRow);
+       vector<string> reqs = getRequiredList(curModName);
+       
+       //if this one doesn't require anything....
+       if (reqs.empty()) {
+               upPB->setEnabled(true);
+               return;
+       }
+
+       string preModName = getSelectedModel()->getIDString(curRow - 1);
+
+       //NOTE This is less flexible than it might be. You could check whether, even 
+       //if this one is required, there is also an earlier one that is required.
+       //enable it if the preceding module isn't required
+       upPB->setEnabled(find(reqs.begin(), reqs.end(), preModName) == reqs.end());
+}
+
+void ModuleSelMan::updateDelPB() 
+{
+       int const srows = selectedModel->rowCount();
+       if (srows == 0) {
+               deletePB->setEnabled(false);
+               return;
+       }
+       QModelIndexList const selSels = 
+                       selectedLV->selectionModel()->selectedIndexes();
+       if (selSels.empty() || selSels.first().row() < 0) {
+               deletePB->setEnabled(false);
+               return;
+       }
+       
+       //determine whether some LATER module requires this one
+       //NOTE Things are arranged so that this is the only way there
+       //can be a problem. At least, we hope so.
+       QModelIndex const & curIdx = 
+               selectedLV->selectionModel()->currentIndex();
+       int const curRow = curIdx.row();
+       if (curRow < 0 || curRow >= srows) { //this shouldn't happen
+               deletePB->setEnabled(false);
+               return;
+       }
+               
+       QString const curModName = curIdx.data().toString();
+       
+       //We're looking here for a reason NOT to enable the button. If we
+       //find one, we disable it and return. If we don't, we'll end up at
+       //the end of the function, and then we enable it.
+       for (int i = curRow + 1; i < srows; ++i) {
+               string const thisMod = getSelectedModel()->getIDString(i);
+               vector<string> reqs = getRequiredList(thisMod);
+               //does this one require us?
+               if (find(reqs.begin(), reqs.end(), fromqstr(curModName)) == reqs.end())
+                       //no...
+                       continue;
+
+               //OK, so this module requires us
+               //is there an EARLIER module that also satisfies the require?
+               //NOTE We demand that it be earlier to keep the list of modules
+               //consistent with the rule that a module must be proceeded by a
+               //required module. There would be more flexible ways to proceed,
+               //but that would be a lot more complicated, and the logic here is
+               //already complicated. (That's why I've left the debugging code.)
+               //lyxerr << "Testing " << thisMod << std::endl;
+               bool foundOne = false;
+               for (int j = 0; j < curRow; ++j) {
+                       string const mod = getSelectedModel()->getIDString(j);
+                       //lyxerr << "In loop: Testing " << mod << std::endl;
+                       //do we satisfy the require? 
+                       if (find(reqs.begin(), reqs.end(), mod) != reqs.end()) {
+                               //lyxerr << mod << " does the trick." << std::endl;
+                               foundOne = true;
+                               break;
+                       }
+               }
+               //did we find a module to satisfy the require?
+               if (!foundOne) {
+                       //lyxerr << "No matching module found." << std::endl;
+                       deletePB->setEnabled(false);
+                       return;
+               }
+       }
+       //lyxerr << "All's well that ends well." << std::endl;  
+       deletePB->setEnabled(true);
+}
+
+
 /////////////////////////////////////////////////////////////////////
 //
 // PreambleModule
@@ -205,11 +491,10 @@ void PreambleModule::closeEvent(QCloseEvent * e)
 
 
 GuiDocument::GuiDocument(GuiView & lv)
-       : GuiDialog(lv, "document")
+       : GuiDialog(lv, "document", qt_("Document Settings")), current_id_(0)
 {
        setupUi(this);
-       setViewTitle(_("Document Settings"));
-
+       
        lang_ = getSecond(getLanguageData(false));
 
        connect(okPB, SIGNAL(clicked()), this, SLOT(slotOK()));
@@ -233,7 +518,7 @@ GuiDocument::GuiDocument(GuiView & lv)
                this, SLOT(change_adaptor()));
        connect(textLayoutModule->lspacingCO, SIGNAL(activated(int)),
                this, SLOT(setLSpacing(int)));
-       connect(textLayoutModule->lspacingLE, SIGNAL(textChanged(const QString&)),
+       connect(textLayoutModule->lspacingLE, SIGNAL(textChanged(const QString &)),
                this, SLOT(change_adaptor()));
        connect(textLayoutModule->skipRB, SIGNAL(clicked()),
                this, SLOT(change_adaptor()));
@@ -251,6 +536,8 @@ GuiDocument::GuiDocument(GuiView & lv)
                this, SLOT(enableSkip(bool)));
        connect(textLayoutModule->twoColumnCB, SIGNAL(clicked()),
                this, SLOT(change_adaptor()));
+       connect(textLayoutModule->twoColumnCB, SIGNAL(clicked()),
+               this, SLOT(setColSep()));
        connect(textLayoutModule->listingsED, SIGNAL(textChanged()),
                this, SLOT(change_adaptor()));
        connect(textLayoutModule->bypassCB, SIGNAL(clicked()), 
@@ -433,6 +720,10 @@ GuiDocument::GuiDocument(GuiView & lv)
                this, SLOT(change_adaptor()));
        connect(marginsModule->footskipUnit, SIGNAL(activated(int)),
                this, SLOT(change_adaptor()));
+       connect(marginsModule->columnsepLE, SIGNAL(textChanged(const QString&)),
+               this, SLOT(change_adaptor()));
+       connect(marginsModule->columnsepUnit, SIGNAL(activated(int)),
+               this, SLOT(change_adaptor()));
        marginsModule->topLE->setValidator(unsignedLengthValidator(
                marginsModule->topLE));
        marginsModule->bottomLE->setValidator(unsignedLengthValidator(
@@ -447,6 +738,8 @@ GuiDocument::GuiDocument(GuiView & lv)
                marginsModule->headheightLE));
        marginsModule->footskipLE->setValidator(unsignedLengthValidator(
                marginsModule->footskipLE));
+       marginsModule->columnsepLE->setValidator(unsignedLengthValidator(
+               marginsModule->columnsepLE));
 
        bc().addCheckedLineEdit(marginsModule->topLE,
                marginsModule->topL);
@@ -462,6 +755,8 @@ GuiDocument::GuiDocument(GuiView & lv)
                marginsModule->headheightL);
        bc().addCheckedLineEdit(marginsModule->footskipLE,
                marginsModule->footskipL);
+       bc().addCheckedLineEdit(marginsModule->columnsepLE,
+               marginsModule->columnsepL);
 
 
        langModule = new UiWidget<Ui::LanguageUi>;
@@ -556,17 +851,17 @@ GuiDocument::GuiDocument(GuiView & lv)
 
        latexModule = new UiWidget<Ui::LaTeXUi>;
        // latex class
-       connect(latexModule->classCO, SIGNAL(activated(int)),
-               this, SLOT(change_adaptor()));
        connect(latexModule->optionsLE, SIGNAL(textChanged(const QString &)),
                this, SLOT(change_adaptor()));
        connect(latexModule->psdriverCO, SIGNAL(activated(int)),
                this, SLOT(change_adaptor()));
        connect(latexModule->classCO, SIGNAL(activated(int)),
                this, SLOT(classChanged()));
+       connect(latexModule->classCO, SIGNAL(activated(int)),
+               this, SLOT(change_adaptor()));
        
        selectionManager = 
-               new GuiSelectionManager(latexModule->availableLV, latexModule->selectedLV, 
+               new ModuleSelMan(latexModule->availableLV, latexModule->selectedLV, 
                        latexModule->addPB, latexModule->deletePB, 
                        latexModule->upPB, latexModule->downPB, 
                        availableModel(), selectedModel());
@@ -748,13 +1043,6 @@ void GuiDocument::set_listings_msg()
 }
 
 
-void GuiDocument::closeEvent(QCloseEvent * e)
-{
-       slotClose();
-       e->accept();
-}
-
-
 void GuiDocument::setLSpacing(int item)
 {
        textLayoutModule->lspacingLE->setEnabled(item == 3);
@@ -804,6 +1092,12 @@ void GuiDocument::setCustomPapersize(int papersize)
 }
 
 
+void GuiDocument::setColSep()
+{
+       setCustomMargins(marginsModule->marginCB->checkState() == Qt::Checked);
+}
+
+
 void GuiDocument::setCustomMargins(bool custom)
 {
        marginsModule->topL->setEnabled(!custom);
@@ -833,6 +1127,12 @@ void GuiDocument::setCustomMargins(bool custom)
        marginsModule->footskipL->setEnabled(!custom);
        marginsModule->footskipLE->setEnabled(!custom);
        marginsModule->footskipUnit->setEnabled(!custom);
+
+       bool const enableColSep = !custom && 
+                       textLayoutModule->twoColumnCB->checkState() == Qt::Checked;
+       marginsModule->columnsepL->setEnabled(enableColSep);
+       marginsModule->columnsepLE->setEnabled(enableColSep);
+       marginsModule->columnsepUnit->setEnabled(enableColSep);
 }
 
 
@@ -913,56 +1213,117 @@ void GuiDocument::updatePagestyle(string const & items, string const & sel)
 void GuiDocument::classChanged()
 {
        textclass_type const tc = latexModule->classCO->currentIndex();
-       bp_.setJustBaseClass(tc);
-       if (lyxrc.auto_reset_options)
+       bp_.setBaseClass(tc);
+       if (lyxrc.auto_reset_options) {
+               if (applyPB->isEnabled()) {
+                       int const ret = Alert::prompt(_("Unapplied changes"),
+                                       _("Some changes in the dialog were not yet applied."
+                                       "If you do not apply now, they will be lost after this action."),
+                                       1, 1, _("&Apply"), _("&Dismiss"));
+                       if (ret == 0)
+                               applyView();
+               }
                bp_.useClassDefaults();
-       updateContents();
+               forceUpdate();
+       }
+}
+
+
+namespace {
+       //This is an insanely complicated attempt to make this sort of thing
+       //work with RTL languages.
+       docstring formatStrVec(vector<string> const & v, docstring const & s) 
+       {
+               //this mess formats the list as "v[0], v[1], ..., [s] v[n]"
+               int const vSize = v.size();
+               if (v.size() == 0)
+                       return docstring();
+               else if (v.size() == 1) 
+                       return from_ascii(v[0]);
+               else if (v.size() == 2) {
+                       docstring retval = _("%1$s and %2$s");
+                       retval = subst(retval, _("and"), s);
+                       return bformat(retval, from_ascii(v[0]), from_ascii(v[1]));
+               }
+               //The idea here is to format all but the last two items...
+               docstring t2 = _("%1$s, %2$s");
+               docstring retval = from_ascii(v[0]);
+               for (int i = 1; i < vSize - 2; ++i)
+                       retval = bformat(t2, retval, from_ascii(v[i])); 
+               //...and then to  plug them, and the last two, into this schema
+               docstring t = _("%1$s, %2$s, and %3$s");
+               t = subst(t, _("and"), s);
+               return bformat(t, retval, from_ascii(v[vSize - 2]), from_ascii(v[vSize - 1]));
+       }
+       
+       vector<string> idsToNames(vector<string> const & idList)
+       {
+               vector<string> retval;
+               vector<string>::const_iterator it  = idList.begin();
+               vector<string>::const_iterator end = idList.end();
+               for (; it != end; ++it) {
+                       LyXModule const * const mod = moduleList[*it];
+                       if (!mod)
+                               retval.push_back(*it + " (Unavailable)");
+                       else
+                               retval.push_back(mod->getName());
+               }
+               return retval;
+       }
 }
 
 
 void GuiDocument::updateModuleInfo()
 {
        selectionManager->update();
+       
        //Module description
-       QListView const * const lv = selectionManager->selectedFocused() ?
-                                    latexModule->selectedLV :
-                       latexModule->availableLV;
-       if (lv->selectionModel()->selectedIndexes().isEmpty())
+       bool const focusOnSelected = selectionManager->selectedFocused();
+       QListView const * const lv = 
+                       focusOnSelected ? latexModule->selectedLV : latexModule->availableLV;
+       if (lv->selectionModel()->selectedIndexes().isEmpty()) {
                latexModule->infoML->document()->clear();
-       else {
-               QModelIndex const idx = lv->selectionModel()->currentIndex();
-               string const modName = fromqstr(idx.data().toString());
-               string desc = getModuleDescription(modName);
-               vector<string> pkgList = getPackageList(modName);
-               string pkgdesc;
-               //this mess formats the package list as "pkg1, pkg2, and pkg3"
-               int const pkgListSize = pkgList.size();
-               for (int i = 0; i < pkgListSize; ++i) {
-                       if (i == 1) {
-                               if (i == pkgListSize - 1) //last element
-                                       pkgdesc += " and ";
-                               else
-                                       pkgdesc += ", ";
-                       } else if (i > 1) {
-                               if (i == pkgListSize - 1) //last element
-                                       pkgdesc += ", and ";
-                               else
-                                       pkgdesc += ", ";
-                       }
-                       pkgdesc += pkgList[i];
-               }
-               if (!pkgdesc.empty()) {
-                       if (!desc.empty())
-                               desc += " ";
-                       desc += ("Requires " + pkgdesc + ".");
-               }
-               if (!isModuleAvailable(modName)) {
-                       if (!desc.empty())
-                               desc += "\n";
-                       desc += "WARNING: Some packages are unavailable!";
-               }
-               latexModule->infoML->document()->setPlainText(toqstr(desc));
+               return;
+       }
+       QModelIndex const & idx = lv->selectionModel()->currentIndex();
+       GuiIdListModel const & idModel = 
+                       focusOnSelected  ? selected_model_ : available_model_;
+       string const modName = idModel.getIDString(idx.row());
+       docstring desc = getModuleDescription(modName);
+
+       vector<string> pkgList = getPackageList(modName);
+       docstring pkgdesc = formatStrVec(pkgList, _("and"));
+       if (!pkgdesc.empty()) {
+               if (!desc.empty())
+                       desc += "\n";
+               desc += bformat(_("Package(s) required: %1$s."), pkgdesc);
+       }
+
+       pkgList = getRequiredList(modName);
+       if (!pkgList.empty()) {
+               vector<string> const reqDescs = idsToNames(pkgList);
+               pkgdesc = formatStrVec(reqDescs, _("or"));
+               if (!desc.empty())
+                       desc += "\n";
+               desc += bformat(_("Module required: %1$s."), pkgdesc);
+       }
+
+       pkgList = getExcludedList(modName);
+       if (!pkgList.empty()) {
+               vector<string> const reqDescs = idsToNames(pkgList);
+               pkgdesc = formatStrVec(reqDescs, _( "and"));
+               if (!desc.empty())
+                       desc += "\n";
+               desc += bformat(_("Modules excluded: %1$s."), pkgdesc);
+       }
+
+       if (!isModuleAvailable(modName)) {
+               if (!desc.empty())
+                       desc += "\n";
+               desc += _("WARNING: Some packages are unavailable!");
        }
+
+       latexModule->infoML->document()->setPlainText(toqstr(desc));
 }
 
 
@@ -970,10 +1331,10 @@ void GuiDocument::updateEmbeddedFileList()
 {
        embeddedFilesModule->filesLW->clear();
        // add current embedded files
-       EmbeddedFiles & files = buffer().embeddedFiles();
-       files.update();
-       EmbeddedFiles::EmbeddedFileList::iterator fit = files.begin();
-       EmbeddedFiles::EmbeddedFileList::iterator fit_end = files.end();
+       EmbeddedFileList & files = buffer().embeddedFiles();
+       files.update(buffer());
+       EmbeddedFileList::iterator fit = files.begin();
+       EmbeddedFileList::iterator fit_end = files.end();
        for (; fit != fit_end; ++fit) {
                QString label = toqstr(fit->relFilename(buffer().filePath()));
                if (fit->refCount() > 1)
@@ -993,7 +1354,7 @@ void GuiDocument::updateEmbeddedFileList()
 
 void GuiDocument::updateNumbering()
 {
-       TextClass const & tclass = bp_.getTextClass();
+       TextClass const & tclass = bp_.textClass();
 
        numberingModule->tocTW->setUpdatesEnabled(false);
        numberingModule->tocTW->clear();
@@ -1002,15 +1363,13 @@ void GuiDocument::updateNumbering()
        int const toc = numberingModule->tocSL->value();
        QString const no = qt_("No");
        QString const yes = qt_("Yes");
-       TextClass::const_iterator end = tclass.end();
-       TextClass::const_iterator cit = tclass.begin();
        QTreeWidgetItem * item = 0;
-       for ( ; cit != end ; ++cit) {
-               int const toclevel = (*cit)->toclevel;
-               if (toclevel != Layout::NOT_IN_TOC
-                   && (*cit)->labeltype == LABEL_COUNTER) {
+       for (size_t i = 0; i != tclass.layoutCount(); ++i) {
+               Layout const & lt = *tclass.layout(i);
+               int const toclevel = lt.toclevel;
+               if (toclevel != Layout::NOT_IN_TOC && lt.labeltype == LABEL_COUNTER) {
                        item = new QTreeWidgetItem(numberingModule->tocTW);
-                       item->setText(0, toqstr(translateIfPossible((*cit)->name())));
+                       item->setText(0, toqstr(translateIfPossible(lt.name())));
                        item->setText(1, (toclevel <= depth) ? yes : no);
                        item->setText(2, (toclevel <= toc) ? yes : no);
                }
@@ -1082,7 +1441,7 @@ void GuiDocument::apply(BufferParams & params)
        params.language = lyx::languages.getLanguage(lang_[pos]);
 
        // numbering
-       if (params.getTextClass().hasTocLevels()) {
+       if (params.textClass().hasTocLevels()) {
                params.tocdepth = numberingModule->tocSL->value();
                params.secnumdepth = numberingModule->depthSL->value();
        }
@@ -1097,12 +1456,15 @@ void GuiDocument::apply(BufferParams & params)
        params.graphicsDriver =
                tex_graphics[latexModule->psdriverCO->currentIndex()];
        
+       // text layout
+       params.setBaseClass(latexModule->classCO->currentIndex());
+
        // Modules
        params.clearLayoutModules();
-       QStringList const selMods = selectedModel()->stringList();
-       for (int i = 0; i != selMods.size(); ++i)
-               params.addLayoutModule(lyx::fromqstr(selMods[i]));
-
+       int const srows = selected_model_.rowCount();
+       vector<string> selModList;
+       for (int i = 0; i < srows; ++i)
+               params.addLayoutModule(selected_model_.getIDString(i));
 
        if (mathsModule->amsautoCB->isChecked()) {
                params.use_amsmath = BufferParams::package_auto;
@@ -1122,9 +1484,6 @@ void GuiDocument::apply(BufferParams & params)
                        params.use_esint = BufferParams::package_off;
        }
 
-       // text layout
-       params.setJustBaseClass(latexModule->classCO->currentIndex());
-
        if (pageLayoutModule->pagestyleCO->currentIndex() == 0)
                params.pagestyle = "default";
        else {
@@ -1261,6 +1620,7 @@ void GuiDocument::apply(BufferParams & params)
        params.headheight = widgetsToLength(m->headheightLE, m->headheightUnit);
        params.headsep = widgetsToLength(m->headsepLE, m->headsepUnit);
        params.footskip = widgetsToLength(m->footskipLE, m->footskipUnit);
+       params.columnsep = widgetsToLength(m->columnsepLE, m->columnsepUnit);
 
        branchesModule->apply(params);
 
@@ -1570,6 +1930,9 @@ void GuiDocument::updateParams(BufferParams const & params)
        lengthToWidgets(m->footskipLE, m->footskipUnit,
                params.footskip, defaultUnit);
 
+       lengthToWidgets(m->columnsepLE, m->columnsepUnit,
+               params.columnsep, defaultUnit);
+
        branchesModule->update(params);
 
        // PDF support
@@ -1617,37 +1980,78 @@ void GuiDocument::saveDocDefault()
 }
 
 
-void GuiDocument::updateContents()
+void GuiDocument::updateAvailableModules() 
+{
+       available_model_.clear();
+       vector<modInfoStruct> const modInfoList = getModuleInfo();
+       int const mSize = modInfoList.size();
+       for (int i = 0; i < mSize; ++i) {
+               modInfoStruct const & modInfo = modInfoList[i];
+               available_model_.insertRow(i, qt_(modInfo.name), modInfo.id);
+       }
+}
+
+
+void GuiDocument::updateSelectedModules() 
 {
-       //update list of available modules
-       QStringList strlist;
-       vector<string> const modNames = getModuleNames();
-       vector<string>::const_iterator it = modNames.begin();
-       for (; it != modNames.end(); ++it)
-               strlist.push_back(toqstr(*it));
-       available_model_.setStringList(strlist);
        //and selected ones, too
-       QStringList strlist2;
-       vector<string> const & selMods = getSelectedModules();
-       it = selMods.begin();
-       for (; it != selMods.end(); ++it)
-               strlist2.push_back(toqstr(*it));
-       selected_model_.setStringList(strlist2);
+       selected_model_.clear();
+       vector<modInfoStruct> const selModList = getSelectedModules();
+       int const sSize = selModList.size();
+       for (int i = 0; i < sSize; ++i) {
+               modInfoStruct const & modInfo = selModList[i];
+               selected_model_.insertRow(i, qt_(modInfo.name), modInfo.id);
+       }
+}
 
+
+void GuiDocument::updateContents()
+{
+       if (id() == current_id_)
+               return;
+
+       updateAvailableModules();
+       updateSelectedModules();
+       
+       //FIXME It'd be nice to make sure here that the selected
+       //modules are consistent: That required modules are actually
+       //selected, and that we don't have conflicts. If so, we could
+       //at least pop up a warning.
        updateParams(bp_);
+       current_id_ = id();
 }
 
+
+void GuiDocument::forceUpdate()
+{
+       // reset to force dialog update
+       current_id_ = 0;
+       updateContents();
+}
+
+
 void GuiDocument::useClassDefaults()
 {
-       bp_.setJustBaseClass(latexModule->classCO->currentIndex());
+       if (applyPB->isEnabled()) {
+               int const ret = Alert::prompt(_("Unapplied changes"),
+                               _("Some changes in the dialog were not yet applied."
+                                 "If you do not apply now, they will be lost after this action."),
+                               1, 1, _("&Apply"), _("&Dismiss"));
+               if (ret == 0)
+                       applyView();
+       }
+
+       bp_.setBaseClass(latexModule->classCO->currentIndex());
        bp_.useClassDefaults();
-       updateContents();
+       forceUpdate();
 }
 
 
 bool GuiDocument::isValid()
 {
-       return validate_listings_params().empty();
+       return (validate_listings_params().empty() &&
+               (textLayoutModule->skipCO->currentIndex() != 3 ||
+                !textLayoutModule->skipLE->text().isEmpty()));
 }
 
 
@@ -1664,7 +2068,7 @@ char const * GuiDocument::fontfamilies_gui[5] = {
 bool GuiDocument::initialiseParams(string const &)
 {
        bp_ = buffer().params();
-       loadModuleNames();
+       loadModuleInfo();
        return true;
 }
 
@@ -1681,40 +2085,29 @@ BufferId GuiDocument::id() const
 }
 
 
-vector<string> GuiDocument::getModuleNames()
+vector<GuiDocument::modInfoStruct> const & GuiDocument::getModuleInfo()
 {
        return moduleNames_;
 }
 
 
-vector<string> const & GuiDocument::getSelectedModules()
-{
-       return params().getModules();
-}
-
-
-string GuiDocument::getModuleDescription(string const & modName) const
-{
-       LyXModule const * const mod = moduleList[modName];
-       if (!mod)
-               return string("Module not found!");
-       return mod->description;
-}
-
-
-vector<string> GuiDocument::getPackageList(string const & modName) const
-{
-       LyXModule const * const mod = moduleList[modName];
-       if (!mod)
-               return vector<string>(); //empty such thing
-       return mod->packageList;
-}
-
-
-bool GuiDocument::isModuleAvailable(string const & modName) const
-{
-       LyXModule * mod = moduleList[modName];
-       return mod->isAvailable();
+vector<GuiDocument::modInfoStruct> const GuiDocument::getSelectedModules()
+{
+       vector<string> const & mods = params().getModules();
+       vector<string>::const_iterator it =  mods.begin();
+       vector<string>::const_iterator end = mods.end();
+       vector<modInfoStruct> mInfo;
+       for (; it != end; ++it) {
+               modInfoStruct m;
+               m.id = *it;
+               LyXModule * mod = moduleList[*it];
+               if (mod)
+                       m.name = mod->getName();
+               else 
+                       m.name = *it + " (Not Found)";
+               mInfo.push_back(m);
+       }
+       return mInfo;
 }
 
 
@@ -1742,8 +2135,6 @@ void GuiDocument::dispatchParams()
 
        // Apply the BufferParams. Note that this will set the base class
        // and then update the buffer's layout.
-       //FIXME Could this be done last? Then, I think, we'd get the automatic
-       //update mentioned in the next FIXME...
        dispatch_bufferparams(*this, params(), LFUN_BUFFER_PARAMS_APPLY);
 
        // Generate the colours requested by each new branch.
@@ -1836,14 +2227,17 @@ bool GuiDocument::providesScale(string const & font) const
 }
 
 
-void GuiDocument::loadModuleNames ()
+void GuiDocument::loadModuleInfo()
 {
        moduleNames_.clear();
-       LyXModuleList::const_iterator it = moduleList.begin();
-       for (; it != moduleList.end(); ++it)
-               moduleNames_.push_back(it->name);
-       if (!moduleNames_.empty())
-               sort(moduleNames_.begin(), moduleNames_.end());
+       LyXModuleList::const_iterator it  = moduleList.begin();
+       LyXModuleList::const_iterator end = moduleList.end();
+       for (; it != end; ++it) {
+               modInfoStruct m;
+               m.id = it->getID();
+               m.name = it->getName();
+               moduleNames_.push_back(m);
+       }
 }