]> 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 660ef36229cb97750f2d407ac13ac87ab822b934..3a6057fb14293e8491d45389ac7abeff1c96cd12 100644 (file)
-/* This file is part of
- * ======================================================
- * 
- *           LyX, The Document Processor
- *      
- *           Copyright 2000-2001 The LyX Team.
+/**
+ * \file FormDocument.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- *           @author Jürgen Vigna
+ * \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 "lyx_gui_misc.h"
 
+#include "ControlDocument.h"
 #include "FormDocument.h"
-#include "form_document.h"
-#include "Dialogs.h"
-#include "layout.h"
-#include "combox.h"
-#include "tex-strings.h"
-#include "bufferparams.h"
-#include "insets/insetquotes.h"
-#include "vspace.h"
+#include "forms/form_document.h"
+#include "xformsBC.h"
+#include "ButtonController.h"
+
 #include "bmtable.h"
-#include "support/filetools.h"
-#include "support/lstrings.h"
-#include "language.h"
-#include "LyXView.h"
-#include "lyxfunc.h"
-#include "lyxrc.h"
-#include "BufferView.h"
-#include "buffer.h"
-#include "Liason.h"
+#include "checkedwidgets.h"
+#include "input_validators.h" // fl_unsigned_float_filter
+#include "xforms_helpers.h"
+
 #include "CutAndPaste.h"
-#include "bufferview_funcs.h"
-#include "xforms_helpers.h" 
 #include "debug.h"
+#include "language.h"
+#include "lyxrc.h"
+#include "lyxtextclasslist.h"
+#include "tex-strings.h"
+
+#include "controllers/frnt_lang.h"
+#include "controllers/helper_funcs.h"
+
+#include "support/lstrings.h" // contains_functor, getStringFromVector
+#include "support/filetools.h" // LibFileSearch
 
-using Liason::setMinibuffer;
-using SigC::slot;
+#include XPM_H_LOCATION
+#include FORMS_H_LOCATION
+#include "combox.h"
+
+#include <boost/bind.hpp>
+
+#include <functional>
+
+using std::bind2nd;
+using std::vector;
+
+
+typedef FormCB<ControlDocument, FormDB<FD_document> > base_class;
 
-FormDocument::FormDocument(LyXView * lv, Dialogs * d)
-       : FormBaseBD(lv, d, _("Document Layout")),
+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()
 {
-       if( form() && form()->visible )
-               fl_redraw_form( form() );
+       if (form() && form()->visible)
+               fl_redraw_form(form());
        else
                return;
 
-       FL_FORM * outer_form = fl_get_active_folder(dialog_->tabbed_folder);
+       FL_FORM * outer_form = fl_get_active_folder(dialog_->tabfolder);
        if (outer_form && outer_form->visible)
-               fl_redraw_form( outer_form );
-}
-
-
-FL_FORM * FormDocument::form() const
-{
-       if (dialog_.get()) return dialog_->form;
-       return 0;
+               fl_redraw_form(outer_form);
 }
 
 
 void FormDocument::build()
 {
-       int n;
-
        // the tabbed folder
-       dialog_.reset(build_tabbed_document());
+       dialog_.reset(build_document(this));
 
-       // Manage the restore, ok, apply, restore and cancel/close buttons
-       bc().setOK(dialog_->button_ok);
-       bc().setApply(dialog_->button_apply);
-       bc().setCancel(dialog_->button_cancel);
-       bc().setRestore(dialog_->button_restore);
-       bc().addReadOnly (dialog_->button_save_defaults);
-       bc().addReadOnly (dialog_->button_reset_defaults);
-
-       // the document paper form
-       paper_.reset(build_doc_paper());
-       fl_addto_choice(paper_->choice_papersize2,
-                       _(" Default | Custom | USletter | USlegal "
-                         "| USexecutive | A3 | A4 | A5 | B3 | B4 | B5 "));
-       fl_addto_choice(paper_->choice_paperpackage,
-                       _(" None "
-                         "| Small Margins "
-                         "| Very small Margins "
-                         "| Very wide Margins "));
+       // Allow the base class to control messages
+       setMessageWidget(dialog_->text_warning);
 
-       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_left_margin, FL_RETURN_CHANGED);
-       fl_set_input_return(paper_->input_right_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);
-
-       bc().addReadOnly (paper_->choice_paperpackage);
-       bc().addReadOnly (paper_->greoup_radio_orientation);
-       bc().addReadOnly (paper_->radio_portrait);
-       bc().addReadOnly (paper_->radio_landscape);
-       bc().addReadOnly (paper_->choice_papersize2);
-       bc().addReadOnly (paper_->push_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_left_margin);
-       bc().addReadOnly (paper_->input_right_margin);
-       bc().addReadOnly (paper_->input_head_height);
-       bc().addReadOnly (paper_->input_head_sep);
-       bc().addReadOnly (paper_->input_foot_skip);
+       // Manage the restore, ok, apply, restore and cancel/close buttons
+       bcview().setOK(dialog_->button_ok);
+       bcview().setApply(dialog_->button_apply);
+       bcview().setCancel(dialog_->button_close);
+       bcview().setRestore(dialog_->button_restore);
 
        // the document class form
-       class_.reset(build_doc_class());
+       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(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;
-       // 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_->tabbed_folder);
-       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());
+
+       // 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 | Other "));
+                       _(" Single | OneHalf | Double | Custom "));
        fl_addto_choice(class_->choice_doc_fontsize, "default|10|11|12");
-       for (n=0; tex_fonts[n][0]; ++n) {
+       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_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);
+                       _(" 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);
+       setPrehandler(paper_->input_bottom_margin);
+       setPrehandler(paper_->input_inner_margin);
+       setPrehandler(paper_->input_outer_margin);
+       setPrehandler(paper_->input_head_height);
+       setPrehandler(paper_->input_head_sep);
+       setPrehandler(paper_->input_foot_skip);
+
+       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 "));
 
-       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);
+       fl_addto_choice(paper_->choice_custom_width_units,  units.c_str());
+       fl_addto_choice(paper_->choice_custom_height_units, units.c_str());
+       fl_addto_choice(paper_->choice_top_margin_units,    units.c_str());
+       fl_addto_choice(paper_->choice_bottom_margin_units, units.c_str());
+       fl_addto_choice(paper_->choice_inner_margin_units,  units.c_str());
+       fl_addto_choice(paper_->choice_outer_margin_units,  units.c_str());
+       fl_addto_choice(paper_->choice_head_height_units,   units.c_str());
+       fl_addto_choice(paper_->choice_head_sep_units,      units.c_str());
+       fl_addto_choice(paper_->choice_foot_skip_units,     units.c_str());
 
        // the document language form
-       language_.reset(build_doc_language());
+       language_.reset(build_document_language(this));
+
+       // 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);
+
        fl_addto_choice(language_->choice_inputenc,
-                       "default|auto|latin1|latin2|latin5|latin9"
-                       "|koi8-r|koi8-u|cp866|cp1251|iso88595");
-
-       // 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_->tabbed_folder);
-       combo_language->shortcut("#L",1);
-       combo_language->setcallback(ComboInputCB, this);
-       fl_end_form();
-
-       for (Languages::const_iterator cit = languages.begin();
-            cit != languages.end(); ++cit) {
-               combo_language->addto(cit->second.lang());
+                       "default|auto|latin1|latin2|latin3|latin4|latin5|latin9"
+                       "|koi8-r|koi8-u|cp866|cp1251|iso88595|pt154");
+
+       vector<frnt::LanguagePair> const langs = frnt::getLanguageData(false);
+       // Store the identifiers for later
+       lang_ = getSecond(langs);
+
+       vector<frnt::LanguagePair>::const_iterator lit  = langs.begin();
+       vector<frnt::LanguagePair>::const_iterator lend = langs.end();
+       for (; lit != lend; ++lit) {
+               fl_addto_combox(language_->combox_language,
+                               lit->first.c_str());
        }
+       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());
+       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_input_return(options_->input_float_placement, FL_RETURN_CHANGED);
-       fl_set_counter_bounds(options_->slider_secnumdepth,-2,5);
-       fl_set_counter_bounds(options_->slider_tocdepth,-1,5);
-       fl_set_counter_step(options_->slider_secnumdepth,1,1);
-       fl_set_counter_step(options_->slider_tocdepth,1,1);
-       fl_set_counter_precision(options_->slider_secnumdepth, 0);
-       fl_set_counter_precision(options_->slider_tocdepth, 0);
-       for (n=0; tex_graphics[n][0]; ++n) {
+
+       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_->slider_secnumdepth);
-       bc_.addReadOnly (options_->slider_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);
+
        fl_set_input_maxchars(bullets_->input_bullet_latex, 80);
 
-       bc().addReadOnly (bullets_->bmtable_bullet_panel);
-       bc().addReadOnly (bullets_->choice_bullet_size);
-       bc().addReadOnly (bullets_->input_bullet_latex);
-       bc().addReadOnly (bullets_->radio_bullet_depth);
-       bc().addReadOnly (bullets_->radio_bullet_panel);
+       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_->tabbed_folder,_("Document"),
+       fl_addto_tabfolder(dialog_->tabfolder,_("Document"),
                           class_->form);
-       fl_addto_tabfolder(dialog_->tabbed_folder,_("Paper"),
+       fl_addto_tabfolder(dialog_->tabfolder,_("Paper"),
                           paper_->form);
-       fl_addto_tabfolder(dialog_->tabbed_folder,_("Language"),
+       fl_addto_tabfolder(dialog_->tabfolder,_("Language"),
                           language_->form);
-       fl_addto_tabfolder(dialog_->tabbed_folder,_("Extra"),
+       fl_addto_tabfolder(dialog_->tabfolder,_("Extra"),
                           options_->form);
-       fbullet = fl_addto_tabfolder(dialog_->tabbed_folder,_("Bullets"),
+       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 "
@@ -272,32 +348,13 @@ void FormDocument::build()
 
 void FormDocument::apply()
 {
-       if (!lv_->view()->available() || !dialog_.get())
-               return;
-
-       bool redo = class_apply();
-       paper_apply();
-       redo = language_apply() || redo;
-       redo = options_apply() || redo;
-       bullets_apply();
+       BufferParams & params = controller().params();
 
-       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);
 }
 
 
@@ -308,14 +365,7 @@ void FormDocument::update()
 
        checkReadOnly();
 
-       BufferParams const & params = lv_->buffer()->params;
-
-//     fl_set_object_label(dialog_->button_save_defaults,
-//                         _("Save as Defaults"));
-//     fl_set_object_label(dialog_->button_reset_defaults,
-//                         _("Use Class Defaults"));
-       setEnabled(dialog_->button_reset_defaults,
-                  !params.hasClassDefaults());
+       BufferParams const & params = controller().params();
 
        class_update(params);
        paper_update(params);
@@ -325,201 +375,230 @@ void FormDocument::update()
 }
 
 
-bool FormDocument::input( FL_OBJECT * ob, long data )
+ButtonPolicy::SMInput FormDocument::input(FL_OBJECT * ob, long)
 {
-       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:
+
+       } 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);
-               break;
-       default:
-               break;
-       }
 
-       if (ob == options_->check_use_natbib) {
+       } else if (ob == class_->choice_doc_spacing) {
+               setEnabled(class_->input_doc_spacing,
+                          fl_get_choice(class_->choice_doc_spacing) == 4);
+
+       } 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_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 > BufferParams::PAPER_EXECUTIVEPAPER)
+                       || paperchoice == 2 || paperchoice > 5;
+               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());
+
+       } 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) {
-               lv_->getLyXFunc()->dispatch(LFUN_LAYOUT_SAVE_DEFAULT);
-       }
+       } 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);
+                             BufferParams::PACKAGE_NONE + 1);
+
+       } 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;
+               bool const a3size = paperchoice == 6;
+               bool const b3size = paperchoice == 9;
+               bool const b4size = paperchoice == 10;
 
-       if (ob == paper_->choice_papersize2) {
-               if (fl_get_choice(paper_->choice_papersize2) == 2)
+               if (custom)
                        fl_set_button(paper_->radio_portrait, 1);
-               setEnabled(paper_->input_custom_width,
-                       fl_get_choice(paper_->choice_papersize2) == 2);
-               setEnabled(paper_->input_custom_height,
-                       fl_get_choice(paper_->choice_papersize2) == 2);
-               setEnabled(paper_->radio_portrait,
-                       fl_get_choice(paper_->choice_papersize2) != 2);
-               setEnabled(paper_->radio_landscape,
-                       fl_get_choice(paper_->choice_papersize2) != 2);
-       }
 
-       if (ob == paper_->choice_papersize2
-               || paper_->radio_portrait
-               || paper_->radio_landscape)
-               setEnabled(paper_->choice_paperpackage,
-                          //either default papersize (preferences)
-                          //or document papersize has to be A4
-                          (fl_get_choice(paper_->choice_papersize2) == 7
-                           || fl_get_choice(paper_->choice_papersize2) == 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_->push_use_geometry, 0);
-                       setEnabled(paper_->input_top_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_bottom_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_left_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_right_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_head_height,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_head_sep,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_foot_skip,
-                                  fl_get_button(paper_->push_use_geometry));
-               }
-       }
+               bool const use_geom = (custom || a3size || b3size || b4size);
+
+               fl_set_button(paper_->check_use_geometry, int(use_geom));
+
+               setEnabled(paper_->input_top_margin,    use_geom);
+               setEnabled(paper_->input_bottom_margin, use_geom);
+               setEnabled(paper_->input_inner_margin,  use_geom);
+               setEnabled(paper_->input_outer_margin,  use_geom);
+               setEnabled(paper_->input_head_height,   use_geom);
+               setEnabled(paper_->input_head_sep,      use_geom);
+               setEnabled(paper_->input_foot_skip,     use_geom);
+               setEnabled(paper_->choice_top_margin_units,    use_geom);
+               setEnabled(paper_->choice_bottom_margin_units, use_geom);
+               setEnabled(paper_->choice_inner_margin_units,  use_geom);
+               setEnabled(paper_->choice_outer_margin_units,  use_geom);
+               setEnabled(paper_->choice_head_height_units,   use_geom);
+               setEnabled(paper_->choice_head_sep_units,      use_geom);
+               setEnabled(paper_->choice_foot_skip_units,     use_geom);
+               setEnabled(paper_->choice_custom_width_units,  use_geom);
+               setEnabled(paper_->choice_custom_height_units, use_geom);
+
+               setEnabled(paper_->input_custom_width,  custom);
+               setEnabled(paper_->input_custom_height, custom);
+               setEnabled(paper_->choice_custom_width_units,  custom);
+               setEnabled(paper_->choice_custom_height_units, custom);
+               setEnabled(paper_->radio_portrait,  !custom);
+               setEnabled(paper_->radio_landscape, !custom);
+
+               // 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 > BufferParams::PAPER_EXECUTIVEPAPER)
+                       || paperchoice == 2 || paperchoice > 5;
+               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);
 
-       if (ob == paper_->push_use_geometry) {
-               //don't allow switching geometry off in custom papersize mode
-               if (fl_get_choice(paper_->choice_papersize2) == 2)
-                       fl_set_button(paper_->push_use_geometry, 1);
                fl_set_choice(paper_->choice_paperpackage,
                              BufferParams::PACKAGE_NONE + 1);
-               setEnabled(paper_->input_top_margin,
-                          fl_get_button(paper_->push_use_geometry));
-               setEnabled(paper_->input_bottom_margin,
-                          fl_get_button(paper_->push_use_geometry));
-               setEnabled(paper_->input_left_margin,
-                          fl_get_button(paper_->push_use_geometry));
-               setEnabled(paper_->input_right_margin,
-                          fl_get_button(paper_->push_use_geometry));
-               setEnabled(paper_->input_head_height,
-                          fl_get_button(paper_->push_use_geometry));
-               setEnabled(paper_->input_head_sep,
-                          fl_get_button(paper_->push_use_geometry));
-               setEnabled(paper_->input_foot_skip,
-                          fl_get_button(paper_->push_use_geometry));
-       }
 
-       setEnabled(dialog_->button_reset_defaults,
-                  lv_->buffer()->params.hasClassDefaults());
-
-       switch (data) {
-       case INPUT:
-       case CHECKCHOICECLASS:
-       case CHOICEBULLETSIZE:
-       case INPUTBULLETLATEX:
-       case BULLETBMTABLE:
-               return CheckDocumentInput(ob, 0);
-       default:
-               break;
+               bool const use_geom = fl_get_button(paper_->check_use_geometry);
+               setEnabled(paper_->input_top_margin,    use_geom);
+               setEnabled(paper_->input_bottom_margin, use_geom);
+               setEnabled(paper_->input_inner_margin,  use_geom);
+               setEnabled(paper_->input_outer_margin,  use_geom);
+               setEnabled(paper_->input_head_height,   use_geom);
+               setEnabled(paper_->input_head_sep,      use_geom);
+               setEnabled(paper_->input_foot_skip,     use_geom);
+               setEnabled(paper_->choice_top_margin_units,    use_geom);
+               setEnabled(paper_->choice_bottom_margin_units, use_geom);
+               setEnabled(paper_->choice_inner_margin_units,  use_geom);
+               setEnabled(paper_->choice_outer_margin_units,  use_geom);
+               setEnabled(paper_->choice_head_height_units,   use_geom);
+               setEnabled(paper_->choice_head_sep_units,      use_geom);
+               setEnabled(paper_->choice_foot_skip_units,     use_geom);
        }
 
-       return true;
-}
-
+       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));
+       }
 
-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;
 }
 
 
-bool FormDocument::class_apply()
+bool FormDocument::class_apply(BufferParams &params)
 {
        bool redo = false;
-       BufferParams &params = lv_->buffer()->params;
 
        // 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);
-       }
-       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);
-
-       unsigned int const new_class = combo_doc_class->get() - 1;
-       
-       if (params.textclass != new_class) {
-               // try to load new_class
-               if (textclasslist.Load(new_class)) {
-                       // successfully loaded
-                       redo = true;
-                       setMinibuffer(lv_, _("Converting document to new document class..."));
-                       int ret = CutAndPaste::SwitchLayoutsBetweenClasses(
-                               params.textclass, new_class,
-                               lv_->buffer()->paragraph);
-                       if (ret) {
-                               string s;
-                               if (ret==1) {
-                                       s = _("One paragraph couldn't be converted");
-                               } else {
-                                       s += tostr(ret);
-                                       s += _(" paragraphs couldn't be converted");
-                               }
-                               WriteAlert(_("Conversion Errors!"),s,
-                                          _("into chosen document class"));
-                       }
-                       
-                       params.textclass = new_class;
-               } else {
-                       // problem changing class -- warn user and retain old style
-                       WriteAlert(_("Conversion Errors!"),
-                                  _("Errors loading new document class."),
-                                  _("Reverting to original document class."));
-                       combo_doc_class->select(int(params.textclass) + 1);
-               }
+       // 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 = getString(class_->choice_doc_fonts);
+       params.fontsize = getString(class_->choice_doc_fontsize);
+       params.pagestyle = getString(class_->choice_doc_pagestyle);
+
+       params.textclass = fl_get_combox(class_->combox_doc_class) - 1;
+
        BufferParams::PARSEP tmpsep = params.paragraph_separation;
        if (fl_get_button(class_->radio_doc_indent))
                params.paragraph_separation = BufferParams::PARSEP_INDENT;
@@ -527,30 +606,35 @@ bool FormDocument::class_apply()
                params.paragraph_separation = BufferParams::PARSEP_SKIP;
        if (tmpsep != params.paragraph_separation)
                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;
        case 2:
                params.setDefSkip(VSpace(VSpace::MEDSKIP));
                break;
-       case 3:
-           params.setDefSkip(VSpace(VSpace::BIGSKIP));
-           break;
+       case 3:
+               params.setDefSkip(VSpace(VSpace::BIGSKIP));
+               break;
        case 4:
-               params.setDefSkip
-                       (VSpace(LyXGlueLength(fl_get_input(class_->input_doc_skip))));
+       {
+               string const length =
+                       getLengthFromWidgets(class_->input_doc_skip,
+                                            class_->choice_doc_skip_units);
+
+               params.setDefSkip(VSpace(LyXGlueLength(length)));
                break;
-               // DocumentDefskipCB assures that this never happens
+       }
        default:
+               // DocumentDefskipCB assures that this never happens
                params.setDefSkip(VSpace(VSpace::MEDSKIP));
                break;
        }
        if (!(tmpdefskip == params.getDefSkip()))
                redo = true;
-       
+
        if (fl_get_button(class_->radio_doc_columns_two))
                params.columns = 2;
        else
@@ -559,7 +643,7 @@ bool FormDocument::class_apply()
                params.sides = LyXTextClass::TwoSides;
        else
                params.sides = LyXTextClass::OneSide;
-       
+
        Spacing tmpSpacing = params.spacing;
        switch (fl_get_choice(class_->choice_doc_spacing)) {
        case 1:
@@ -576,48 +660,76 @@ bool FormDocument::class_apply()
                break;
        case 4:
                lyxerr[Debug::INFO] << "Spacing: OTHER\n";
-               params.spacing.set(Spacing::Other, 
-                                  fl_get_input(class_->input_doc_spacing));
+               params.spacing.set(Spacing::Other,
+                                  getString(class_->input_doc_spacing));
                break;
        }
        if (tmpSpacing != params.spacing)
                redo = true;
-       
-       params.options = fl_get_input(class_->input_doc_extra);
-       
+
+       params.options = getString(class_->input_doc_extra);
+
        return redo;
 }
 
 
-void FormDocument::paper_apply()
+void FormDocument::paper_apply(BufferParams & params)
 {
-       BufferParams & params = lv_->buffer()->params;
+       params.papersize2 = char(fl_get_choice(paper_->choice_papersize) - 1);
 
-       params.papersize2 =
-               static_cast<char>(fl_get_choice(paper_->choice_papersize2)-1);
        params.paperpackage =
-               static_cast<char>(fl_get_choice(paper_->choice_paperpackage)-1);
-       params.use_geometry = fl_get_button(paper_->push_use_geometry);
+               char(fl_get_choice(paper_->choice_paperpackage) - 1);
+
+       // set params.papersize from params.papersize2 and params.paperpackage
+       params.setPaperStuff();
+
+       params.use_geometry = fl_get_button(paper_->check_use_geometry);
+
        if (fl_get_button(paper_->radio_landscape))
                params.orientation = BufferParams::ORIENTATION_LANDSCAPE;
        else
                params.orientation = BufferParams::ORIENTATION_PORTRAIT;
-       params.paperwidth = fl_get_input(paper_->input_custom_width);
-       params.paperheight = fl_get_input(paper_->input_custom_height);
-       params.leftmargin = fl_get_input(paper_->input_left_margin);
-       params.topmargin = fl_get_input(paper_->input_top_margin);
-       params.rightmargin = fl_get_input(paper_->input_right_margin);
-       params.bottommargin = fl_get_input(paper_->input_bottom_margin);
-       params.headheight = fl_get_input(paper_->input_head_height);
-       params.headsep = fl_get_input(paper_->input_head_sep);
-       params.footskip = fl_get_input(paper_->input_foot_skip);
-       lv_->buffer()->setPaperStuff();
+
+       params.paperwidth =
+               getLengthFromWidgets(paper_->input_custom_width,
+                                    paper_->choice_custom_width_units);
+
+       params.paperheight =
+               getLengthFromWidgets(paper_->input_custom_height,
+                                    paper_->choice_custom_height_units);
+
+       params.leftmargin =
+               getLengthFromWidgets(paper_->input_inner_margin,
+                                    paper_->choice_inner_margin_units);
+
+       params.topmargin =
+               getLengthFromWidgets(paper_->input_top_margin,
+                                    paper_->choice_top_margin_units);
+
+       params.rightmargin =
+               getLengthFromWidgets(paper_->input_outer_margin,
+                                    paper_->choice_outer_margin_units);
+
+       params.bottommargin =
+               getLengthFromWidgets(paper_->input_bottom_margin,
+                                    paper_->choice_bottom_margin_units);
+
+       params.headheight =
+               getLengthFromWidgets(paper_->input_head_height,
+                                    paper_->choice_head_height_units);
+
+       params.headsep =
+               getLengthFromWidgets(paper_->input_head_sep,
+                                    paper_->choice_head_sep_units);
+
+       params.footskip =
+               getLengthFromWidgets(paper_->input_foot_skip,
+                                    paper_->choice_foot_skip_units);
 }
 
 
-bool FormDocument::language_apply()
+bool FormDocument::language_apply(BufferParams & params)
 {
-       BufferParams & params = lv_->buffer()->params;
        InsetQuotes::quote_language lga = InsetQuotes::EnglishQ;
        bool redo = false;
 
@@ -642,157 +754,167 @@ bool FormDocument::language_apply()
                break;
        }
        params.quotes_language = lga;
-       if (fl_get_button(language_->radio_single))   
+       if (fl_get_button(language_->radio_single))
                params.quotes_times = InsetQuotes::SingleQ;
        else
                params.quotes_times = InsetQuotes::DoubleQ;
 
-       Language const * old_language = params.language;
-       Language const * new_language = 
-               languages.getLanguage(combo_language->getline());
+       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::options_apply()
+bool FormDocument::options_apply(BufferParams & params)
 {
-       BufferParams & params = lv_->buffer()->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_->slider_secnumdepth));
+       int tmpchar = int(fl_get_counter_value(options_->counter_secnumdepth));
        if (params.secnumdepth != tmpchar)
                redo = true;
        params.secnumdepth = tmpchar;
-   
-       params.tocdepth = int(fl_get_counter_value(options_->slider_tocdepth));
+
+       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;
 }
 
 
-void FormDocument::bullets_apply()
+void FormDocument::bullets_apply(BufferParams & params)
 {
        /* update the bullet settings */
-       BufferParams & param = lv_->buffer()->params;
+       BufferParams & buf_params = controller().params();
 
-       // a little bit of loop unrolling
-       param.user_defined_bullets[0] = param.temp_bullets[0];
-       param.user_defined_bullets[1] = param.temp_bullets[1];
-       param.user_defined_bullets[2] = param.temp_bullets[2];
-       param.user_defined_bullets[3] = param.temp_bullets[3];
+       for (int i = 0; i < 4; ++i) {
+               params.user_defined_bullets[i] = buf_params.temp_bullets[i];
+       }
 }
 
 
-void FormDocument::class_update(BufferParams const & params)
+void FormDocument::UpdateClassParams(BufferParams const & params)
 {
-       if (!class_.get())
-               return;
+       // These are the params that have to be updated on any class change
+       // (even if the class defaults are not used) (JSpitzm 2002-04-08)
 
-       LyXTextClass const & tclass = textclasslist.TextClass(params.textclass);
+       LyXTextClass const & tclass = textclasslist[params.textclass];
 
-       combo_doc_class->select(
-               textclasslist.DescOfClass(params.textclass));
-       fl_set_choice_text(class_->choice_doc_fonts, params.fonts.c_str());
+       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(class_->choice_doc_fontsize,
-                     tokenPos(tclass.opt_fontsize(), '|', params.fontsize)+2);
+       fl_set_choice_text(class_->choice_doc_fontsize,
+                          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(class_->choice_doc_pagestyle,
-                     tokenPos(tclass.opt_pagestyle(), '|', params.pagestyle)+2);
-       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);
-       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: 
-               fl_set_choice (class_->choice_doc_skip, 4);
-               fl_set_input (class_->input_doc_skip,
-                             params.getDefSkip().asLyXCommand().c_str());
-               break;
-       default:
-               fl_set_choice (class_->choice_doc_skip, 2);
-               break;
+       fl_set_choice_text(class_->choice_doc_pagestyle,
+                          params.pagestyle.c_str());
+
+}
+
+void FormDocument::class_update(BufferParams const & params)
+{
+       if (!class_.get())
+               return;
+
+       UpdateClassParams(params);
+
+       fl_set_choice_text(class_->choice_doc_fonts, params.fonts.c_str());
+
+       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);
+
+       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_button(class_->radio_doc_sides_one, 0);
-       fl_set_button(class_->radio_doc_sides_two, 0);
-       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);
-       fl_set_input(class_->input_doc_spacing, "");
+       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 > 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_doc_skip_units,
+                                             length, default_unit);
+
+       } else {
+               fl_set_input(class_->input_doc_skip, "");
+       }
+       setEnabled(class_->choice_doc_skip, !indent);
+       setEnabled(class_->input_doc_skip, length_input);
+       setEnabled(class_->choice_doc_skip_units, length_input);
+
+       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);
+
+       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);
+       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());
 }
 
 
@@ -801,7 +923,9 @@ void FormDocument::language_update(BufferParams const & params)
        if (!language_.get())
                return;
 
-       combo_language->select(params.language->lang());
+       int const pos = int(findPos(lang_, params.language->lang()));
+       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);
        fl_set_button(language_->radio_single, 0);
@@ -820,13 +944,13 @@ 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);
        setEnabled(options_->choice_citation_format, params.use_natbib);
-       fl_set_counter_value(options_->slider_secnumdepth, params.secnumdepth);
-       fl_set_counter_value(options_->slider_tocdepth, params.tocdepth);
+       fl_set_counter_value(options_->counter_secnumdepth, params.secnumdepth);
+       fl_set_counter_value(options_->counter_tocdepth, params.tocdepth);
        if (!params.float_placement.empty())
                fl_set_input(options_->input_float_placement,
                             params.float_placement.c_str());
@@ -840,15 +964,19 @@ void FormDocument::paper_update(BufferParams const & params)
        if (!paper_.get())
                return;
 
-       fl_set_choice(paper_->choice_papersize2, params.papersize2 + 1);
+       fl_set_choice(paper_->choice_papersize, params.papersize2 + 1);
        fl_set_choice(paper_->choice_paperpackage, params.paperpackage + 1);
-       fl_set_button(paper_->push_use_geometry, params.use_geometry);
+       fl_set_button(paper_->check_use_geometry, params.use_geometry);
+
+       int const paperchoice = fl_get_choice(paper_->choice_papersize);
+       bool const useCustom = paperchoice == 2;
+       bool const useGeom = fl_get_button(paper_->check_use_geometry);
+
        fl_set_button(paper_->radio_portrait, 0);
-       setEnabled(paper_->radio_portrait,
-                  fl_get_choice(paper_->choice_papersize2) != 2);
+       setEnabled(paper_->radio_portrait, !useCustom);
        fl_set_button(paper_->radio_landscape, 0);
-       setEnabled(paper_->radio_landscape,
-                  fl_get_choice(paper_->choice_papersize2) != 2);
+       setEnabled(paper_->radio_landscape, !useCustom);
+
        if (params.orientation == BufferParams::ORIENTATION_LANDSCAPE)
                fl_set_button(paper_->radio_landscape, 1);
        else
@@ -856,38 +984,69 @@ void FormDocument::paper_update(BufferParams const & params)
        setEnabled(paper_->choice_paperpackage,
                   //either default papersize (preferences)
                   //or document papersize has to be A4
-                  (fl_get_choice(paper_->choice_papersize2) == 7
-                   || fl_get_choice(paper_->choice_papersize2) == 1
-                   && lyxrc.default_papersize == 5)
+                  (paperchoice == 7
+                   || paperchoice == 1 && lyxrc.default_papersize == BufferParams::PAPER_A4PAPER)
                   && fl_get_button(paper_->radio_portrait));
-       fl_set_input(paper_->input_custom_width, params.paperwidth.c_str());
-       setEnabled(paper_->input_custom_width,
-                  fl_get_choice(paper_->choice_papersize2) == 2);
-       fl_set_input(paper_->input_custom_height, params.paperheight.c_str());
-       setEnabled(paper_->input_custom_height,
-                  fl_get_choice(paper_->choice_papersize2) == 2);
-       fl_set_input(paper_->input_left_margin, params.leftmargin.c_str());
-       setEnabled(paper_->input_left_margin,
-                  fl_get_button(paper_->push_use_geometry));
-       fl_set_input(paper_->input_top_margin, params.topmargin.c_str());
-       setEnabled(paper_->input_top_margin,
-                  fl_get_button(paper_->push_use_geometry));
-       fl_set_input(paper_->input_right_margin, params.rightmargin.c_str());
-       setEnabled(paper_->input_right_margin,
-                  fl_get_button(paper_->push_use_geometry));
-       fl_set_input(paper_->input_bottom_margin, params.bottommargin.c_str());
-       setEnabled(paper_->input_bottom_margin,
-                  fl_get_button(paper_->push_use_geometry));
-       fl_set_input(paper_->input_head_height, params.headheight.c_str());
-       setEnabled(paper_->input_head_height,
-                  fl_get_button(paper_->push_use_geometry));
-       fl_set_input(paper_->input_head_sep, params.headsep.c_str());
-       setEnabled(paper_->input_head_sep,
-                  fl_get_button(paper_->push_use_geometry));
-       fl_set_input(paper_->input_foot_skip, params.footskip.c_str());
-       setEnabled(paper_->input_foot_skip,
-                  fl_get_button(paper_->push_use_geometry));
-       fl_set_focus_object(paper_->form, paper_->choice_papersize2);
+
+       // Default unit choice is cm if metric, inches if US paper.
+       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,
+                                     paper_->choice_custom_width_units,
+                                     params.paperwidth, default_unit);
+       setEnabled(paper_->input_custom_width, useCustom);
+       setEnabled(paper_->choice_custom_width_units, useCustom);
+
+       updateWidgetsFromLengthString(paper_->input_custom_height,
+                                     paper_->choice_custom_height_units,
+                                     params.paperheight, default_unit);
+       setEnabled(paper_->input_custom_height, useCustom);
+       setEnabled(paper_->choice_custom_height_units, useCustom);
+
+       updateWidgetsFromLengthString(paper_->input_inner_margin,
+                                     paper_->choice_inner_margin_units,
+                                     params.leftmargin, default_unit);
+       setEnabled(paper_->input_inner_margin, useGeom);
+       setEnabled(paper_->choice_inner_margin_units, useGeom);
+
+       updateWidgetsFromLengthString(paper_->input_top_margin,
+                                     paper_->choice_top_margin_units,
+                                     params.topmargin, default_unit);
+       setEnabled(paper_->input_top_margin, useGeom);
+       setEnabled(paper_->choice_top_margin_units, useGeom);
+
+       updateWidgetsFromLengthString(paper_->input_outer_margin,
+                                     paper_->choice_outer_margin_units,
+                                     params.rightmargin, default_unit);
+       setEnabled(paper_->input_outer_margin, useGeom);
+       setEnabled(paper_->choice_outer_margin_units, useGeom);
+
+       updateWidgetsFromLengthString(paper_->input_bottom_margin,
+                                     paper_->choice_bottom_margin_units,
+                                     params.bottommargin, default_unit);
+       setEnabled(paper_->input_bottom_margin, useGeom);
+       setEnabled(paper_->choice_bottom_margin_units, useGeom);
+
+       updateWidgetsFromLengthString(paper_->input_head_height,
+                                     paper_->choice_head_height_units,
+                                     params.headheight, default_unit);
+       setEnabled(paper_->input_head_height, useGeom);
+       setEnabled(paper_->choice_head_height_units, useGeom);
+
+       updateWidgetsFromLengthString(paper_->input_head_sep,
+                                     paper_->choice_head_sep_units,
+                                     params.headsep, default_unit);
+       setEnabled(paper_->input_head_sep, useGeom);
+       setEnabled(paper_->choice_head_sep_units, useGeom);
+
+       updateWidgetsFromLengthString(paper_->input_foot_skip,
+                                     paper_->choice_foot_skip_units,
+                                     params.footskip, default_unit);
+       setEnabled(paper_->input_foot_skip, useGeom);
+       setEnabled(paper_->choice_foot_skip_units, useGeom);
+
+       fl_set_focus_object(paper_->form, paper_->choice_papersize);
 }
 
 
@@ -898,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;
@@ -913,176 +1073,20 @@ void FormDocument::bullets_update(BufferParams const & params)
 
 void FormDocument::checkReadOnly()
 {
-       if (bc().readOnly(lv_->buffer()->isReadonly())) {
-               combo_doc_class->deactivate();
-               combo_language->deactivate();
-               fl_set_object_label(dialog_->text_warning,
-                                   _("Document is read-only."
-                                     " No changes to layout permitted."));
-               fl_show_object(dialog_->text_warning);
+       if (bc().readOnly(controller().bufferIsReadonly())) {
+               postWarning(_("Document is read-only."
+                             " No changes to layout permitted."));
        } else {
-               combo_doc_class->activate();
-               combo_language->activate();
-               fl_hide_object(dialog_->text_warning);
+               clearMessage();
        }
 }
 
 
-void FormDocument::checkMarginValues()
+void FormDocument::ChoiceBulletSize(FL_OBJECT * ob, long /*data*/)
 {
-//this is not needed after the recent redesign (JSpitzm)
-#if 0
-       bool const not_empty =
-               strlen(fl_get_input(paper_->input_top_margin)) ||
-               strlen(fl_get_input(paper_->input_bottom_margin)) ||
-               strlen(fl_get_input(paper_->input_left_margin)) ||
-               strlen(fl_get_input(paper_->input_right_margin)) ||
-               strlen(fl_get_input(paper_->input_head_height)) ||
-               strlen(fl_get_input(paper_->input_head_sep)) ||
-               strlen(fl_get_input(paper_->input_foot_skip)) ||
-               strlen(fl_get_input(paper_->input_custom_width)) ||
-               strlen(fl_get_input(paper_->input_custom_height));
-#endif
-}
-
-
+       BufferParams & param = controller().params();
 
-bool FormDocument::CheckDocumentInput(FL_OBJECT * ob, long)
-{
-       string str;
-       int val;
-       bool ok = true;
-       char const * input;
-    
-       checkMarginValues();
-       if (ob == paper_->choice_papersize2) {
-               val = fl_get_choice(paper_->choice_papersize2)-1;
-               if (val == BufferParams::VM_PAPER_DEFAULT) {
-                       fl_set_button(paper_->push_use_geometry, 0);
-                       setEnabled(paper_->input_top_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_bottom_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_left_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_right_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_head_height,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_head_sep,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_foot_skip,
-                                  fl_get_button(paper_->push_use_geometry));
-                       checkMarginValues();
-               } else {
-                       if ((val != BufferParams::VM_PAPER_USLETTER) &&
-                           (val != BufferParams::VM_PAPER_USLEGAL) &&
-                           (val != BufferParams::VM_PAPER_USEXECUTIVE) &&
-                           (val != BufferParams::VM_PAPER_A4) &&
-                           (val != BufferParams::VM_PAPER_A5) &&
-                           (val != BufferParams::VM_PAPER_B5)) {
-                               fl_set_button(paper_->push_use_geometry, 1);
-                               setEnabled(paper_->input_top_margin,
-                                          fl_get_button(paper_->push_use_geometry));
-                               setEnabled(paper_->input_bottom_margin,
-                                          fl_get_button(paper_->push_use_geometry));
-                               setEnabled(paper_->input_left_margin,
-                                          fl_get_button(paper_->push_use_geometry));
-                               setEnabled(paper_->input_right_margin,
-                                          fl_get_button(paper_->push_use_geometry));
-                               setEnabled(paper_->input_head_height,
-                                          fl_get_button(paper_->push_use_geometry));
-                               setEnabled(paper_->input_head_sep,
-                                          fl_get_button(paper_->push_use_geometry));
-                               setEnabled(paper_->input_foot_skip,
-                                          fl_get_button(paper_->push_use_geometry));
-                       }
-                       fl_set_choice(paper_->choice_paperpackage,
-                                     BufferParams::PACKAGE_NONE + 1);
-               }
-       } else if (ob == paper_->choice_paperpackage) {
-               val = fl_get_choice(paper_->choice_paperpackage)-1;
-               if (val != BufferParams::PACKAGE_NONE) {
-                       fl_set_button(paper_->push_use_geometry, 0);
-                       setEnabled(paper_->input_top_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_bottom_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_left_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_right_margin,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_head_height,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_head_sep,
-                                  fl_get_button(paper_->push_use_geometry));
-                       setEnabled(paper_->input_foot_skip,
-                                  fl_get_button(paper_->push_use_geometry));
-               }
-       } else if (ob == class_->input_doc_spacing) {
-               input = fl_get_input(class_->input_doc_spacing);
-               if (!*input) {
-                       fl_set_choice (class_->choice_doc_spacing, 1);
-               } else {
-                       fl_set_choice(class_->choice_doc_spacing, 4);
-               }
-       }
-       // 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.
-       str = fl_get_input(paper_->input_custom_width);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_custom_height);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_left_margin);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_right_margin);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_top_margin);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_bottom_margin);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_head_height);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_head_sep);
-       ok = ok && (str.empty() || isValidLength(str));
-       str = fl_get_input(paper_->input_foot_skip);
-       ok = ok && (str.empty() || isValidLength(str));
-       // "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 (ob == class_->input_doc_skip) {
-               if (!*input) {
-                       fl_set_choice (class_->choice_doc_skip, 2);
-               } else if (isValidGlueLength (input)) {
-                       fl_set_choice (class_->choice_doc_skip, 4);
-               } else {
-                       fl_set_choice(class_->choice_doc_skip, 4);
-                       ok = false;
-               }
-       } else {
-               if (*input && !isValidGlueLength(input))
-                       ok = false;
-       }
-
-       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;
-
-       // convert from 1-6 range to -1-4 
+       // convert from 1-6 range to -1-4
        param.temp_bullets[current_bullet_depth].setSize(fl_get_choice(ob) - 2);
        fl_set_input(bullets_->input_bullet_latex,
                     param.temp_bullets[current_bullet_depth].getText().c_str());
@@ -1091,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 */
@@ -1108,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)) {
@@ -1134,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) {
@@ -1160,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,
@@ -1196,15 +1191,15 @@ void FormDocument::BulletPanel(FL_OBJECT * /*ob*/, State cb)
 }
 
 
-void FormDocument::BulletBMTable(FL_OBJECT * ob, long /*data*/ )
+void FormDocument::BulletBMTable(FL_OBJECT * ob, long /*data*/)
 {
        /* handle the user input by setting the current bullet depth's pixmap */
        /* 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 */
        /*  fl_set_bmtable(ob, 1, bmtable_button); */
        param.temp_bullets[current_bullet_depth].setFont(current_bullet_panel);
@@ -1214,32 +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.Load(tc)) {
-               // 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)
+                       UpdateClassParams(params);
                }
+
        } else {
-               // unable to load new style
-               WriteAlert(_("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();
 }