]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/xforms/FormCitation.C
Introduce LFUN_PRINT.
[lyx.git] / src / frontends / xforms / FormCitation.C
index fac2d3b8c032db54bbd144f071d1899901c51f50..ffa4567d73021d658117884fb5f2b8065905731c 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 <algorithm>
 
-#include "gettext.h"
-#include FORMS_H_LOCATION
-#include "xform_macros.h"
 #include "FormCitation.h"
-#include "Dialogs.h"
-#include "LyXView.h"
-#include "lyxfunc.h"
-#include "insets/insetcite.h"
-#include "form_citation.h"
-#include "buffer.h"
-#include "BufferView.h"
-#include "support/filetools.h"
-
-#ifdef __GNUG__
-#pragma implementation
-#endif
+#include "ControlCitation.h"
+#include "forms/form_citation.h"
 
-using std::vector;
-using std::pair;
-using std::max;
-using std::min;
-using std::find;
+#include "Tooltips.h"
+#include "xforms_helpers.h"
+#include "xformsBC.h"
 
-C_RETURNCB(FormCitation, WMHideCB)
-C_GENERICCB(FormCitation, OKCB)
-C_GENERICCB(FormCitation, CancelCB)
-C_GENERICCB(FormCitation, InputCB)
+#include "support/lstrings.h"
 
-FormCitation::FormCitation(LyXView * lv, Dialogs * d)
-       : dialog_(0), lv_(lv), d_(d), h_(0), inset_(0), dialogIsOpen(false)
-{
-       // 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));
-       d->hideCitation.connect(slot(this, &FormCitation::hideInset));
-}
+#include "lyx_forms.h"
 
+using lyx::support::getStringFromVector;
+using lyx::support::getVectorFromString;
+using lyx::support::trim;
 
-FormCitation::~FormCitation()
-{
-       free();
-}
+using std::find;
+using std::max;
 
+using std::vector;
+using std::string;
 
-void FormCitation::build()
+
+namespace {
+
+// shamelessly stolen from Menubar_pimpl.C
+int string_width(string const & str)
 {
-  dialog_ = build_citation();
+       return fl_get_string_widthTAB(FL_NORMAL_STYLE, FL_NORMAL_SIZE,
+                                     str.c_str(),
+                                     static_cast<int>(str.length()));
 }
 
 
-void FormCitation::showInset( InsetCitation * inset )
+void fillChoice(FD_citation * dialog, vector<string> vec)
 {
-       if( dialogIsOpen || inset == 0 ) return;
+       // 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;
 
-       inset_ = inset;
+       // They will be changed. Proceed
+       bool const noVec = vec.empty();
+       string const str = noVec ? string() : getStringFromVector(vec, "|");
 
-       textAfter = inset->getOptions();
-       updateCitekeys(inset->getContents());
-       show();
+       fl_clear_choice(dialog->choice_style);
+       fl_addto_choice(dialog->choice_style, str.c_str());
+       setEnabled(dialog->choice_style, !noVec);
 }
 
 
-void FormCitation::createInset( string const & arg )
+void updateStyle(FD_citation * dialog, string command)
 {
-       if( dialogIsOpen ) return;
-
-       string keys;
-       if (contains(arg, "|")) {
-               keys = token(arg, '|', 0);
-               textAfter = token(arg, '|', 1);
-       } else {
-               keys = arg;
-               textAfter.erase();
-       }
-
-       updateCitekeys(keys);
-       show();
+       // 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::hideInset(InsetCitation * ti)
-{
-       if (inset_ == ti) {
-               inset_ = 0;
-               hide();
-       }
-}
 
+typedef FormController<ControlCitation, FormView<FD_citation> > base_class;
 
-void FormCitation::show()
-{
-       if (!dialog_) {
-               build();
-               fl_set_form_atclose(dialog_->form_citation,
-                                   C_FormCitationWMHideCB, 0);
-       }
 
-       update();  // make sure its up-to-date
-
-       dialogIsOpen = true;
-       if (dialog_->form_citation->visible) {
-               fl_raise_form(dialog_->form_citation);
-       } else {
-               fl_show_form(dialog_->form_citation,
-                            FL_PLACE_MOUSE | FL_FREE_SIZE,
-                            FL_TRANSIENT,
-                            _("Citation"));
-               u_ = d_->updateBufferDependent.
-                        connect(slot(this, &FormCitation::update));
-               h_ = d_->hideBufferDependent.
-                        connect(slot(this, &FormCitation::hide));
-       }
-}
+FormCitation::FormCitation(Dialog & parent)
+       : base_class(parent, _("Citation"))
+{}
 
 
-void FormCitation::update()
+void FormCitation::apply()
 {
-       bibkeys.clear();
-       bibkeysInfo.clear();
+       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);
+       }
 
-       vector<pair<string,string> > blist =
-               lv_->buffer()->getBibkeyList();
+       controller().params().setCmdName(command);
+       controller().params().setContents(getStringFromVector(citekeys));
 
-       for( unsigned int i = 0; i < blist.size(); ++i ) {
-               bibkeys.push_back(blist[i].first);
-               bibkeysInfo.push_back(blist[i].second);
-       }
+       string const before  = getString(dialog_->input_before);
+       controller().params().setSecOptions(before);
+       
+       string const after  = getString(dialog_->input_after);
+       controller().params().setOptions(after);
+}
 
-       blist.clear();
 
-       fl_freeze_form( dialog_->form_citation );
+void FormCitation::hide()
+{
+       citekeys.clear();
+       bibkeys.clear();
 
-       updateBrowser( dialog_->bibBrsr, bibkeys );
-       updateBrowser( dialog_->citeBrsr, citekeys );
-       fl_clear_browser( dialog_->infoBrsr );
+       FormDialogView::hide();
+}
 
-       // No keys have been selected yet, so...
-       setBibButtons( OFF );
-       setCiteButtons( OFF );
 
-       int noKeys = max( bibkeys.size(), citekeys.size() );
+void FormCitation::build()
+{
+       dialog_.reset(build_citation(this));
+
+       // 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);
+
+       // 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);
+
+       // trigger an input event for cut&paste with middle mouse button.
+       setPrehandler(dialog_->input_search);
+       setPrehandler(dialog_->input_before);
+       setPrehandler(dialog_->input_after);
+
+       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);
+
+       //set up the tooltip mechanism
+       string str = _("Add the selected entry to the current citation reference.");
+       tooltips().init(dialog_->button_add, str);
+
+       str = _("Delete the selected entry from the current citation reference.");
+       tooltips().init(dialog_->button_del, str);
+
+       str = _("Move the selected entry upwards (in the current list).");
+       tooltips().init(dialog_->button_up, str);
+
+       str = _("Move the selected entry downwards (in the current list).");
+       tooltips().init(dialog_->button_down, str);
+
+       str = _("The entries which will be cited. Select them with the arrow buttons from the right browser window.");
+       tooltips().init(dialog_->browser_cite, str);
+#if FL_VERSION == 0 || (FL_REVISION == 0 && FL_FIXLEVEL == 0)
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_cite);
+#endif
 
-       // Place bounds, so that 4 <= noKeys <= 15
-       noKeys = max( 4, min(15, noKeys) );
+       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);
+#if FL_VERSION == 0 || (FL_REVISION == 0 && FL_FIXLEVEL == 0)
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_bib);
+#endif
 
-       // Re-size the form to accommodate the new browser size
-       int size = 20 * noKeys;
-       bool bibPresent = ( bibkeys.size() > 0 );
-       setSize( size, bibPresent );
+       str = _("Information about the selected entry");
+       tooltips().init(dialog_->browser_info, str);
+#if FL_VERSION == 0 || (FL_REVISION == 0 && FL_FIXLEVEL == 0)
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_info);
+#endif
 
-       fl_set_input( dialog_->textAftr, textAfter.c_str() );
+       str = _("Here you may select how the citation label should look inside the text (Natbib).");
+       tooltips().init(dialog_->choice_style, str);
 
-       fl_unfreeze_form( dialog_->form_citation );
-}
+       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);
 
+       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);
 
-void FormCitation::updateCitekeys( string const & keysIn )
-{
-       citekeys.clear();
+       str = _("Optional text which appears before the citation reference, e.g. \"see <Ref>\"");
+       tooltips().init(dialog_->input_before, str);
 
-       string tmp;
-       string keys = keysIn;
-       keys = frontStrip( split(keys, tmp, ',') );
-       while( !tmp.empty() ) {
-               citekeys.push_back( tmp );
-               keys = frontStrip( split(keys, tmp, ',') );
-       }
-}
+       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);
 
-void FormCitation::updateBrowser( FL_OBJECT * browser,
-                                 vector<string> const & keys ) const
-{
-       fl_clear_browser( browser );
+       str = _("Activate if you want to have case sensitive search: \"bibtex\" finds \"bibtex\", but not \"BibTeX\".");
+       tooltips().init(dialog_->check_search_case, 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 enter Regular Expressions.");
+       tooltips().init(dialog_->check_search_type, str);
 }
 
 
-void FormCitation::setBibButtons( State status ) const
+void FormCitation::findBiblio(biblio::Direction const dir)
 {
-       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;
-       }
+       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);
 }
 
 
-void FormCitation::setCiteButtons( State status ) const
+ButtonPolicy::SMInput FormCitation::input(FL_OBJECT * ob, long)
 {
-       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 );
-               }
+       ButtonPolicy::SMInput activate = ButtonPolicy::SMI_NOOP;
 
-               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 );
+       biblio::InfoMap const & theMap = controller().bibkeysInfo();
 
-               fl_deactivate_object( dialog_->upBtn );
-               fl_set_object_lcol( dialog_->upBtn, FL_INACTIVE );
-
-               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 brsrHeight, bool bibPresent ) const
-{
-       int const infoHeight  = 110;
-       int const otherHeight = 140;
-       brsrHeight = max( brsrHeight, 175 );
-       int formHeight = brsrHeight + otherHeight;
-
-       if( bibPresent ) formHeight += infoHeight + 30;
-       fl_set_form_size( dialog_->form_citation, 430, formHeight );
-
-       // No resizing is alowed in the y-direction
-       fl_set_form_minsize( dialog_->form_citation, 430, formHeight );
-       fl_set_form_maxsize( dialog_->form_citation, 1000, formHeight );
-
-       int ypos = 0;
-       fl_set_object_geometry( dialog_->box,      0,   ypos, 430, formHeight );
-       ypos += 30;
-       fl_set_object_geometry( dialog_->citeBrsr, 10,  ypos, 180, brsrHeight );
-       fl_set_object_geometry( dialog_->bibBrsr,  240, ypos, 180, brsrHeight );
-       fl_set_object_geometry( dialog_->addBtn,   200, ypos,  30, 30 );
-       ypos += 35;
-       fl_set_object_geometry( dialog_->delBtn,   200, ypos,  30, 30 );
-       ypos += 35;
-       fl_set_object_geometry( dialog_->upBtn,    200, ypos,  30, 30 );
-       ypos += 35;
-       fl_set_object_geometry( dialog_->downBtn,  200, ypos,  30, 30 );
-
-       ypos = brsrHeight+30; // base of Citation/Bibliography browsers
-
-       // awaiting natbib support
-       fl_hide_object( dialog_->style );
-
-       if( bibPresent ) {
-               ypos += 30;
-               fl_set_object_geometry( dialog_->infoBrsr, 10, ypos, 410, infoHeight );
-               fl_show_object( dialog_->infoBrsr );
-               ypos += infoHeight;
-       }
-       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;
 
-       ypos += 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, ypos);
-//     fl_set_object_geometry( dialog_->textAftr, 100, ypos,   250, 30 );
-       fl_set_object_geometry( dialog_->ok,       230, ypos+50, 90, 30 );
-       fl_set_object_geometry( dialog_->cancel,   330, ypos+50, 90, 30 );
-}
+               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( State cb )
-{
-       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);
 
-       string tmp;
-       for( unsigned int i = 0; i < citekeys.size(); ++i ) {
-               if (i > 0) tmp += ", ";
-               tmp += citekeys[i];
-       }
+       // Ditto for the keys cited in this inset
+       citekeys = getVectorFromString(controller().params().getContents());
+       updateBrowser(dialog_->browser_cite, citekeys);
 
-       textAfter = fl_get_input(dialog_->textAftr);
+       // Use the first citekey to fill choice_style
+       string key;
+       if (!citekeys.empty())
+               key = citekeys[0];
 
-       if( inset_ != 0 )
-       {
-               inset_->setContents( tmp );
-               inset_->setOptions( textAfter );
-               lv_->view()->updateInset( inset_, true );
-       } else {
-               string arg = tmp + '|' + textAfter;
-               lv_->getLyXFunc()->Dispatch( LFUN_INSERT_CITATION, arg.c_str() );
-       }
-}
+       fillChoice(dialog_.get(), controller().getCiteStrings(key));
 
+       // Use the citation command to update the GUI
+       updateStyle(dialog_.get(), controller().params().getCmdName());
 
-void FormCitation::hide()
-{
-       if (dialog_
-           && dialog_->form_citation
-           && dialog_->form_citation->visible) {
-               fl_hide_form(dialog_->form_citation);
-               u_.disconnect();
-               h_.disconnect();
-       }
+       bool const natbib = controller().usingNatbib();
+       setEnabled(dialog_->check_full_author_list, natbib);
+       setEnabled(dialog_->check_force_uppercase, natbib);
+       setEnabled(dialog_->choice_style, natbib || controller().usingJurabib());
+       setEnabled(dialog_->input_before, natbib || controller().usingJurabib());
 
-       // free up the dialog for another inset
-       inset_ = 0;
-       dialogIsOpen = false;
-}
+       // 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.
+       fl_set_input(dialog_->input_after,
+                    controller().params().getOptions().c_str());
 
-void FormCitation::free()
-{
-       // we don't need to delete u and h here because
-       // hide() does that after disconnecting.
-       if (dialog_) {
-               if (dialog_->form_citation
-                   && dialog_->form_citation->visible) {
-                       hide();
-               }
-               fl_free_form(dialog_->form_citation);
-               delete dialog_;
-               dialog_ = 0;
-       }
+       fl_set_input(dialog_->input_before, 
+                    controller().params().getSecOptions().c_str());
 }
 
 
-int FormCitation::WMHideCB(FL_FORM * form, void *)
+void FormCitation::updateBrowser(FL_OBJECT * browser,
+                                vector<string> const & keys) const
 {
-       // Ensure that the signals (u and h) are disconnected even if the
-       // window manager is used to close the dialog.
-       FormCitation * pre = static_cast<FormCitation*>(form->u_vdata);
-       pre->hide();
-       return FL_CANCEL;
-}
+       // 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;
+       }
 
+       // They will be changed. Proceed.
+       fl_clear_browser(browser);
 
-void FormCitation::OKCB(FL_OBJECT * ob, long)
-{
-       FormCitation * pre = static_cast<FormCitation*>(ob->form->u_vdata);
-       pre->apply();
-       pre->hide();
+       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::CancelCB(FL_OBJECT * ob, long)
+void FormCitation::setBibButtons(State status) const
 {
-       FormCitation * pre = static_cast<FormCitation*>(ob->form->u_vdata);
-       pre->hide();
+       setEnabled(dialog_->button_add, (status == ON));
 }
 
 
-void FormCitation::InputCB(FL_OBJECT * ob, long data)
+void FormCitation::setCiteButtons(State status) const
 {
-       FormCitation * pre = static_cast<FormCitation*>(ob->form->u_vdata);
-       pre->input( static_cast<FormCitation::State>(data) );
+       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);
 }