]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/xforms/FormDocument.C
remove defaults stuff, let Qt handle no toolbar
[lyx.git] / src / frontends / xforms / FormDocument.C
index 40e553cc31ab9426169710a657e351d22c48c40b..3a6057fb14293e8491d45389ac7abeff1c96cd12 100644 (file)
@@ -1,69 +1,60 @@
 /**
  * \file FormDocument.C
- * Copyright 2000-2001 The LyX Team.
- * See the file COPYING.
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- * \author Jürgen Vigna, jug@sad.it
+ * \author Jürgen Vigna
+ * \author Rob Lahaye
+ *
+ * Full author contact details are available in file CREDITS
  */
 
 #include <config.h>
 
-#include FORMS_H_LOCATION
-#include XPM_H_LOCATION
-
-#ifdef __GNUG__
-#pragma implementation
-#endif
 
+#include "ControlDocument.h"
 #include "FormDocument.h"
-#include "form_document.h"
-#include "Alert.h"
-#include "Dialogs.h"
-#include "lyxtextclasslist.h"
-#include "combox.h"
-#include "tex-strings.h"
-#include "bufferparams.h"
-#include "vspace.h"
+#include "forms/form_document.h"
+#include "xformsBC.h"
+#include "ButtonController.h"
+
 #include "bmtable.h"
-#include "language.h"
-#include "frnt_lang.h"
-#include "frontends/LyXView.h"
-#include "lyxfunc.h"
-#include "lyxrc.h"
-#include "BufferView.h"
-#include "buffer.h"
-#include "Liason.h"
-#include "CutAndPaste.h"
-#include "bufferview_funcs.h"
+#include "checkedwidgets.h"
+#include "input_validators.h" // fl_unsigned_float_filter
 #include "xforms_helpers.h"
+
+#include "CutAndPaste.h"
 #include "debug.h"
-#include "input_validators.h" // fl_unsigned_float_filter
-#include "helper_funcs.h"
-#include "lyx_main.h" // for user_lyxdir
+#include "language.h"
+#include "lyxrc.h"
+#include "lyxtextclasslist.h"
+#include "tex-strings.h"
+
+#include "controllers/frnt_lang.h"
+#include "controllers/helper_funcs.h"
 
-#include "insets/insetquotes.h"
+#include "support/lstrings.h" // contains_functor, getStringFromVector
+#include "support/filetools.h" // LibFileSearch
 
-#include "support/filetools.h"
-#include "support/lstrings.h"
+#include XPM_H_LOCATION
+#include FORMS_H_LOCATION
+#include "combox.h"
+
+#include <boost/bind.hpp>
 
 #include <functional>
 
 using std::bind2nd;
-
-using Liason::setMinibuffer;
-using SigC::slot;
 using std::vector;
 
-FormDocument::FormDocument(LyXView * lv, Dialogs * d)
-       : FormBaseBD(lv, d, _("Document Layout")),
+
+typedef FormCB<ControlDocument, FormDB<FD_document> > base_class;
+
+FormDocument::FormDocument()
+       : base_class(_("Document Layout"), false),
          ActCell(0), Confirmed(0),
          current_bullet_panel(0), current_bullet_depth(0), fbullet(0)
-{
-       // let the dialog be shown
-       // This is a permanent connection so we won't bother
-       // storing a copy because we won't be disconnecting.
-       d->showDocument.connect(slot(this, &FormDocument::show));
-}
+{}
 
 
 void FormDocument::redraw()
@@ -79,52 +70,128 @@ void FormDocument::redraw()
 }
 
 
-FL_FORM * FormDocument::form() const
-{
-       if (dialog_.get()) return dialog_->form;
-       return 0;
-}
-
-
 void FormDocument::build()
 {
-       int n;
-
        // the tabbed folder
-       dialog_.reset(build_tabbed_document());
+       dialog_.reset(build_document(this));
 
        // Allow the base class to control messages
        setMessageWidget(dialog_->text_warning);
 
        // Manage the restore, ok, apply, restore and cancel/close buttons
-       bc().setOK(dialog_->button_ok);
-       bc().setApply(dialog_->button_apply);
-       bc().setCancel(dialog_->button_close);
-       bc().setRestore(dialog_->button_restore);
-       bc().addReadOnly (dialog_->button_save_defaults);
-       bc().addReadOnly (dialog_->button_reset_defaults);
+       bcview().setOK(dialog_->button_ok);
+       bcview().setApply(dialog_->button_apply);
+       bcview().setCancel(dialog_->button_close);
+       bcview().setRestore(dialog_->button_restore);
 
-       // the document paper form
-       paper_.reset(build_doc_paper());
-       fl_addto_choice(paper_->choice_papersize,
-                       _(" Default | Custom | US letter | US legal "
-                         "| US executive | A3 | A4 | A5 | B3 | B4 | B5 "));
-       fl_addto_choice(paper_->choice_paperpackage,
-                       _(" None "
-                         "| Small Margins "
-                         "| Very small Margins "
-                         "| Very wide Margins "));
+       // the document class form
+       class_.reset(build_document_class(this));
+
+       // disable for read-only documents
+       bcview().addReadOnly(class_->combox_doc_class);
+       bcview().addReadOnly(class_->radio_doc_indent);
+       bcview().addReadOnly(class_->radio_doc_skip);
+       bcview().addReadOnly(class_->choice_doc_pagestyle);
+       bcview().addReadOnly(class_->choice_doc_fonts);
+       bcview().addReadOnly(class_->choice_doc_fontsize);
+       bcview().addReadOnly(class_->radio_doc_sides_one);
+       bcview().addReadOnly(class_->radio_doc_sides_two);
+       bcview().addReadOnly(class_->radio_doc_columns_one);
+       bcview().addReadOnly(class_->radio_doc_columns_two);
+       bcview().addReadOnly(class_->input_doc_extra);
+       bcview().addReadOnly(class_->input_doc_skip);
+       bcview().addReadOnly(class_->choice_doc_skip);
+       bcview().addReadOnly(class_->choice_doc_spacing);
+       bcview().addReadOnly(class_->input_doc_spacing);
+
+       // check validity of "length + unit" input
+       addCheckedGlueLength(bcview(), class_->input_doc_skip,
+                            class_->choice_doc_skip);
+
+       // trigger an input event for cut&paste with middle mouse button.
+       setPrehandler(class_->input_doc_extra);
+       setPrehandler(class_->input_doc_skip);
+       setPrehandler(class_->input_doc_spacing);
 
-       fl_set_input_return(paper_->input_custom_width,  FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_custom_height, FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_top_margin,    FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_bottom_margin, FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_inner_margin,  FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_outer_margin,  FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_head_height,   FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_head_sep,      FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_foot_skip,     FL_RETURN_CHANGED);
+       fl_set_input_return(class_->input_doc_extra, FL_RETURN_CHANGED);
+       fl_set_input_return(class_->input_doc_skip, FL_RETURN_CHANGED);
+       fl_set_input_return(class_->input_doc_spacing, FL_RETURN_CHANGED);
+
+       FL_OBJECT * obj;
+
+       // Fill the combox and choices.
+       obj = class_->combox_doc_class;
+       LyXTextClassList::const_iterator tit  = textclasslist.begin();
+       LyXTextClassList::const_iterator tend = textclasslist.end();
+       for (; tit != tend; ++tit) {
+               fl_addto_combox(obj, tit->description().c_str());
+       }
+       fl_set_combox_browser_height(obj, 400);
 
+       fl_addto_choice(class_->choice_doc_spacing,
+                       _(" Single | OneHalf | Double | Custom "));
+       fl_addto_choice(class_->choice_doc_fontsize, "default|10|11|12");
+       for (int n = 0; tex_fonts[n][0]; ++n) {
+               fl_addto_choice(class_->choice_doc_fonts,tex_fonts[n]);
+       }
+
+       // Create the contents of the unit choices; don't include the
+       // "%" terms...
+       vector<string> units_vec = getLatexUnits();
+       vector<string>::iterator ret =
+               std::remove_if(units_vec.begin(),
+                              units_vec.end(),
+                              bind2nd(contains_functor(), "%"));
+       units_vec.erase(ret, units_vec.end());
+
+       string const units = getStringFromVector(units_vec, "|");
+
+       fl_addto_choice(class_->choice_doc_pagestyle,
+                       "default|empty|plain|headings|fancy");
+       fl_addto_choice(class_->choice_doc_skip,
+                       _(" SmallSkip | MedSkip | BigSkip | Length "));
+       fl_addto_choice(class_->choice_doc_skip_units,  units.c_str());
+
+       // Set input filters on doc spacing to make it accept only
+       // unsigned numbers.
+       fl_set_input_filter(class_->input_doc_spacing,
+                           fl_unsigned_float_filter);
+
+       // disable for read-only documents
+       bcview().addReadOnly(dialog_->button_save_defaults);
+       bcview().addReadOnly(dialog_->button_reset_defaults);
+
+       // the document paper form
+       paper_.reset(build_document_paper(this));
+
+       // disable for read-only documents
+       bcview().addReadOnly(paper_->choice_paperpackage);
+       bcview().addReadOnly(paper_->radio_portrait);
+       bcview().addReadOnly(paper_->radio_landscape);
+       bcview().addReadOnly(paper_->choice_papersize);
+       bcview().addReadOnly(paper_->check_use_geometry);
+       bcview().addReadOnly(paper_->input_custom_width);
+       bcview().addReadOnly(paper_->input_custom_height);
+       bcview().addReadOnly(paper_->input_top_margin);
+       bcview().addReadOnly(paper_->input_bottom_margin);
+       bcview().addReadOnly(paper_->input_inner_margin);
+       bcview().addReadOnly(paper_->input_outer_margin);
+       bcview().addReadOnly(paper_->input_head_height);
+       bcview().addReadOnly(paper_->input_head_sep);
+       bcview().addReadOnly(paper_->input_foot_skip);
+
+       // check validity of "length + unit" input
+       addCheckedGlueLength(bcview(), paper_->input_custom_width);
+       addCheckedGlueLength(bcview(), paper_->input_custom_height);
+       addCheckedGlueLength(bcview(), paper_->input_top_margin);
+       addCheckedGlueLength(bcview(), paper_->input_bottom_margin);
+       addCheckedGlueLength(bcview(), paper_->input_inner_margin);
+       addCheckedGlueLength(bcview(), paper_->input_outer_margin);
+       addCheckedGlueLength(bcview(), paper_->input_head_height);
+       addCheckedGlueLength(bcview(), paper_->input_head_sep);
+       addCheckedGlueLength(bcview(), paper_->input_foot_skip);
+
+       // trigger an input event for cut&paste with middle mouse button.
        setPrehandler(paper_->input_custom_width);
        setPrehandler(paper_->input_custom_height);
        setPrehandler(paper_->input_top_margin);
@@ -135,23 +202,24 @@ void FormDocument::build()
        setPrehandler(paper_->input_head_sep);
        setPrehandler(paper_->input_foot_skip);
 
-       // Create the contents of the unit choices
-       // Don't include the "%" terms...
-       vector<string> units_vec = getLatexUnits();
-#if 0
-       for (vector<string>::iterator it = units_vec.begin();
-            it != units_vec.end(); ++it) {
-               if (contains(*it, "%"))
-                       it = units_vec.erase(it, it+1) - 1;
-       }
-#else
-       vector<string>::iterator ret =
-               std::remove_if(units_vec.begin(),
-                              units_vec.end(),
-                              bind2nd(contains_functor(), "%"));
-       units_vec.erase(ret, units_vec.end());
-#endif
-       string units = getStringFromVector(units_vec, "|");
+       fl_set_input_return(paper_->input_custom_width,  FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_custom_height, FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_top_margin,    FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_bottom_margin, FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_inner_margin,  FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_outer_margin,  FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_head_height,   FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_head_sep,      FL_RETURN_CHANGED);
+       fl_set_input_return(paper_->input_foot_skip,     FL_RETURN_CHANGED);
+
+       fl_addto_choice(paper_->choice_papersize,
+                       _(" Default | Custom | US letter | US legal "
+                         "| US executive | A3 | A4 | A5 | B3 | B4 | B5 "));
+       fl_addto_choice(paper_->choice_paperpackage,
+                       _(" None "
+                         "| Small Margins "
+                         "| Very small Margins "
+                         "| Very wide Margins "));
 
        fl_addto_choice(paper_->choice_custom_width_units,  units.c_str());
        fl_addto_choice(paper_->choice_custom_height_units, units.c_str());
@@ -163,166 +231,96 @@ void FormDocument::build()
        fl_addto_choice(paper_->choice_head_sep_units,      units.c_str());
        fl_addto_choice(paper_->choice_foot_skip_units,     units.c_str());
 
-       bc().addReadOnly (paper_->choice_paperpackage);
-       bc().addReadOnly (paper_->radio_portrait);
-       bc().addReadOnly (paper_->radio_landscape);
-       bc().addReadOnly (paper_->choice_papersize);
-       bc().addReadOnly (paper_->check_use_geometry);
-       bc().addReadOnly (paper_->input_custom_width);
-       bc().addReadOnly (paper_->input_custom_height);
-       bc().addReadOnly (paper_->input_top_margin);
-       bc().addReadOnly (paper_->input_bottom_margin);
-       bc().addReadOnly (paper_->input_inner_margin);
-       bc().addReadOnly (paper_->input_outer_margin);
-       bc().addReadOnly (paper_->input_head_height);
-       bc().addReadOnly (paper_->input_head_sep);
-       bc().addReadOnly (paper_->input_foot_skip);
-
-       // the document class form
-       class_.reset(build_doc_class());
-
-       FL_OBJECT * obj;
-       // The language is a combo-box and has to be inserted manually
-       obj = class_->choice_doc_class;
-       fl_deactivate_object(obj);
-       fl_addto_form(class_->form);
-       combo_doc_class.reset(new Combox(FL_COMBOX_DROPLIST));
-       combo_doc_class->add(obj->x, obj->y, obj->w, obj->h, 400,
-                            dialog_->tabfolder);
-       combo_doc_class->shortcut("#C",1);
-       combo_doc_class->setcallback(ComboInputCB, this);
-       fl_end_form();
-       for (LyXTextClassList::const_iterator cit = textclasslist.begin();
-            cit != textclasslist.end(); ++cit) {
-               combo_doc_class->addto(cit->description());
-       }
-
-       fl_addto_choice(class_->choice_doc_spacing,
-                       _(" Single | OneHalf | Double | Custom "));
-       fl_addto_choice(class_->choice_doc_fontsize, "default|10|11|12");
-       for (n=0; tex_fonts[n][0]; ++n) {
-               fl_addto_choice(class_->choice_doc_fonts,tex_fonts[n]);
-       }
-       fl_addto_choice(class_->choice_doc_pagestyle,
-                       "default|empty|plain|headings|fancy");
-       fl_addto_choice(class_->choice_doc_skip,
-                       _(" Smallskip | Medskip | Bigskip | Length "));
-       fl_addto_choice(class_->choice_default_skip_units,  units.c_str());
-       fl_set_input_return(class_->input_doc_extra, FL_RETURN_CHANGED);
-       fl_set_input_return(class_->input_doc_skip, FL_RETURN_CHANGED);
-       fl_set_input_return(class_->input_doc_spacing, FL_RETURN_CHANGED);
+       // the document language form
+       language_.reset(build_document_language(this));
 
-       setPrehandler(class_->input_doc_extra);
-       setPrehandler(class_->input_doc_skip);
-       setPrehandler(class_->input_doc_spacing);
+       // disable for read-only documents
+       bcview().addReadOnly(language_->combox_language);
+       bcview().addReadOnly(language_->choice_inputenc);
+       bcview().addReadOnly(language_->choice_quotes_language);
+       bcview().addReadOnly(language_->radio_single);
+       bcview().addReadOnly(language_->radio_double);
 
-       // Set input filters on doc spacing to make it accept only
-       // unsigned numbers.
-       fl_set_input_filter(class_->input_doc_spacing, fl_unsigned_float_filter);
-
-       bc().addReadOnly (class_->radio_doc_indent);
-       bc().addReadOnly (class_->radio_doc_skip);
-
-       bc().addReadOnly (class_->choice_doc_pagestyle);
-       bc().addReadOnly (class_->choice_doc_fonts);
-       bc().addReadOnly (class_->choice_doc_fontsize);
-       bc().addReadOnly (class_->radio_doc_sides_one);
-       bc().addReadOnly (class_->radio_doc_sides_two);
-       bc().addReadOnly (class_->radio_doc_columns_one);
-       bc().addReadOnly (class_->radio_doc_columns_two);
-       bc().addReadOnly (class_->input_doc_extra);
-       bc().addReadOnly (class_->input_doc_skip);
-       bc().addReadOnly (class_->choice_doc_skip);
-       bc().addReadOnly (class_->choice_doc_spacing);
-       bc().addReadOnly (class_->input_doc_spacing);
-
-       // the document language form
-       language_.reset(build_doc_language());
        fl_addto_choice(language_->choice_inputenc,
                        "default|auto|latin1|latin2|latin3|latin4|latin5|latin9"
-                       "|koi8-r|koi8-u|cp866|cp1251|iso88595");
+                       "|koi8-r|koi8-u|cp866|cp1251|iso88595|pt154");
 
        vector<frnt::LanguagePair> const langs = frnt::getLanguageData(false);
        // Store the identifiers for later
        lang_ = getSecond(langs);
 
-       // The language is a combo-box and has to be inserted manually
-       obj = language_->choice_language;
-       fl_deactivate_object(obj);
-       fl_addto_form(language_->form);
-       combo_language.reset(new Combox(FL_COMBOX_DROPLIST));
-       combo_language->add(obj->x, obj->y, obj->w, obj->h, 400,
-                           dialog_->tabfolder);
-       combo_language->shortcut("#L",1);
-       combo_language->setcallback(ComboInputCB, this);
-       fl_end_form();
-
        vector<frnt::LanguagePair>::const_iterator lit  = langs.begin();
        vector<frnt::LanguagePair>::const_iterator lend = langs.end();
        for (; lit != lend; ++lit) {
-               combo_language->addto(lit->first);
+               fl_addto_combox(language_->combox_language,
+                               lit->first.c_str());
        }
-       combo_language->select(1);
+       fl_set_combox_browser_height(language_->combox_language, 400);
 
        fl_addto_choice(language_->choice_quotes_language,
                        _(" ``text'' | ''text'' | ,,text`` | ,,text'' |"
                          " «text» | »text« "));
 
-       bc().addReadOnly (language_->choice_inputenc);
-       bc().addReadOnly (language_->choice_quotes_language);
-       bc().addReadOnly (language_->radio_single);
-       bc().addReadOnly (language_->radio_double);
-
        // the document options form
-       options_.reset(build_doc_options());
-       fl_set_input_return(options_->input_float_placement, FL_RETURN_CHANGED);
+       options_.reset(build_document_options(this));
+
+       // disable for read-only documents
+       bcview().addReadOnly(options_->counter_secnumdepth);
+       bcview().addReadOnly(options_->counter_tocdepth);
+       bcview().addReadOnly(options_->choice_ams_math);
+       bcview().addReadOnly(options_->check_use_natbib);
+       bcview().addReadOnly(options_->choice_citation_format);
+       bcview().addReadOnly(options_->input_float_placement);
+       bcview().addReadOnly(options_->choice_postscript_driver);
+
+       // trigger an input event for cut&paste with middle mouse button.
        setPrehandler(options_->input_float_placement);
 
-       fl_set_counter_bounds(options_->counter_secnumdepth,-2,5);
-       fl_set_counter_bounds(options_->counter_tocdepth,-1,5);
-       fl_set_counter_step(options_->counter_secnumdepth,1,1);
-       fl_set_counter_step(options_->counter_tocdepth,1,1);
-       fl_set_counter_precision(options_->counter_secnumdepth, 0);
-       fl_set_counter_precision(options_->counter_tocdepth, 0);
-       for (n=0; tex_graphics[n][0]; ++n) {
+       fl_set_input_return(options_->input_float_placement, FL_RETURN_CHANGED);
+
+       fl_addto_choice(options_->choice_ams_math,
+                       _("Never | Automatically | Yes "));
+
+       for (int n = 0; tex_graphics[n][0]; ++n) {
                fl_addto_choice(options_->choice_postscript_driver,
                                tex_graphics[n]);
        }
        fl_addto_choice(options_->choice_citation_format,
                        _(" Author-year | Numerical "));
 
-       bc_.addReadOnly (options_->counter_secnumdepth);
-       bc_.addReadOnly (options_->counter_tocdepth);
-       bc_.addReadOnly (options_->check_use_amsmath);
-       bc_.addReadOnly (options_->check_use_natbib);
-       bc_.addReadOnly (options_->choice_citation_format);
-       bc_.addReadOnly (options_->input_float_placement);
-       bc_.addReadOnly (options_->choice_postscript_driver);
-
        // the document bullets form
-       bullets_.reset(build_doc_bullet());
+       bullets_.reset(build_document_bullet(this));
+
+       // disable for read-only documents
+       bcview().addReadOnly(bullets_->radio_bullet_depth_1);
+       bcview().addReadOnly(bullets_->radio_bullet_depth_2);
+       bcview().addReadOnly(bullets_->radio_bullet_depth_3);
+       bcview().addReadOnly(bullets_->radio_bullet_depth_4);
+       bcview().addReadOnly(bullets_->radio_bullet_panel_standard);
+       bcview().addReadOnly(bullets_->radio_bullet_panel_maths);
+       bcview().addReadOnly(bullets_->radio_bullet_panel_ding1);
+       bcview().addReadOnly(bullets_->radio_bullet_panel_ding2);
+       bcview().addReadOnly(bullets_->radio_bullet_panel_ding3);
+       bcview().addReadOnly(bullets_->radio_bullet_panel_ding4);
+       bcview().addReadOnly(bullets_->bmtable_bullet_panel);
+       bcview().addReadOnly(bullets_->choice_bullet_size);
+       bcview().addReadOnly(bullets_->input_bullet_latex);
+
+       // trigger an input event for cut&paste with middle mouse button.
+       setPrehandler(bullets_->input_bullet_latex);
+
+       fl_set_input_return(bullets_->input_bullet_latex, FL_RETURN_CHANGED);
+
        fl_addto_choice(bullets_->choice_bullet_size,
-                       _(" default | tiny | script | footnote | small |"
-                         " normal | large | Large | LARGE | huge | Huge"));
+                       _(" Default | Tiny | Smallest | Smaller | Small |"
+                         " Normal | Large | Larger | Largest | Huge | Huger "));
        fl_set_choice(bullets_->choice_bullet_size, 1);
-       fl_set_input_return(bullets_->input_bullet_latex, FL_RETURN_CHANGED);
-       setPrehandler(bullets_->input_bullet_latex);
+
        fl_set_input_maxchars(bullets_->input_bullet_latex, 80);
 
-       bc_.addReadOnly(bullets_->radio_bullet_depth_1);
-       bc_.addReadOnly(bullets_->radio_bullet_depth_2);
-       bc_.addReadOnly(bullets_->radio_bullet_depth_3);
-       bc_.addReadOnly(bullets_->radio_bullet_depth_4);
-       bc_.addReadOnly(bullets_->radio_bullet_panel_standard);
-       bc_.addReadOnly(bullets_->radio_bullet_panel_maths);
-       bc_.addReadOnly(bullets_->radio_bullet_panel_ding1);
-       bc_.addReadOnly(bullets_->radio_bullet_panel_ding2);
-       bc_.addReadOnly(bullets_->radio_bullet_panel_ding3);
-       bc_.addReadOnly(bullets_->radio_bullet_panel_ding4);
-
-       bc().addReadOnly (bullets_->bmtable_bullet_panel);
-       bc().addReadOnly (bullets_->choice_bullet_size);
-       bc().addReadOnly (bullets_->input_bullet_latex);
+       string const bmtablefile = LibFileSearch("images", "standard", "xpm");
+       fl_set_bmtable_pixmap_file(bullets_->bmtable_bullet_panel, 6, 6,
+                                  bmtablefile.c_str());
 
        fl_addto_tabfolder(dialog_->tabfolder,_("Document"),
                           class_->form);
@@ -334,6 +332,10 @@ void FormDocument::build()
                           options_->form);
        fbullet = fl_addto_tabfolder(dialog_->tabfolder,_("Bullets"),
                                     bullets_->form);
+
+       // work-around xforms bug re update of folder->x, folder->y coords.
+       setPrehandler(dialog_->tabfolder);
+
        if ((XpmVersion < 4) || (XpmVersion == 4 && XpmRevision < 7)) {
                lyxerr << _("Your version of libXpm is older than 4.7.\n"
                            "The `bullet' tab of the document dialog "
@@ -346,32 +348,13 @@ void FormDocument::build()
 
 void FormDocument::apply()
 {
-       if (!lv_->view()->available() || !dialog_.get())
-               return;
+       BufferParams & params = controller().params();
 
-       bool redo = class_apply();
-       paper_apply();
-       redo = language_apply() || redo;
-       redo = options_apply() || redo;
-       bullets_apply();
-
-       if (redo) {
-               lv_->view()->redoCurrentBuffer();
-       }
-       lv_->buffer()->markDirty();
-       setMinibuffer(lv_, _("Document layout set"));
-}
-
-
-void FormDocument::cancel()
-{
-       // this avoids confusion when reopening
-       BufferParams & param = lv_->buffer()->params;
-       param.temp_bullets[0] = param.user_defined_bullets[0];
-       param.temp_bullets[1] = param.user_defined_bullets[1];
-       param.temp_bullets[2] = param.user_defined_bullets[2];
-       param.temp_bullets[3] = param.user_defined_bullets[3];
-       hide();
+       class_apply(params);
+       paper_apply(params);
+       language_apply(params);
+       options_apply(params);
+       bullets_apply(params);
 }
 
 
@@ -382,7 +365,7 @@ void FormDocument::update()
 
        checkReadOnly();
 
-       BufferParams const & params = lv_->buffer()->params;
+       BufferParams const & params = controller().params();
 
        class_update(params);
        paper_update(params);
@@ -392,126 +375,79 @@ void FormDocument::update()
 }
 
 
-namespace {
-// should this go elsewhere? Maybe a ControllerDocument? (JMarc)
-/** Save the buffer's parameters as user default.
-    This function saves a file \c user_lyxdir/templates/defaults.lyx
-    which parameters are those of the current buffer. This file
-    is used as a default template when creating a new
-    file. Returns \c true on success.
-*/
-bool saveParamsAsDefault(BufferParams const &params)
+ButtonPolicy::SMInput FormDocument::input(FL_OBJECT * ob, long)
 {
-       if (!Alert::askQuestion(_("Do you want to save the current settings"),
-                               _("for the document layout as default?"),
-                               _("(they will be valid for any new document)")))
-               return false;
-
-       string const fname = AddName(AddPath(user_lyxdir, "templates/"),
-                                    "defaults.lyx");
-       Buffer defaults = Buffer(fname);
-       defaults.params = params;
-
-       // add an empty paragraph. Is this enough?
-       defaults.paragraph = new Paragraph;
-
-       return defaults.writeFile(defaults.fileName(), false);
-}
-
-} //namespace
-
-
-bool FormDocument::input(FL_OBJECT * ob, long data)
-{
-       State cb = static_cast<State>(data);
-
-       switch (cb) {
-       case CHECKCHOICECLASS:
-               CheckChoiceClass(ob, 0);
-               break;
-       case CHOICEBULLETSIZE:
+       if (ob == bullets_->choice_bullet_size) {
                ChoiceBulletSize(ob, 0);
-               break;
-       case INPUTBULLETLATEX:
+
+       } else if (ob == bullets_->input_bullet_latex) {
                InputBulletLaTeX(ob, 0);
-               break;
-       case BULLETDEPTH1:
-       case BULLETDEPTH2:
-       case BULLETDEPTH3:
-       case BULLETDEPTH4:
-               BulletDepth(ob, cb);
-               break;
-       case BULLETPANEL1:
-       case BULLETPANEL2:
-       case BULLETPANEL3:
-       case BULLETPANEL4:
-       case BULLETPANEL5:
-       case BULLETPANEL6:
-               BulletPanel(ob, cb);
-               break;
-       case BULLETBMTABLE:
-               BulletBMTable(ob, 0);
-               break;
-       default:
-               break;
-       }
 
-       bool const length_input = fl_get_choice(class_->choice_doc_skip) == 4;
-       if (ob == class_->choice_doc_skip) {
-               setEnabled(class_->input_doc_skip, length_input);
-               setEnabled(class_->choice_default_skip_units, length_input);
-       }
+       } else if (ob == bullets_->radio_bullet_depth_1 ||
+                  ob == bullets_->radio_bullet_depth_2 ||
+                  ob == bullets_->radio_bullet_depth_3 ||
+                  ob == bullets_->radio_bullet_depth_4) {
+               BulletDepth(ob);
+
+       } else if (ob == bullets_->radio_bullet_panel_standard ||
+                  ob == bullets_->radio_bullet_panel_maths ||
+                  ob == bullets_->radio_bullet_panel_ding1 ||
+                  ob == bullets_->radio_bullet_panel_ding2 ||
+                  ob == bullets_->radio_bullet_panel_ding3 ||
+                  ob == bullets_->radio_bullet_panel_ding4) {
+               BulletPanel(ob);
+
+       } else if (ob == bullets_->bmtable_bullet_panel) {
+               BulletBMTable(ob, 0);
 
-       if (ob == class_->choice_doc_spacing)
+       } else if (ob == class_->choice_doc_spacing) {
                setEnabled(class_->input_doc_spacing,
                           fl_get_choice(class_->choice_doc_spacing) == 4);
 
-       bool const skip_used = fl_get_button(class_->radio_doc_skip);
-       if (ob == class_->radio_doc_skip ||
-           ob == class_->radio_doc_indent) {
+       } else if (ob == class_->combox_doc_class) {
+               CheckChoiceClass();
+       } else if (ob == class_->radio_doc_skip ||
+                  ob == class_->radio_doc_indent ||
+                  ob == class_->choice_doc_skip) {
+               bool const skip_used = fl_get_button(class_->radio_doc_skip);
                setEnabled(class_->choice_doc_skip, skip_used);
+
+               bool const length_input =
+                       fl_get_choice(class_->choice_doc_skip) == 4;
                setEnabled(class_->input_doc_skip,
                           skip_used && length_input);
-               setEnabled(class_->choice_default_skip_units,
+               setEnabled(class_->choice_doc_skip_units,
                           skip_used && length_input);
+
                // Default unit choice is cm if metric, inches if US paper.
                // If papersize is default, check the lyxrc-settings
                int const paperchoice = fl_get_choice(paper_->choice_papersize);
-               bool const metric = (paperchoice == 1 && lyxrc.default_papersize > 3)
+               bool const metric = (paperchoice == 1 && lyxrc.default_papersize > BufferParams::PAPER_EXECUTIVEPAPER)
                        || paperchoice == 2 || paperchoice > 5;
-               int const default_unit = metric ? 8 : 9;
-               if (strip(fl_get_input(class_->input_doc_skip)).empty())
-                       fl_set_choice(class_->choice_default_skip_units,
-                                     default_unit);
-       }
+               string const default_unit = metric ? "cm" : "in";
+               if (getString(class_->input_doc_skip).empty())
+                       fl_set_choice_text(class_->choice_doc_skip_units,
+                                          default_unit.c_str());
 
-       if (ob == options_->check_use_natbib) {
+       } else if (ob == options_->check_use_natbib) {
                setEnabled(options_->choice_citation_format,
                           fl_get_button(options_->check_use_natbib));
-       }
 
-       if (ob == dialog_->button_save_defaults) {
-               BufferParams params;
-               class_apply(params);
-               paper_apply(params);
-               language_apply(params);
-               options_apply(params);
-               bullets_apply(params);
-               saveParamsAsDefault(params);
-       }
+       } else if (ob == dialog_->button_save_defaults) {
+               apply();
+               controller().saveAsDefault();
 
-       if (ob == dialog_->button_reset_defaults) {
-               BufferParams params = lv_->buffer()->params;
-               params.textclass = combo_doc_class->get() - 1;
+       } else if (ob == dialog_->button_reset_defaults) {
+               BufferParams & params = controller().params();
+               params.textclass = fl_get_combox(class_->combox_doc_class) - 1;
                params.useClassDefaults();
                UpdateLayoutDocument(params);
-       }
 
-       if (ob == paper_->radio_landscape)
+       } else if (ob == paper_->radio_landscape) {
                fl_set_choice(paper_->choice_paperpackage,
                              BufferParams::PACKAGE_NONE + 1);
 
-       if (ob == paper_->choice_papersize) {
+       } else if (ob == paper_->choice_papersize) {
                int const paperchoice = fl_get_choice(paper_->choice_papersize);
                bool const defsize = paperchoice == 1;
                bool const custom = paperchoice == 2;
@@ -552,75 +488,59 @@ bool FormDocument::input(FL_OBJECT * ob, long data)
 
                // Default unit choice is cm if metric, inches if US paper.
                // If papersize is default, use the lyxrc-settings
-               bool const metric = (defsize && lyxrc.default_papersize > 3)
+               bool const metric = (defsize && lyxrc.default_papersize > BufferParams::PAPER_EXECUTIVEPAPER)
                        || paperchoice == 2 || paperchoice > 5;
-               int const default_unit = metric ? 8 : 9;
-               if (strip(fl_get_input(paper_->input_custom_width)).empty())
-                       fl_set_choice(paper_->choice_custom_width_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_custom_height)).empty())
-                       fl_set_choice(paper_->choice_custom_height_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_top_margin)).empty())
-                       fl_set_choice(paper_->choice_top_margin_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_bottom_margin)).empty())
-                       fl_set_choice(paper_->choice_bottom_margin_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_inner_margin)).empty())
-                       fl_set_choice(paper_->choice_inner_margin_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_outer_margin)).empty())
-                       fl_set_choice(paper_->choice_outer_margin_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_head_height)).empty())
-                       fl_set_choice(paper_->choice_head_height_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_head_sep)).empty())
-                       fl_set_choice(paper_->choice_head_sep_units,
-                                     default_unit);
-               if (strip(fl_get_input(paper_->input_foot_skip)).empty())
-                       fl_set_choice(paper_->choice_foot_skip_units,
-                                     default_unit);
-       }
-
-       if (ob == paper_->choice_papersize ||
-           ob == paper_->radio_portrait ||
-           ob == paper_->radio_landscape) {
-
-               setEnabled(paper_->choice_paperpackage,
-                          //either default papersize (preferences)
-                          //or document papersize has to be A4
-                          (fl_get_choice(paper_->choice_papersize) == 7
-                           || fl_get_choice(paper_->choice_papersize) == 1
-                           && lyxrc.default_papersize == 5)
-                          && fl_get_button(paper_->radio_portrait));
-       }
-
-       if (ob == paper_->choice_paperpackage) {
-               if (fl_get_choice(paper_->choice_paperpackage) != 1) {
-                       fl_set_button(paper_->check_use_geometry, 0);
-
-                       setEnabled(paper_->input_top_margin,    false);
-                       setEnabled(paper_->input_bottom_margin, false);
-                       setEnabled(paper_->input_inner_margin,  false);
-                       setEnabled(paper_->input_outer_margin,  false);
-                       setEnabled(paper_->input_head_height,   false);
-                       setEnabled(paper_->input_head_sep,      false);
-                       setEnabled(paper_->input_foot_skip,     false);
-                       setEnabled(paper_->choice_top_margin_units,    false);
-                       setEnabled(paper_->choice_bottom_margin_units, false);
-                       setEnabled(paper_->choice_inner_margin_units,  false);
-                       setEnabled(paper_->choice_outer_margin_units,  false);
-                       setEnabled(paper_->choice_head_height_units,   false);
-                       setEnabled(paper_->choice_head_sep_units,      false);
-                       setEnabled(paper_->choice_foot_skip_units,     false);
-               }
-       }
-
-       if (ob == paper_->check_use_geometry) {
-               //don't allow switching geometry off in custom papersize mode
-               //nor in A3, B3, and B4
+               string const default_unit = metric ? "cm" : "in";
+               if (getString(paper_->input_custom_width).empty())
+                       fl_set_choice_text(paper_->choice_custom_width_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_custom_height).empty())
+                       fl_set_choice_text(paper_->choice_custom_height_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_top_margin).empty())
+                       fl_set_choice_text(paper_->choice_top_margin_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_bottom_margin).empty())
+                       fl_set_choice_text(paper_->choice_bottom_margin_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_inner_margin).empty())
+                       fl_set_choice_text(paper_->choice_inner_margin_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_outer_margin).empty())
+                       fl_set_choice_text(paper_->choice_outer_margin_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_head_height).empty())
+                       fl_set_choice_text(paper_->choice_head_height_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_head_sep).empty())
+                       fl_set_choice_text(paper_->choice_head_sep_units,
+                                          default_unit.c_str());
+               if (getString(paper_->input_foot_skip).empty())
+                       fl_set_choice_text(paper_->choice_foot_skip_units,
+                                          default_unit.c_str());
+
+       } else if (ob == paper_->choice_paperpackage &&
+                  fl_get_choice(paper_->choice_paperpackage) != 1) {
+
+               fl_set_button(paper_->check_use_geometry, 0);
+               setEnabled(paper_->input_top_margin,    false);
+               setEnabled(paper_->input_bottom_margin, false);
+               setEnabled(paper_->input_inner_margin,  false);
+               setEnabled(paper_->input_outer_margin,  false);
+               setEnabled(paper_->input_head_height,   false);
+               setEnabled(paper_->input_head_sep,      false);
+               setEnabled(paper_->input_foot_skip,     false);
+               setEnabled(paper_->choice_top_margin_units,    false);
+               setEnabled(paper_->choice_bottom_margin_units, false);
+               setEnabled(paper_->choice_inner_margin_units,  false);
+               setEnabled(paper_->choice_outer_margin_units,  false);
+               setEnabled(paper_->choice_head_height_units,   false);
+               setEnabled(paper_->choice_head_sep_units,      false);
+               setEnabled(paper_->choice_foot_skip_units,     false);
+
+       } else if (ob == paper_->check_use_geometry) {
+               // don't allow switching geometry off in custom papersize
+               // mode nor in A3, B3, and B4
                int const choice = fl_get_choice(paper_->choice_papersize);
                if (choice == 2 || choice == 6 || choice == 9 || choice == 10)
                        fl_set_button(paper_->check_use_geometry, 1);
@@ -645,27 +565,21 @@ bool FormDocument::input(FL_OBJECT * ob, long data)
                setEnabled(paper_->choice_foot_skip_units,     use_geom);
        }
 
-       switch (data) {
-       case INPUT:
-       case CHECKCHOICECLASS:
-       case CHOICEBULLETSIZE:
-       case INPUTBULLETLATEX:
-       case BULLETBMTABLE:
-               return CheckDocumentInput(ob, 0);
-       default:
-               break;
+       if (ob == paper_->choice_papersize || ob == paper_->radio_portrait
+           || ob == paper_->radio_landscape) {
+               // either default papersize (preferences) or document
+               // papersize has to be A4
+               bool const enable = ( fl_get_choice(paper_->choice_papersize) == 1
+                                     && lyxrc.default_papersize == BufferParams::PAPER_A4PAPER )
+                       || fl_get_choice(paper_->choice_papersize) == 7;
+               if (!enable)
+                       fl_set_choice(paper_->choice_paperpackage,
+                                     BufferParams::PACKAGE_NONE + 1);
+               setEnabled(paper_->choice_paperpackage,
+                          enable && fl_get_button(paper_->radio_portrait));
        }
 
-       return true;
-}
-
-
-void FormDocument::ComboInputCB(int, void * v, Combox * combox)
-{
-       FormDocument * pre = static_cast<FormDocument*>(v);
-       if (combox == pre->combo_doc_class.get())
-               pre->CheckChoiceClass(0, 0);
-       pre->bc().valid(pre->CheckDocumentInput(0,0));
+       return ButtonPolicy::SMI_VALID;
 }
 
 
@@ -674,16 +588,16 @@ bool FormDocument::class_apply(BufferParams &params)
        bool redo = false;
 
        // If default skip is a "Length" but there's no text in the
-       // input field, reset the kind to "Medskip", which is the default.
-       if ((fl_get_choice (class_->choice_doc_skip) == 4) &&
-           !*(fl_get_input (class_->input_doc_skip))) {
-               fl_set_choice (class_->choice_doc_skip, 2);
+       // input field, reset the kind to "MedSkip", which is the default.
+       if (fl_get_choice(class_->choice_doc_skip) == 4 &&
+           getString(class_->input_doc_skip).empty()) {
+               fl_set_choice(class_->choice_doc_skip, 2);
        }
-       params.fonts = fl_get_choice_text(class_->choice_doc_fonts);
-       params.fontsize = fl_get_choice_text(class_->choice_doc_fontsize);
-       params.pagestyle = fl_get_choice_text(class_->choice_doc_pagestyle);
+       params.fonts = getString(class_->choice_doc_fonts);
+       params.fontsize = getString(class_->choice_doc_fontsize);
+       params.pagestyle = getString(class_->choice_doc_pagestyle);
 
-       params.textclass = combo_doc_class->get() - 1;
+       params.textclass = fl_get_combox(class_->combox_doc_class) - 1;
 
        BufferParams::PARSEP tmpsep = params.paragraph_separation;
        if (fl_get_button(class_->radio_doc_indent))
@@ -694,7 +608,7 @@ bool FormDocument::class_apply(BufferParams &params)
                redo = true;
 
        VSpace tmpdefskip = params.getDefSkip();
-       switch (fl_get_choice (class_->choice_doc_skip)) {
+       switch (fl_get_choice(class_->choice_doc_skip)) {
        case 1:
                params.setDefSkip(VSpace(VSpace::SMALLSKIP));
                break;
@@ -708,7 +622,7 @@ bool FormDocument::class_apply(BufferParams &params)
        {
                string const length =
                        getLengthFromWidgets(class_->input_doc_skip,
-                                            class_->choice_default_skip_units);
+                                            class_->choice_doc_skip_units);
 
                params.setDefSkip(VSpace(LyXGlueLength(length)));
                break;
@@ -747,56 +661,13 @@ bool FormDocument::class_apply(BufferParams &params)
        case 4:
                lyxerr[Debug::INFO] << "Spacing: OTHER\n";
                params.spacing.set(Spacing::Other,
-                                  fl_get_input(class_->input_doc_spacing));
+                                  getString(class_->input_doc_spacing));
                break;
        }
        if (tmpSpacing != params.spacing)
                redo = true;
 
-       params.options = fl_get_input(class_->input_doc_extra);
-
-       return redo;
-}
-
-
-bool FormDocument::class_apply()
-{
-       BufferParams &params = lv_->buffer()->params;
-
-       unsigned int const old_class = params.textclass;
-
-       bool redo = class_apply(params);
-
-       if (params.textclass != old_class) {
-               // try to load new_class
-               if (textclasslist[params.textclass].load()) {
-                       // successfully loaded
-                       redo = true;
-                       setMinibuffer(lv_, _("Converting document to new document class..."));
-                       int ret = CutAndPaste::SwitchLayoutsBetweenClasses(
-                               old_class, params.textclass,
-                               lv_->buffer()->paragraph,
-                               lv_->buffer()->params);
-                       if (ret) {
-                               string s;
-                               if (ret == 1) {
-                                       s = _("One paragraph couldn't be converted");
-                               } else {
-                                       s += tostr(ret);
-                                       s += _(" paragraphs couldn't be converted");
-                               }
-                               Alert::alert(_("Conversion Errors!"),s,
-                                            _("into chosen document class"));
-                       }
-
-               } else {
-                       // problem changing class -- warn user and retain old style
-                       Alert::alert(_("Conversion Errors!"),
-                                    _("Errors loading new document class."),
-                                    _("Reverting to original document class."));
-                       combo_doc_class->select(int(old_class) + 1);
-               }
-       }
+       params.options = getString(class_->input_doc_extra);
 
        return redo;
 }
@@ -804,10 +675,10 @@ bool FormDocument::class_apply()
 
 void FormDocument::paper_apply(BufferParams & params)
 {
-       params.papersize2 = char(fl_get_choice(paper_->choice_papersize)-1);
+       params.papersize2 = char(fl_get_choice(paper_->choice_papersize) - 1);
 
        params.paperpackage =
-               char(fl_get_choice(paper_->choice_paperpackage)-1);
+               char(fl_get_choice(paper_->choice_paperpackage) - 1);
 
        // set params.papersize from params.papersize2 and params.paperpackage
        params.setPaperStuff();
@@ -857,12 +728,6 @@ void FormDocument::paper_apply(BufferParams & params)
 }
 
 
-void FormDocument::paper_apply()
-{
-       paper_apply(lv_->buffer()->params);
-}
-
-
 bool FormDocument::language_apply(BufferParams & params)
 {
        InsetQuotes::quote_language lga = InsetQuotes::EnglishQ;
@@ -894,44 +759,28 @@ bool FormDocument::language_apply(BufferParams & params)
        else
                params.quotes_times = InsetQuotes::DoubleQ;
 
-       int const pos = combo_language->get();
-       Language const * old_language = params.language;
+       int const pos = fl_get_combox(language_->combox_language);
        Language const * new_language = languages.getLanguage(lang_[pos-1]);
        if (!new_language)
                new_language = default_language;
 
-       if (old_language != new_language
-           && old_language->RightToLeft() == new_language->RightToLeft()
-           && !lv_->buffer()->isMultiLingual())
-               lv_->buffer()->changeLanguage(old_language, new_language);
-
-       if (old_language != new_language) {
-               redo = true;
-       }
-
        params.language = new_language;
-       params.inputenc = fl_get_choice_text(language_->choice_inputenc);
+       params.inputenc = getString(language_->choice_inputenc);
 
        return redo;
 }
 
 
-bool FormDocument::language_apply()
-{
-       return language_apply(lv_->buffer()->params);
-}
-
-
 bool FormDocument::options_apply(BufferParams & params)
 {
        bool redo = false;
 
-       params.graphicsDriver =
-               fl_get_choice_text(options_->choice_postscript_driver);
-       params.use_amsmath = fl_get_button(options_->check_use_amsmath);
+       params.graphicsDriver = getString(options_->choice_postscript_driver);
+       params.use_amsmath = static_cast<BufferParams::AMS>(
+               fl_get_choice(options_->choice_ams_math) - 1);
        params.use_natbib  = fl_get_button(options_->check_use_natbib);
        params.use_numerical_citations  =
-               fl_get_choice(options_->choice_citation_format)-1;
+               fl_get_choice(options_->choice_citation_format) - 1;
 
        int tmpchar = int(fl_get_counter_value(options_->counter_secnumdepth));
        if (params.secnumdepth != tmpchar)
@@ -941,36 +790,23 @@ bool FormDocument::options_apply(BufferParams & params)
        params.tocdepth = int(fl_get_counter_value(options_->counter_tocdepth));
 
        params.float_placement =
-               fl_get_input(options_->input_float_placement);
+               getString(options_->input_float_placement);
 
        return redo;
 }
 
 
-bool FormDocument::options_apply()
-{
-       return options_apply(lv_->buffer()->params);
-}
-
-
 void FormDocument::bullets_apply(BufferParams & params)
 {
        /* update the bullet settings */
-       BufferParams & buf_params = lv_->buffer()->params;
+       BufferParams & buf_params = controller().params();
 
-       // a little bit of loop unrolling
-       params.user_defined_bullets[0] = buf_params.temp_bullets[0];
-       params.user_defined_bullets[1] = buf_params.temp_bullets[1];
-       params.user_defined_bullets[2] = buf_params.temp_bullets[2];
-       params.user_defined_bullets[3] = buf_params.temp_bullets[3];
+       for (int i = 0; i < 4; ++i) {
+               params.user_defined_bullets[i] = buf_params.temp_bullets[i];
+       }
 }
 
 
-void FormDocument::bullets_apply()
-{
-       bullets_apply(lv_->buffer()->params);
-}
-
 void FormDocument::UpdateClassParams(BufferParams const & params)
 {
        // These are the params that have to be updated on any class change
@@ -978,19 +814,19 @@ void FormDocument::UpdateClassParams(BufferParams const & params)
 
        LyXTextClass const & tclass = textclasslist[params.textclass];
 
-       combo_doc_class->select(tclass.description());
+       fl_set_combox(class_->combox_doc_class, params.textclass + 1);
        fl_clear_choice(class_->choice_doc_fontsize);
        fl_addto_choice(class_->choice_doc_fontsize, "default");
        fl_addto_choice(class_->choice_doc_fontsize,
                        tclass.opt_fontsize().c_str());
        fl_set_choice_text(class_->choice_doc_fontsize,
-                       params.fontsize.c_str());
+                          params.fontsize.c_str());
        fl_clear_choice(class_->choice_doc_pagestyle);
        fl_addto_choice(class_->choice_doc_pagestyle, "default");
        fl_addto_choice(class_->choice_doc_pagestyle,
                        tclass.opt_pagestyle().c_str());
        fl_set_choice_text(class_->choice_doc_pagestyle,
-                       params.pagestyle.c_str());
+                          params.pagestyle.c_str());
 
 }
 
@@ -1001,92 +837,84 @@ void FormDocument::class_update(BufferParams const & params)
 
        UpdateClassParams(params);
 
-       fl_set_choice_text(class_->choice_doc_fonts,
-               params.fonts.c_str());
-       fl_set_button(class_->radio_doc_indent, 0);
-       fl_set_button(class_->radio_doc_skip, 0);
-       if (params.paragraph_separation == BufferParams::PARSEP_INDENT)
-               fl_set_button(class_->radio_doc_indent, 1);
-       else
-               fl_set_button(class_->radio_doc_skip, 1);
+       fl_set_choice_text(class_->choice_doc_fonts, params.fonts.c_str());
 
-       bool const input_length = fl_get_choice(class_->choice_doc_skip) == 4;
-       setEnabled(class_->choice_default_skip_units, input_length);
-       setEnabled(class_->input_doc_skip, input_length);
+       bool const indent = params.paragraph_separation == BufferParams::PARSEP_INDENT;
+       fl_set_button(class_->radio_doc_indent, indent);
+       fl_set_button(class_->radio_doc_skip, !indent);
 
-       switch (params.getDefSkip().kind()) {
-       case VSpace::SMALLSKIP:
-               fl_set_choice (class_->choice_doc_skip, 1);
-               break;
-       case VSpace::MEDSKIP:
-               fl_set_choice (class_->choice_doc_skip, 2);
-               break;
-       case VSpace::BIGSKIP:
-               fl_set_choice (class_->choice_doc_skip, 3);
-               break;
-       case VSpace::LENGTH:
-       {
+       int pos;
+       if (indent) {
+               pos = 2; // VSpace::MEDSKIP is default
+       } else {
+               switch (params.getDefSkip().kind()) {
+               case VSpace::LENGTH:
+                       pos = 4;
+                       break;
+               case VSpace::BIGSKIP:
+                       pos = 3;
+                       break;
+               case VSpace::SMALLSKIP:
+                       pos = 1;
+                       break;
+               case VSpace::MEDSKIP:
+               default:
+                       pos = 2;
+                       break;
+               }
+       }
+       fl_set_choice (class_->choice_doc_skip, pos);
+
+       bool const length_input = pos == 4;
+       if (length_input) {
                int const paperchoice = fl_get_choice(paper_->choice_papersize);
-               bool const metric = (paperchoice == 1 && lyxrc.default_papersize > 3)
+               bool const metric = (paperchoice == 1 && lyxrc.default_papersize > BufferParams::PAPER_EXECUTIVEPAPER)
                        || paperchoice == 2 || paperchoice > 5;
                string const default_unit = metric ? "cm" : "in";
                string const length = params.getDefSkip().asLyXCommand();
                updateWidgetsFromLengthString(class_->input_doc_skip,
-                                             class_->choice_default_skip_units,
+                                             class_->choice_doc_skip_units,
                                              length, default_unit);
-               break;
-       }
-       default:
-               fl_set_choice (class_->choice_doc_skip, 2);
-               break;
+
+       } else {
+               fl_set_input(class_->input_doc_skip, "");
        }
-       fl_set_button(class_->radio_doc_sides_one, 0);
-       fl_set_button(class_->radio_doc_sides_two, 0);
-       setEnabled(class_->choice_doc_skip,
-                  fl_get_button(class_->radio_doc_skip));
+       setEnabled(class_->choice_doc_skip, !indent);
+       setEnabled(class_->input_doc_skip, length_input);
+       setEnabled(class_->choice_doc_skip_units, length_input);
 
-       if (params.sides == LyXTextClass::TwoSides)
-               fl_set_button(class_->radio_doc_sides_two, 1);
-       else
-               fl_set_button(class_->radio_doc_sides_one, 1);
-       fl_set_button(class_->radio_doc_columns_one, 0);
-       fl_set_button(class_->radio_doc_columns_two, 0);
-       if (params.columns == 2)
-               fl_set_button(class_->radio_doc_columns_two, 1);
-       else
-               fl_set_button(class_->radio_doc_columns_one, 1);
+       bool const two_sides = params.sides == LyXTextClass::TwoSides;
+       fl_set_button(class_->radio_doc_sides_one, !two_sides);
+       fl_set_button(class_->radio_doc_sides_two, two_sides);
 
-       fl_set_input(class_->input_doc_spacing, "");
-       setEnabled(class_->input_doc_spacing, false);
+       bool const two_columns = params.columns == 2;
+       fl_set_button(class_->radio_doc_columns_one, !two_columns);
+       fl_set_button(class_->radio_doc_columns_two, two_columns);
+
+       fl_set_input(class_->input_doc_extra, params.options.c_str());
 
        switch (params.spacing.getSpace()) {
-       case Spacing::Default: // nothing bad should happen with this
-       case Spacing::Single:
-               // \singlespacing
-               fl_set_choice(class_->choice_doc_spacing, 1);
+       case Spacing::Other:
+               pos = 4;
                break;
-       case Spacing::Onehalf:
-               // \onehalfspacing
-               fl_set_choice(class_->choice_doc_spacing, 2);
+       case Spacing::Double: // \doublespacing
+               pos = 3;
                break;
-       case Spacing::Double:
-               // \doublespacing
-               fl_set_choice(class_->choice_doc_spacing, 3);
+       case Spacing::Onehalf: // \onehalfspacing
+               pos = 2;
                break;
-       case Spacing::Other:
-       {
-               fl_set_choice(class_->choice_doc_spacing, 4);
-               char sval[20];
-               sprintf(sval,"%g",params.spacing.getValue());
-               fl_set_input(class_->input_doc_spacing, sval);
-               setEnabled(class_->input_doc_spacing, true);
+       case Spacing::Single: // \singlespacing
+       case Spacing::Default: // nothing bad should happen with this
+       default:
+               pos = 1;
                break;
        }
-       }
-       if (!params.options.empty())
-               fl_set_input(class_->input_doc_extra, params.options.c_str());
-       else
-               fl_set_input(class_->input_doc_extra, "");
+       fl_set_choice(class_->choice_doc_spacing, pos);
+
+       bool const spacing_input = pos == 4;
+       setEnabled(class_->input_doc_spacing, spacing_input);
+       string const input = spacing_input ? tostr(params.spacing.getValue()) : string();
+       fl_set_input(class_->input_doc_spacing, input.c_str());
 }
 
 
@@ -1096,7 +924,7 @@ void FormDocument::language_update(BufferParams const & params)
                return;
 
        int const pos = int(findPos(lang_, params.language->lang()));
-       combo_language->select(pos+1);
+       fl_set_combox(language_->combox_language, pos+1);
 
        fl_set_choice_text(language_->choice_inputenc, params.inputenc.c_str());
        fl_set_choice(language_->choice_quotes_language, params.quotes_language + 1);
@@ -1116,7 +944,7 @@ void FormDocument::options_update(BufferParams const & params)
 
        fl_set_choice_text(options_->choice_postscript_driver,
                           params.graphicsDriver.c_str());
-       fl_set_button(options_->check_use_amsmath, params.use_amsmath);
+       fl_set_choice(options_->choice_ams_math, params.use_amsmath + 1);
        fl_set_button(options_->check_use_natbib,  params.use_natbib);
        fl_set_choice(options_->choice_citation_format,
                      int(params.use_numerical_citations)+1);
@@ -1157,11 +985,11 @@ void FormDocument::paper_update(BufferParams const & params)
                   //either default papersize (preferences)
                   //or document papersize has to be A4
                   (paperchoice == 7
-                   || paperchoice == 1 && lyxrc.default_papersize == 5)
+                   || paperchoice == 1 && lyxrc.default_papersize == BufferParams::PAPER_A4PAPER)
                   && fl_get_button(paper_->radio_portrait));
 
        // Default unit choice is cm if metric, inches if US paper.
-       bool const metric = (paperchoice == 1 && lyxrc.default_papersize > 3)
+       bool const metric = (paperchoice == 1 && lyxrc.default_papersize > BufferParams::PAPER_EXECUTIVEPAPER)
                || paperchoice == 2 || paperchoice > 5;
        string const default_unit = metric ? "cm" : "in";
        updateWidgetsFromLengthString(paper_->input_custom_width,
@@ -1229,7 +1057,8 @@ void FormDocument::bullets_update(BufferParams const & params)
             (XpmVersion==4 && XpmRevision<7)))
                return;
 
-       bool const isLinuxDoc = lv_->buffer()->isLinuxDoc();
+       bool const isLinuxDoc =
+               controller().docType() == ControlDocument::LINUXDOC;
        setEnabled(fbullet, !isLinuxDoc);
 
        if (isLinuxDoc) return;
@@ -1244,89 +1073,18 @@ void FormDocument::bullets_update(BufferParams const & params)
 
 void FormDocument::checkReadOnly()
 {
-       if (bc().readOnly(lv_->buffer()->isReadonly())) {
-               combo_doc_class->deactivate();
-               combo_language->deactivate();
+       if (bc().readOnly(controller().bufferIsReadonly())) {
                postWarning(_("Document is read-only."
                              " No changes to layout permitted."));
        } else {
-               combo_doc_class->activate();
-               combo_language->activate();
                clearMessage();
        }
 }
 
 
-bool FormDocument::CheckDocumentInput(FL_OBJECT * ob, long)
-{
-       string str;
-       bool ok = true;
-       char const * input;
-
-       // this has to be all out of if/elseif because it has to deactivate
-       // the document buttons and so the whole stuff has to be tested again.
-       // disable OK/Apply if input is not valid
-       str = fl_get_input(class_->input_doc_skip);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_custom_width);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_custom_height);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_outer_margin);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_inner_margin);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_top_margin);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_bottom_margin);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_head_height);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_head_sep);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-       str = fl_get_input(paper_->input_foot_skip);
-       ok = ok && (str.empty() || isValidLength(str) || isStrDbl(str));
-
-       //display warning if input is not valid
-       if (ob == class_->input_doc_skip
-           || ob == paper_->input_custom_width
-           || ob == paper_->input_custom_height
-           || ob == paper_->input_outer_margin
-           || ob == paper_->input_inner_margin
-           || ob == paper_->input_top_margin
-           || ob == paper_->input_bottom_margin
-           || ob == paper_->input_head_height
-           || ob == paper_->input_head_sep
-           || ob == paper_->input_foot_skip) {
-               if (!ok) {
-                       postWarning(_("Invalid Length (valid example: 10mm)"));
-                       return false;
-               } else {
-                       clearMessage();
-                       return true;
-               }
-       }
-
-       // "Synchronize" the choice and the input field, so that it
-       // is impossible to commit senseless data.
-       input = fl_get_input (class_->input_doc_skip);
-       if ((fl_get_choice(class_->choice_doc_skip) == 4) && !*input)
-               ok = false;
-       else if (fl_get_choice(class_->choice_doc_skip) != 4)
-               fl_set_input (class_->input_doc_skip, "");
-
-       input = fl_get_input(class_->input_doc_spacing);
-       if ((fl_get_choice(class_->choice_doc_spacing) == 4) && !*input)
-               ok = false;
-       else if (fl_get_choice(class_->choice_doc_spacing) != 4)
-               fl_set_input (class_->input_doc_spacing, "");
-       return ok;
-}
-
-
 void FormDocument::ChoiceBulletSize(FL_OBJECT * ob, long /*data*/)
 {
-       BufferParams & param = lv_->buffer()->params;
+       BufferParams & param = controller().params();
 
        // convert from 1-6 range to -1-4
        param.temp_bullets[current_bullet_depth].setSize(fl_get_choice(ob) - 2);
@@ -1337,14 +1095,14 @@ void FormDocument::ChoiceBulletSize(FL_OBJECT * ob, long /*data*/)
 
 void FormDocument::InputBulletLaTeX(FL_OBJECT *, long)
 {
-       BufferParams & param = lv_->buffer()->params;
+       BufferParams & param = controller().params();
 
        param.temp_bullets[current_bullet_depth].
-               setText(fl_get_input(bullets_->input_bullet_latex));
+               setText(getString(bullets_->input_bullet_latex));
 }
 
 
-void FormDocument::BulletDepth(FL_OBJECT * ob, State cb)
+void FormDocument::BulletDepth(FL_OBJECT * ob)
 {
        /* Should I do the following:                                 */
        /*  1. change to the panel that the current bullet belongs in */
@@ -1354,16 +1112,16 @@ void FormDocument::BulletDepth(FL_OBJECT * ob, State cb)
        /*                                                            */
        /* I'm inclined to just go with 3 and 4 at the moment and     */
        /* maybe try to support the others later                      */
-       BufferParams & param = lv_->buffer()->params;
+       BufferParams & param = controller().params();
 
        int data = 0;
-       if (cb == BULLETDEPTH1)
+       if (ob == bullets_->radio_bullet_depth_1)
                data = 0;
-       else if (cb == BULLETDEPTH2)
+       else if (ob == bullets_->radio_bullet_depth_2)
                data = 1;
-       else if (cb == BULLETDEPTH3)
+       else if (ob == bullets_->radio_bullet_depth_3)
                data = 2;
-       else if (cb == BULLETDEPTH4)
+       else if (ob == bullets_->radio_bullet_depth_4)
                data = 3;
 
        switch (fl_get_button_numb(ob)) {
@@ -1380,23 +1138,23 @@ void FormDocument::BulletDepth(FL_OBJECT * ob, State cb)
 }
 
 
-void FormDocument::BulletPanel(FL_OBJECT * /*ob*/, State cb)
+void FormDocument::BulletPanel(FL_OBJECT * ob)
 {
        /* Here we have to change the background pixmap to that selected */
        /* by the user. (eg. standard.xpm, psnfss1.xpm etc...)           */
 
        int data = 0;
-       if (cb == BULLETPANEL1)
+       if (ob == bullets_->radio_bullet_panel_standard)
                data = 0;
-       else if (cb == BULLETPANEL2)
+       else if (ob == bullets_->radio_bullet_panel_maths)
                data = 1;
-       else if (cb == BULLETPANEL3)
+       else if (ob == bullets_->radio_bullet_panel_ding2)
                data = 2;
-       else if (cb == BULLETPANEL4)
+       else if (ob == bullets_->radio_bullet_panel_ding3)
                data = 3;
-       else if (cb == BULLETPANEL5)
+       else if (ob == bullets_->radio_bullet_panel_ding4)
                data = 4;
-       else if (cb == BULLETPANEL6)
+       else if (ob == bullets_->radio_bullet_panel_ding1)
                data = 5;
 
        if (data != current_bullet_panel) {
@@ -1406,32 +1164,23 @@ void FormDocument::BulletPanel(FL_OBJECT * /*ob*/, State cb)
                /* free the current pixmap */
                fl_free_bmtable_pixmap(bullets_->bmtable_bullet_panel);
                string new_panel;
-               switch (cb) {
-                       /* display the new one */
-               case BULLETPANEL1 :
+               if (ob == bullets_->radio_bullet_panel_standard) {
                        new_panel = "standard";
-                       break;
-               case BULLETPANEL2 :
+               } else if (ob == bullets_->radio_bullet_panel_maths ) {
                        new_panel = "amssymb";
-                       break;
-               case BULLETPANEL3 :
+               } else if (ob == bullets_->radio_bullet_panel_ding2) {
                        new_panel = "psnfss1";
-                       break;
-               case BULLETPANEL4 :
+               } else if (ob == bullets_->radio_bullet_panel_ding3) {
                        new_panel = "psnfss2";
-                       break;
-               case BULLETPANEL5 :
+               } else if (ob == bullets_->radio_bullet_panel_ding4) {
                        new_panel = "psnfss3";
-                       break;
-               case BULLETPANEL6 :
+               } else if (ob == bullets_->radio_bullet_panel_ding1) {
                        new_panel = "psnfss4";
-                       break;
-               default :
+               } else {
                        /* something very wrong happened */
                        // play it safe for now but should be an exception
                        current_bullet_panel = 0;  // standard panel
                        new_panel = "standard";
-                       break;
                }
                new_panel += ".xpm";
                fl_set_bmtable_pixmap_file(bullets_->bmtable_bullet_panel, 6, 6,
@@ -1448,7 +1197,7 @@ void FormDocument::BulletBMTable(FL_OBJECT * ob, long /*data*/)
        /* to that extracted from the current chosen position of the BMTable  */
        /* Don't forget to free the button's old pixmap first.                */
 
-       BufferParams & param = lv_->buffer()->params;
+       BufferParams & param = controller().params();
        int bmtable_button = fl_get_bmtable(ob);
 
        /* try to keep the button held down till another is pushed */
@@ -1460,38 +1209,29 @@ void FormDocument::BulletBMTable(FL_OBJECT * ob, long /*data*/)
 }
 
 
-void FormDocument::CheckChoiceClass(FL_OBJECT * ob, long)
+void FormDocument::CheckChoiceClass()
 {
-       if (!ob)
-               ob = class_->choice_doc_class;
+       BufferParams & params = controller().params();
 
-       lv_->prohibitInput();
+       lyx::textclass_type const tc =
+               fl_get_combox(class_->combox_doc_class) - 1;
 
-       unsigned int tc = combo_doc_class->get() - 1;
-       if (textclasslist[tc].load()) {
-               // we use a copy of the bufferparams because we do not
-               // want to modify them yet.
-               BufferParams params = lv_->buffer()->params;
+       if (controller().loadTextclass(tc)) {
+               params.textclass = tc;
 
                if (lyxrc.auto_reset_options) {
-                       params.textclass = tc;
                        params.useClassDefaults();
                        UpdateLayoutDocument(params);
                } else {
                        // update the params which are needed in any case
                        // (fontsizes, pagestyle)
-                       params.textclass = tc;
                        UpdateClassParams(params);
                }
 
        } else {
-               // unable to load new style
-               Alert::alert(_("Conversion Errors!"),
-                            _("Unable to switch to new document class."),
-                            _("Reverting to original document class."));
-               combo_doc_class->select(int(lv_->buffer()->params.textclass) + 1);
+               int const revert = int(params.textclass);
+               fl_set_combox(class_->combox_doc_class, revert + 1);
        }
-       lv_->allowInput();
 }