]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/xforms/FormCitation.C
remove defaults stuff, let Qt handle no toolbar
[lyx.git] / src / frontends / xforms / FormCitation.C
index df45cd020cd8acaec9a841cf3fb0ef17bbd2f726..6897380c652c1f3a100a65d5ec5242b3d11525c0 100644 (file)
-// -*- C++ -*-
-/* This file is part of
- * ====================================================== 
+/**
+ * \file FormCitation.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- *           LyX, The Document Processor
+ * \author Angus Leeming
+ * \author Rob Lahaye
  *
- *           Copyright 2000 The LyX Team.
- *
- * ======================================================
+ * Full author contact details are available in file CREDITS
  */
 
 #include <config.h>
-
-#include FORMS_H_LOCATION
-
 #include <algorithm>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
 
+#include "xformsBC.h"
+#include "ControlCitation.h"
+#include "FormCitation.h"
+#include "forms/form_citation.h"
+#include "Kernel.h"
+#include "Tooltips.h"
+#include "helper_funcs.h"
+#include "xforms_helpers.h"
+#include FORMS_H_LOCATION
 
 #include "gettext.h"
-#include "Dialogs.h"
-#include "FormCitation.h"
-#include "LyXView.h"
-#include "buffer.h"
-#include "form_citation.h"
-#include "lyxfunc.h"
-#include "support/filetools.h"
+#include "support/LAssert.h"
+#include "support/lstrings.h"
 
-using std::vector;
-using std::pair;
+using std::find;
 using std::max;
 using std::min;
-using std::find;
+using std::pair;
+using std::sort;
+using std::vector;
 
-static vector<string> citekeys;
-static vector<string> bibkeys;
-static vector<string> bibkeysInfo;
+namespace {
 
-FormCitation::FormCitation(LyXView * lv, Dialogs * d)
-       : FormCommand(lv, d, _("Citation")), dialog_(0)
+// shamelessly stolen from Menubar_pimpl.C
+int string_width(string const & str)
 {
-       // let the dialog be shown
-       // These are permanent connections so we won't bother
-       // storing a copy because we won't be disconnecting.
-       d->showCitation.connect(slot(this, &FormCitation::showInset));
-       d->createCitation.connect(slot(this, &FormCitation::createInset));
+       return fl_get_string_widthTAB(FL_NORMAL_STYLE, FL_NORMAL_SIZE,
+                                     str.c_str(),
+                                     static_cast<int>(str.length()));
 }
 
 
-FormCitation::~FormCitation()
+void fillChoice(FD_citation * dialog, vector<string> vec)
 {
-       free();
-       delete dialog_;
+       // Check whether the current contents of the browser will be
+       // changed by loading the contents of the vec...
+       vector<string> const choice_style = getVector(dialog->choice_style);
+
+       if (vec == choice_style)
+               return;
+
+       // They will be changed. Proceed
+       bool const noVec = vec.empty();
+       string const str = noVec ? string() : getStringFromVector(vec, "|");
+
+       fl_clear_choice(dialog->choice_style);
+       fl_addto_choice(dialog->choice_style, str.c_str());
+       setEnabled(dialog->choice_style, !noVec);
 }
 
 
-void FormCitation::clearStore()
+void updateStyle(FD_citation * dialog, string command)
 {
-       citekeys.clear();
-       bibkeys.clear();
-       bibkeysInfo.clear();
+       // Find the style of the citekeys
+       vector<biblio::CiteStyle> const & styles =
+               ControlCitation::getCiteStyles();
+       biblio::CitationStyle cs = biblio::getCitationStyle(command);
+
+       vector<biblio::CiteStyle>::const_iterator cit =
+               find(styles.begin(), styles.end(), cs.style);
+
+       // Use this to initialise the GUI
+       bool const noStyles = cit == styles.end();
+       int const index = 1 + ( noStyles ? 0 : int(cit - styles.begin()) );
+       fl_set_choice(dialog->choice_style, index);
+
+       // Disable if there are no styles, otherwise use cs member settings.
+       fl_set_button(dialog->check_full_author_list, !noStyles && cs.full);
+       fl_set_button(dialog->check_force_uppercase, !noStyles && cs.forceUCase);
 }
 
+} // namespace anon
 
-void FormCitation::build()
+
+typedef FormController<ControlCitation, FormView<FD_citation> > base_class;
+
+
+FormCitation::FormCitation(Dialog & parent)
+       : base_class(parent, _("Citation"))
+{}
+
+
+void FormCitation::apply()
 {
-       dialog_ = build_citation();
+       string command = "cite";
+       if (isActive(dialog_->choice_style)) {
+               vector<biblio::CiteStyle> const & styles =
+                       ControlCitation::getCiteStyles();
+
+               int const choice =
+                       std::max(0, fl_get_choice(dialog_->choice_style) - 1);
+               bool const full  =
+                       fl_get_button(dialog_->check_full_author_list);
+               bool const force =
+                       fl_get_button(dialog_->check_force_uppercase);
+
+               command = biblio::getCiteCommand(styles[choice], full, force);
+       }
+
+       controller().params().setCmdName(command);
+       controller().params().setContents(getStringFromVector(citekeys));
+
+       string const after  = getString(dialog_->input_after);
+       controller().params().setOptions(after);
 }
 
 
-FL_FORM * const FormCitation::form() const
+void FormCitation::hide()
 {
-       if( dialog_ && dialog_->form_citation )
-               return dialog_->form_citation;
-       else
-               return 0;
+       citekeys.clear();
+       bibkeys.clear();
+
+       FormDialogView::hide();
 }
 
 
-void FormCitation::update()
+void FormCitation::build()
 {
-       bibkeys.clear();
-       bibkeysInfo.clear();
+       dialog_.reset(build_citation(this));
 
-       vector<pair<string,string> > blist =
-               lv_->buffer()->getBibkeyList();
+       // Manage the 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);
 
-       for( unsigned int i = 0; i < blist.size(); ++i ) {
-               bibkeys.push_back(blist[i].first);
-               bibkeysInfo.push_back(blist[i].second);
-       }
-       blist.clear();
+       // disable for read-only documents
+       bcview().addReadOnly(dialog_->button_add);
+       bcview().addReadOnly(dialog_->button_del);
+       bcview().addReadOnly(dialog_->button_up);
+       bcview().addReadOnly(dialog_->button_down);
+       bcview().addReadOnly(dialog_->choice_style);
+       bcview().addReadOnly(dialog_->input_before);
+       bcview().addReadOnly(dialog_->input_after);
+       bcview().addReadOnly(dialog_->check_full_author_list);
+       bcview().addReadOnly(dialog_->check_force_uppercase);
 
-       citekeys.clear();
-       string tmp, keys( params.getContents() );
-       keys = frontStrip( split(keys, tmp, ',') );
-       while( !tmp.empty() ) {
-               citekeys.push_back( tmp );
-               keys = frontStrip( split(keys, tmp, ',') );
-       }
+       // trigger an input event for cut&paste with middle mouse button.
+       setPrehandler(dialog_->input_search);
+       setPrehandler(dialog_->input_before);
+       setPrehandler(dialog_->input_after);
 
-       fl_freeze_form( dialog_->form_citation );
+       fl_set_input_return(dialog_->input_after,  FL_RETURN_CHANGED);
+       fl_set_input_return(dialog_->input_before, FL_RETURN_CHANGED);
+       fl_set_input_return(dialog_->input_search, FL_RETURN_END);
 
-       updateBrowser( dialog_->bibBrsr,  bibkeys );
-       updateBrowser( dialog_->citeBrsr, citekeys );
-       fl_clear_browser( dialog_->infoBrsr );
+       //set up the tooltip mechanism
+       string str = _("Add the selected entry to the current citation reference.");
+       tooltips().init(dialog_->button_add, str);
 
-       // No keys have been selected yet, so...
-       setBibButtons( OFF );
-       setCiteButtons( OFF );
+       str = _("Delete the selected entry from the current citation reference.");
+       tooltips().init(dialog_->button_del, str);
 
-       int noKeys = max( bibkeys.size(), citekeys.size() );
+       str = _("Move the selected entry upwards (in the current list).");
+       tooltips().init(dialog_->button_up, str);
 
-       // Place bounds, so that 4 <= noKeys <= 15
-       noKeys = max( 4, min(15, noKeys) );
+       str = _("Move the selected entry downwards (in the current list).");
+       tooltips().init(dialog_->button_down, str);
 
-       // Re-size the form to accommodate the new browser size
-       int size = 20 * noKeys;
-       bool bibPresent = ( bibkeys.size() > 0 );
-       setSize( size, bibPresent );
+       str = _("The entries which will be cited. Select them with the arrow buttons from the right browser window.");
+       tooltips().init(dialog_->browser_cite, str);
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_cite);
 
-       fl_set_input( dialog_->textAftr, params.getOptions().c_str() );
+       str = _("All entries in the database you have loaded (via \"Insert->Lists&TOC->BibTex Reference\"). Move the ones you want to cite with the arrow buttons into the left browser window.");
+       tooltips().init(dialog_->browser_bib, str);
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_bib);
 
-       fl_unfreeze_form( dialog_->form_citation );
-}
+       str = _("Information about the selected entry");
+       tooltips().init(dialog_->browser_info, str);
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_info);
 
+       str = _("Here you may select how the citation label should look inside the text (Natbib).");
+       tooltips().init(dialog_->choice_style, str);
 
-void FormCitation::updateBrowser( FL_OBJECT * browser,
-                                 vector<string> const & keys ) const
-{
-       fl_clear_browser( browser );
+       str = _("Activate if you want to print all authors in a reference with more than three authors, and not \"<First Author> et al.\" (Natbib).");
+       tooltips().init(dialog_->check_full_author_list, str);
 
-       for( unsigned int i = 0; i < keys.size(); ++i )
-               fl_add_browser_line( browser, keys[i].c_str() );
-}
+       str = _("Activate if you want to print the first character of the author name as uppercase (\"Van Gogh\", not \"van Gogh\"). Useful at the beginning of sentences (Natbib).");
+       tooltips().init(dialog_->check_force_uppercase, str);
 
+       str = _("Optional text which appears before the citation reference, e.g. \"see <Ref>\"");
+       tooltips().init(dialog_->input_before, str);
 
-void FormCitation::setBibButtons( State status ) const
-{
-       switch (status) {
-       case ON:
-               fl_activate_object( dialog_->addBtn );
-               fl_set_object_lcol( dialog_->addBtn, FL_BLACK );
-               break;
-
-       case OFF:
-               fl_deactivate_object( dialog_->addBtn );
-               fl_set_object_lcol( dialog_->addBtn, FL_INACTIVE );
-               break;
-
-       default:
-               break;
-       }
+       str = _("Optional text which appears after the citation reference, e.g. \"pp. 12\"");
+       tooltips().init(dialog_->input_after, str);
+
+       str = _("Search your database (all fields will be searched).");
+       tooltips().init(dialog_->input_search, str);
+
+       str = _("Activate if you want to have case sensitive search: \"bibtex\" finds \"bibtex\", but not \"BibTeX\".");
+       tooltips().init(dialog_->check_search_case, str);
+
+       str = _("Activate if you want to enter Regular Expressions.");
+       tooltips().init(dialog_->check_search_type, str);
 }
 
 
-void FormCitation::setCiteButtons( State status ) const
+void FormCitation::findBiblio(biblio::Direction const dir)
 {
-       switch( status ) {
-       case ON:
-        {
-               fl_activate_object( dialog_->delBtn );
-               fl_set_object_lcol( dialog_->delBtn, FL_BLACK );
-
-               int sel = fl_get_browser( dialog_->citeBrsr );
-
-               if( sel != 1 ) {
-                       fl_activate_object( dialog_->upBtn );
-                       fl_set_object_lcol( dialog_->upBtn, FL_BLACK );
-               } else {
-                       fl_deactivate_object( dialog_->upBtn );
-                       fl_set_object_lcol( dialog_->upBtn, FL_INACTIVE );
-               }
+       string const str = getString(dialog_->input_search);
+       biblio::InfoMap const & theMap = controller().bibkeysInfo();
+       bool const caseSensitive =
+               fl_get_button(dialog_->check_search_case);
+       biblio::Search const type =
+               fl_get_button(dialog_->check_search_type) ?
+               biblio::REGEX : biblio::SIMPLE;
+
+       vector<string>::const_iterator start = bibkeys.begin();
+       int const sel = fl_get_browser(dialog_->browser_bib);
+       if (sel >= 1 && sel <= int(bibkeys.size()))
+               start += sel - 1;
+
+       // Find the NEXT instance...
+       (dir == biblio::FORWARD) ? ++start : --start;
+
+
+       vector<string>::const_iterator const cit =
+               biblio::searchKeys(theMap, bibkeys, str,
+                                  start, type, dir, caseSensitive);
+
+       if (cit == bibkeys.end())
+               return;
+
+       int const found = int(cit - bibkeys.begin()) + 1;
+       if (found == sel)
+               return;
+
+       // Update the display
+       int const top = max(found - 5, 1);
+       fl_set_browser_topline(dialog_->browser_bib, top);
+       fl_select_browser_line(dialog_->browser_bib, found);
+       input(dialog_->browser_bib, 0);
+}
 
-               if( sel != fl_get_browser_maxline(dialog_->citeBrsr)) {
-                       fl_activate_object( dialog_->downBtn );
-                       fl_set_object_lcol( dialog_->downBtn, FL_BLACK );
-               } else {
-                       fl_deactivate_object( dialog_->downBtn );
-                       fl_set_object_lcol( dialog_->downBtn, FL_INACTIVE );
-               }
 
-               break;
-       }
-       case OFF:
-       {
-               fl_deactivate_object( dialog_->delBtn );
-               fl_set_object_lcol( dialog_->delBtn, FL_INACTIVE );
+ButtonPolicy::SMInput FormCitation::input(FL_OBJECT * ob, long)
+{
+       ButtonPolicy::SMInput activate = ButtonPolicy::SMI_NOOP;
 
-               fl_deactivate_object( dialog_->upBtn );
-               fl_set_object_lcol( dialog_->upBtn, FL_INACTIVE );
+       biblio::InfoMap const & theMap = controller().bibkeysInfo();
 
-               fl_deactivate_object( dialog_->downBtn );
-               fl_set_object_lcol( dialog_->downBtn, FL_INACTIVE );
-       }
-       default:
-               break;
-       }
-}
+       string topCitekey;
+       if (!citekeys.empty()) topCitekey = citekeys[0];
 
+       if (ob == dialog_->browser_bib) {
+               fl_deselect_browser(dialog_->browser_cite);
 
-void FormCitation::setSize( int hbrsr, bool bibPresent ) const
-{
-       int const hinfo  = dialog_->infoBrsr->h;
-       int const hother = 140;
-       hbrsr = max( hbrsr, 175 );
-       int wform = dialog_->form_citation->w;
-       int hform = hbrsr + hother;
-
-       if( bibPresent ) hform += hinfo + 30;
-       fl_set_form_size( dialog_->form_citation, wform, hform );
-
-       // No resizing is alowed in the y-direction
-       fl_set_form_minsize( dialog_->form_citation, wform,   hform );
-       fl_set_form_maxsize( dialog_->form_citation, 3*wform, hform );
-
-       int y = 0;
-       fl_set_object_geometry( dialog_->box, 0, y, wform, hform );
-       y += 30;
-       fl_set_object_geometry( dialog_->citeBrsr, 10, y, 180, hbrsr );
-       fl_set_object_geometry( dialog_->bibBrsr, 240, y, 180, hbrsr );
-
-       fl_set_object_position( dialog_->addBtn,  200, y );
-       y += 5 + dialog_->addBtn->h;
-       fl_set_object_position( dialog_->delBtn,  200, y );
-       y += 5 + dialog_->delBtn->h;
-       fl_set_object_position( dialog_->upBtn,   200, y );
-       y += 5 + dialog_->upBtn->h;
-       fl_set_object_position( dialog_->downBtn, 200, y );
-
-       y = dialog_->bibBrsr->y + dialog_->bibBrsr->h;
-
-       // awaiting natbib support
-       fl_hide_object( dialog_->style );
-
-       if( bibPresent ) {
-               y += 30;
-               fl_set_object_position( dialog_->infoBrsr, 10, y );
-               fl_show_object( dialog_->infoBrsr );
-               y += hinfo;
-       }
-       else
-               fl_hide_object( dialog_->infoBrsr );
+               unsigned int const sel = fl_get_browser(dialog_->browser_bib);
+               if (sel < 1 || sel > bibkeys.size())
+                       return ButtonPolicy::SMI_NOOP;
 
-       y += 20;
-       // awaiting natbib support
-       fl_hide_object( dialog_->textBefore );
+               // Put into browser_info the additional info associated with
+               // the selected browser_bib key
+               fl_clear_browser(dialog_->browser_info);
 
-       fl_set_object_position( dialog_->textAftr, 100, y );
-       fl_set_object_position( dialog_->ok,       230, y+50 );
-       fl_set_object_position( dialog_->cancel,   330, y+50 );
-}
+               string const tmp = formatted(biblio::getInfo(theMap,
+                                                            bibkeys[sel - 1]),
+                                             dialog_->browser_info->w - 10);
+               fl_add_browser_line(dialog_->browser_info, tmp.c_str());
 
+               // Highlight the selected browser_bib key in browser_cite if
+               // present
+               vector<string>::const_iterator cit =
+                       find(citekeys.begin(), citekeys.end(), bibkeys[sel - 1]);
 
-void FormCitation::input( long data )
-{
-       State cb = static_cast<FormCitation::State>( data );
-
-       switch( cb ) {
-       case BIBBRSR:
-       {
-               fl_deselect_browser( dialog_->citeBrsr );
-               
-               unsigned int sel = fl_get_browser( dialog_->bibBrsr );
-               if( sel < 1 || sel > bibkeys.size() ) break;
-
-               // Put into infoBrsr the additional info associated with
-               // the selected bibBrsr key
-               fl_clear_browser( dialog_->infoBrsr );
-               fl_add_browser_line( dialog_->infoBrsr,
-                                    bibkeysInfo[sel-1].c_str() );
-
-               // Highlight the selected bibBrsr key in citeBrsr if present
-               vector<string>::iterator it =
-                       find( citekeys.begin(), citekeys.end(), bibkeys[sel-1] );
-
-               if( it != citekeys.end() ) {
-                       int n = it - citekeys.begin();
-                       fl_select_browser_line( dialog_->citeBrsr, n+1 );
-                       fl_set_browser_topline( dialog_->citeBrsr, n+1 );
+               if (cit != citekeys.end()) {
+                       int const n = int(cit - citekeys.begin());
+                       fl_select_browser_line(dialog_->browser_cite, n + 1);
+                       fl_set_browser_topline(dialog_->browser_cite, n + 1);
                }
 
-               if( !lv_->buffer()->isReadonly() ) {
-                       if( it != citekeys.end() ) {
-                               setBibButtons( OFF );
-                               setCiteButtons( ON );
+               if (!kernel().isBufferReadonly()) {
+                       if (cit != citekeys.end()) {
+                               setBibButtons(OFF);
+                               setCiteButtons(ON);
                        } else {
-                               setBibButtons( ON );
-                               setCiteButtons( OFF );
+                               setBibButtons(ON);
+                               setCiteButtons(OFF);
                        }
                }
-       }
-       break;
-       case CITEBRSR:
-       {
-               unsigned int sel = fl_get_browser( dialog_->citeBrsr );
-               if( sel < 1 || sel > citekeys.size() ) break;
-
-               if( !lv_->buffer()->isReadonly() ) {
-                       setBibButtons( OFF );
-                       setCiteButtons( ON );
-               }
 
-               // Highlight the selected citeBrsr key in bibBrsr
-               vector<string>::iterator it =
-                       find( bibkeys.begin(), bibkeys.end(), citekeys[sel-1] );
+       } else if (ob == dialog_->browser_cite) {
+               unsigned int const sel = fl_get_browser(dialog_->browser_cite);
+               if (sel < 1 || sel > citekeys.size())
+                       return ButtonPolicy::SMI_NOOP;
 
-               if (it != bibkeys.end()) {
-                       int n = it - bibkeys.begin();
-                       fl_select_browser_line( dialog_->bibBrsr, n+1 );
-                       fl_set_browser_topline( dialog_->bibBrsr, n+1 );
+               if (!kernel().isBufferReadonly()) {
+                       setBibButtons(OFF);
+                       setCiteButtons(ON);
+               }
 
-                       // Put into infoBrsr the additional info associated with
-                       // the selected citeBrsr key
-                       fl_clear_browser( dialog_->infoBrsr );
-                       fl_add_browser_line( dialog_->infoBrsr,
-                                            bibkeysInfo[n].c_str() );
+               // Highlight the selected browser_cite key in browser_bib
+               vector<string>::const_iterator cit =
+                       find(bibkeys.begin(), bibkeys.end(), citekeys[sel - 1]);
+
+               if (cit != bibkeys.end()) {
+                       int const n = int(cit - bibkeys.begin());
+                       fl_select_browser_line(dialog_->browser_bib, n + 1);
+                       fl_set_browser_topline(dialog_->browser_bib, n + 1);
+
+                       // Put into browser_info the additional info associated
+                       // with the selected browser_cite key
+                       fl_clear_browser(dialog_->browser_info);
+                       string const tmp =
+                               formatted(biblio::getInfo(theMap, citekeys[sel - 1]),
+                                         dialog_->browser_info->w - 10);
+                       fl_add_browser_line(dialog_->browser_info, tmp.c_str());
                }
-       }
-       break;
-       case ADD:
-       {
-               if( lv_->buffer()->isReadonly() ) break;
 
-               unsigned int sel = fl_get_browser( dialog_->bibBrsr );
-               if( sel < 1 || sel > bibkeys.size() ) break;
+       } else if (ob == dialog_->button_add) {
+               unsigned int const sel = fl_get_browser(dialog_->browser_bib);
+               if (sel < 1 || sel > bibkeys.size())
+                       return ButtonPolicy::SMI_NOOP;
 
-               // Add the selected bibBrsr key to citeBrsr
-               fl_addto_browser( dialog_->citeBrsr,
-                                 bibkeys[sel-1].c_str() );
-               citekeys.push_back( bibkeys[sel-1] );
+               // Add the selected browser_bib key to browser_cite
+               fl_addto_browser(dialog_->browser_cite,
+                                 bibkeys[sel - 1].c_str());
+               citekeys.push_back(bibkeys[sel - 1]);
 
-               int n = citekeys.size();
-               fl_select_browser_line( dialog_->citeBrsr, n );
+               int const n = int(citekeys.size());
+               fl_select_browser_line(dialog_->browser_cite, n);
 
-               setBibButtons( OFF );
-               setCiteButtons( ON );
-       }
-       break;
-       case DELETE:
-       {
-               if( lv_->buffer()->isReadonly() ) break;
+               setBibButtons(OFF);
+               setCiteButtons(ON);
+               activate = ButtonPolicy::SMI_VALID;
 
-               unsigned int sel = fl_get_browser( dialog_->citeBrsr );
-               if( sel < 1 || sel > citekeys.size() ) break;
+       } else if (ob == dialog_->button_del) {
+               unsigned int const sel = fl_get_browser(dialog_->browser_cite);
+               if (sel < 1 || sel > citekeys.size())
+                       return ButtonPolicy::SMI_NOOP;
 
-               // Remove the selected key from citeBrsr
-               fl_delete_browser_line( dialog_->citeBrsr, sel ) ;
-               citekeys.erase( citekeys.begin() + sel-1 );
+               // Remove the selected key from browser_cite
+               fl_delete_browser_line(dialog_->browser_cite, sel) ;
+               citekeys.erase(citekeys.begin() + sel - 1);
 
-               setBibButtons( ON );
-               setCiteButtons( OFF );
-       }
-       break;
-       case UP:
-       {
-               if( lv_->buffer()->isReadonly() ) break;
+               setBibButtons(ON);
+               setCiteButtons(OFF);
+               activate = ButtonPolicy::SMI_VALID;
 
-               unsigned int sel = fl_get_browser( dialog_->citeBrsr );
-               if( sel < 2 || sel > citekeys.size() ) break;
+       } else if (ob == dialog_->button_up) {
+               unsigned int const sel = fl_get_browser(dialog_->browser_cite);
+               if (sel < 2 || sel > citekeys.size())
+                       return ButtonPolicy::SMI_NOOP;
 
                // Move the selected key up one line
-               vector<string>::iterator it = citekeys.begin() + sel-1;
-               string tmp = *it;
+               vector<string>::iterator it = citekeys.begin() + sel - 1;
+               string const tmp = *it;
 
-               fl_delete_browser_line( dialog_->citeBrsr, sel );
-               citekeys.erase( it );
+               fl_delete_browser_line(dialog_->browser_cite, sel);
+               citekeys.erase(it);
 
-               fl_insert_browser_line( dialog_->citeBrsr, sel-1, tmp.c_str() );
-               fl_select_browser_line( dialog_->citeBrsr, sel-1 );
-               citekeys.insert( it-1, tmp );
-               setCiteButtons( ON );
-       }
-       break;
-       case DOWN:
-       {
-               if( lv_->buffer()->isReadonly() ) break;
+               fl_insert_browser_line(dialog_->browser_cite, sel - 1, tmp.c_str());
+               fl_select_browser_line(dialog_->browser_cite, sel - 1);
+               citekeys.insert(it - 1, tmp);
+               setCiteButtons(ON);
+               activate = ButtonPolicy::SMI_VALID;
 
-               unsigned int sel = fl_get_browser( dialog_->citeBrsr );
-               if( sel < 1 || sel > citekeys.size()-1 ) break;
+       } else if (ob == dialog_->button_down) {
+               unsigned int const sel = fl_get_browser(dialog_->browser_cite);
+               if (sel < 1 || sel > citekeys.size() - 1)
+                       return ButtonPolicy::SMI_NOOP;
 
                // Move the selected key down one line
-               vector<string>::iterator it = citekeys.begin() + sel-1;
-               string tmp = *it;
+               vector<string>::iterator it = citekeys.begin() + sel - 1;
+               string const tmp = *it;
+
+               fl_delete_browser_line(dialog_->browser_cite, sel);
+               citekeys.erase(it);
+
+               fl_insert_browser_line(dialog_->browser_cite, sel+1, tmp.c_str());
+               fl_select_browser_line(dialog_->browser_cite, sel+1);
+               citekeys.insert(it+1, tmp);
+               setCiteButtons(ON);
+               activate = ButtonPolicy::SMI_VALID;
+
+       } else if (ob == dialog_->button_previous) {
+               findBiblio(biblio::BACKWARD);
+       } else if (ob == dialog_->button_next) {
+               findBiblio(biblio::FORWARD);
+       } else if (ob == dialog_->input_search) {
+               findBiblio(biblio::FORWARD);
+       } else if (ob == dialog_->choice_style ||
+                  ob == dialog_->check_full_author_list ||
+                  ob == dialog_->check_force_uppercase ||
+                  ob == dialog_->input_before ||
+                  ob == dialog_->input_after) {
+               activate = ButtonPolicy::SMI_VALID;
+       }
 
-               fl_delete_browser_line( dialog_->citeBrsr, sel );
-               citekeys.erase( it );
+       string currentCitekey;
+       if (!citekeys.empty())
+               currentCitekey = citekeys[0];
 
-               fl_insert_browser_line( dialog_->citeBrsr, sel+1, tmp.c_str() );
-               fl_select_browser_line( dialog_->citeBrsr, sel+1 );
-               citekeys.insert( it+1, tmp );
-               setCiteButtons( ON );
-       }
-       break;
-       default:
-               break;
+       if (topCitekey != currentCitekey) {
+               int choice = std::max(1, fl_get_choice(dialog_->choice_style));
+               fillChoice(dialog_.get(),
+                          controller().getCiteStrings(currentCitekey));
+               fl_set_choice(dialog_->choice_style, choice);
        }
+
+       return activate;
 }
 
 
-void FormCitation::apply()
+void FormCitation::update()
 {
-       if( lv_->buffer()->isReadonly() ) return;
+       // Make the list of all available bibliography keys
+       bibkeys = biblio::getKeys(controller().bibkeysInfo());
+       updateBrowser(dialog_->browser_bib, bibkeys);
+
+       // Ditto for the keys cited in this inset
+       citekeys = getVectorFromString(controller().params().getContents());
+       updateBrowser(dialog_->browser_cite, citekeys);
+
+       // Use the first citekey to fill choice_style
+       string key;
+       if (!citekeys.empty())
+               key = citekeys[0];
+
+       fillChoice(dialog_.get(), controller().getCiteStrings(key));
+
+       // Use the citation command to update the GUI
+       updateStyle(dialog_.get(), controller().params().getCmdName());
+
+       bool const natbib = controller().usingNatbib();
+       setEnabled(dialog_->check_full_author_list, natbib);
+       setEnabled(dialog_->check_force_uppercase, natbib);
+       setEnabled(dialog_->choice_style, natbib);
+
+       // No keys have been selected yet, so...
+       fl_clear_browser(dialog_->browser_info);
+       setBibButtons(OFF);
+       setCiteButtons(OFF);
+
+       // Natbib can have comments before and after the citation.
+       // This is not yet supported. After only.
+       fl_set_input(dialog_->input_after,
+                    controller().params().getOptions().c_str());
+
+       fl_set_input(dialog_->input_before, _("Not yet supported"));
+       setEnabled(dialog_->input_before, false);
+}
+
 
-       string contents;
-       for( unsigned int i = 0; i < citekeys.size(); ++i ) {
-               if (i > 0) contents += ", ";
-               contents += citekeys[i];
+void FormCitation::updateBrowser(FL_OBJECT * browser,
+                                vector<string> const & keys) const
+{
+       // Check whether the current contents of the browser will be
+       // changed by loading the contents of the vec...
+       vector<string> browser_keys = getVector(browser);
+
+       if (browser_keys == keys) {
+               fl_deselect_browser(browser);
+               fl_set_browser_topline(browser, 1);
+               return;
        }
 
-       params.setContents( contents );
-       params.setOptions( fl_get_input(dialog_->textAftr) );
-
-       if( inset_ != 0 )
-       {
-               // Only update if contents have changed
-               if( params.getCmdName()  != inset_->getCmdName()  ||
-                   params.getContents() != inset_->getContents() ||
-                   params.getOptions()  != inset_->getOptions() ) {
-                       inset_->setParams( params );
-                       lv_->view()->updateInset( inset_, true );
-               }
-       } else {
-               lv_->getLyXFunc()->Dispatch( LFUN_CITATION_INSERT,
-                                            params.getAsString().c_str() );
+       // They will be changed. Proceed.
+       fl_clear_browser(browser);
+
+       for (vector<string>::const_iterator it = keys.begin();
+            it != keys.end(); ++it) {
+               string key = trim(*it);
+               if (!key.empty())
+                       fl_add_browser_line(browser, key.c_str());
        }
 }
+
+
+void FormCitation::setBibButtons(State status) const
+{
+       setEnabled(dialog_->button_add, (status == ON));
+}
+
+
+void FormCitation::setCiteButtons(State status) const
+{
+       int const sel     = fl_get_browser(dialog_->browser_cite);
+       int const maxline = fl_get_browser_maxline(dialog_->browser_cite);
+       bool const activate      = (status == ON);
+       bool const activate_up   = (activate && sel != 1);
+       bool const activate_down = (activate && sel != maxline);
+
+       setEnabled(dialog_->button_del,  activate);
+       setEnabled(dialog_->button_up,   activate_up);
+       setEnabled(dialog_->button_down, activate_down);
+}