]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/xforms/FormGraphics.C
get rid of ejour2; formgraphics tweaks from Juergen S
[lyx.git] / src / frontends / xforms / FormGraphics.C
index e88aab309b554a4ba338f364f9085a15b07d9df5..f353a3faebabf630637c7af9610651825e10a9d3 100644 (file)
@@ -9,6 +9,7 @@
  *
  * \file FormGraphics.C
  * \author Baruch Even, baruch.even@writeme.com
+ * \author Herbert Voss, voss@perce.de
  */
 
 #include <config.h> 
@@ -21,6 +22,7 @@
 #include "ControlGraphics.h"
 #include "FormGraphics.h"
 #include "form_graphics.h"
+#include "Alert.h"
 
 #include "xforms_helpers.h"
 #include "input_validators.h"
@@ -41,6 +43,7 @@ int const WIDTH_MAXDIGITS = 10;
 int const HEIGHT_MAXDIGITS = 10;
 int const ROTATE_MAXCHARS = 4;
 int const FILENAME_MAXCHARS = 1024;
+string defaultUnit = "cm"; 
  
 } // namespace anon
 
@@ -48,17 +51,16 @@ int const FILENAME_MAXCHARS = 1024;
 typedef FormCB<ControlGraphics, FormDB<FD_form_graphics> > base_class;
 
 FormGraphics::FormGraphics(ControlGraphics & c)
-       : base_class(c, _("Graphics"))
+       : base_class(c, _("Graphics"), false)
 {}
 
 
 void FormGraphics::redraw()
 {
-       if(form() && form()->visible)
+       if (form() && form()->visible)
                fl_redraw_form(form());
        else
                return;
-
        FL_FORM * outer_form = fl_get_active_folder(dialog_->tabFolder);
        if (outer_form && outer_form->visible)
                fl_redraw_form(outer_form);
@@ -71,7 +73,7 @@ void FormGraphics::build()
 
         // Manage the ok, apply, restore and cancel/close buttons
        bc().setOK(dialog_->button_ok);
-//     bc().setApply(dialog_->button_apply);
+       bc().setApply(dialog_->button_apply);
        bc().setCancel(dialog_->button_cancel);
        bc().setRestore(dialog_->button_restore);
 
@@ -80,20 +82,39 @@ void FormGraphics::build()
 
        fl_set_input_return (file_->input_filename, FL_RETURN_CHANGED);
        fl_set_input_return (file_->input_subcaption, FL_RETURN_CHANGED);
-       fl_set_input_return (file_->input_lyxwidth, FL_RETURN_CHANGED);
-       fl_set_input_return (file_->input_lyxheight, FL_RETURN_CHANGED);
+       fl_set_input_return (file_->input_rotate_angle, FL_RETURN_CHANGED);
        fl_set_input_maxchars(file_->input_filename,  FILENAME_MAXCHARS);
+       fl_set_input_maxchars(file_->input_rotate_angle, ROTATE_MAXCHARS);
+       fl_set_input_filter(file_->input_rotate_angle, fl_float_filter);
 
-       fl_addto_choice(file_->choice_width_lyxwidth, choice_Length_All.c_str());
-       fl_addto_choice(file_->choice_width_lyxheight, choice_Length_All.c_str());
-       fl_addto_choice(file_->choice_display,
-               _(" Default | Monochrome | Grayscale | Color | Do not display "));
-       //FIXME: This does not work (always set to monochrome by default. Why? JSpitzm)
-       fl_set_choice(file_->choice_display, 1);
+       string const choice_origin = 
+           "center|"                                   // c
+           "leftTop|leftBottom|leftBaseline|"          // lt lb lB
+           "centerTop|centerBottom|centerBaseline|"    // ct cb cB
+           "rightTop|rightBottom|rightBaseline";       // rt rb rB
+       fl_addto_choice(file_->choice_origin, choice_origin.c_str());
 
        bc().addReadOnly(file_->button_browse);
        bc().addReadOnly(file_->check_subcaption);
-       bc().addReadOnly(file_->choice_display);
+       bc().addReadOnly(file_->check_rotate);
+       bc().addReadOnly(file_->button_draft);
+       bc().addReadOnly(file_->button_nounzip);
+
+       // the lyxview section
+       lyxview_.reset(build_lyxview());
+
+       fl_set_input_return (lyxview_->input_lyxwidth, FL_RETURN_CHANGED);
+       fl_set_input_return (lyxview_->input_lyxheight, FL_RETURN_CHANGED);
+       fl_set_input_return (lyxview_->input_lyxscale, FL_RETURN_CHANGED);
+
+       fl_addto_choice(lyxview_->choice_width_lyxwidth, choice_Length_WithUnit.c_str());
+       fl_addto_choice(lyxview_->choice_width_lyxheight, choice_Length_WithUnit.c_str());
+
+       bc().addReadOnly(lyxview_->radio_pref);
+       bc().addReadOnly(lyxview_->radio_mono);
+       bc().addReadOnly(lyxview_->radio_gray);
+       bc().addReadOnly(lyxview_->radio_color);
+       bc().addReadOnly(lyxview_->radio_nodisplay);
 
        // the size section
        size_.reset(build_size());
@@ -111,7 +132,7 @@ void FormGraphics::build()
        fl_addto_choice(size_->choice_width_units, choice_Length_All.c_str());
        fl_addto_choice(size_->choice_height_units, choice_Length_All.c_str());
 
-       bc().addReadOnly(size_->button_default);
+       bc().addReadOnly(size_->button_asis);
        bc().addReadOnly(size_->button_wh);
        bc().addReadOnly(size_->button_scale);
        bc().addReadOnly(size_->check_aspectratio);
@@ -123,38 +144,24 @@ void FormGraphics::build()
        fl_set_input_return (bbox_->input_bb_x1, FL_RETURN_CHANGED);
        fl_set_input_return (bbox_->input_bb_y1, FL_RETURN_CHANGED);
 
-       bc().addReadOnly(bbox_->button_getBB);
-       bc().addReadOnly(bbox_->button_clip);
-       bc().addReadOnly(bbox_->button_draft);
+       fl_set_input_filter(bbox_->input_bb_x0,  fl_unsigned_float_filter);
+       fl_set_input_filter(bbox_->input_bb_y0,  fl_unsigned_float_filter);
+       fl_set_input_filter(bbox_->input_bb_x1,  fl_unsigned_float_filter);
 
        string const bb_units = "pt|cm|in";
-       fl_addto_choice(bbox_->choice_bb_x0, bb_units.c_str());
-       fl_addto_choice(bbox_->choice_bb_y0, bb_units.c_str());
-       fl_addto_choice(bbox_->choice_bb_x1, bb_units.c_str());
-       fl_addto_choice(bbox_->choice_bb_y1, bb_units.c_str());
-
+       fl_addto_choice(bbox_->choice_bb_units, bb_units.c_str());
+       bc().addReadOnly(bbox_->button_getBB);
+       bc().addReadOnly(bbox_->button_clip);
 
        // the rotate section
        special_.reset(build_special());
 
-       fl_set_input_return (special_->input_rotate_angle, FL_RETURN_CHANGED);
-
-       string const choice_origin = 
-           "default|"                                  // not important
-           "leftTop|leftBottom|leftBaseline|"          // lt lb lB
-           "center|"                                   // c
-           "centerTop|centerBottom|centerBaseline|"    // ct cb cB
-           "rightTop|rightBottom|rightBaseline|"       // rt rb rB
-           "referencePoint";                           // special
-       fl_addto_choice(special_->choice_origin, choice_origin.c_str());
-
        fl_set_input_return (special_->input_special, FL_RETURN_CHANGED);
-       fl_set_input_maxchars(special_->input_rotate_angle, ROTATE_MAXCHARS);
-       fl_set_input_filter(special_->input_rotate_angle, fl_float_filter);
 
        // add the different tabfolders
        fl_addto_tabfolder(dialog_->tabFolder, _("File"), file_->form);
-       fl_addto_tabfolder(dialog_->tabFolder, _("Size"), size_->form);
+       fl_addto_tabfolder(dialog_->tabFolder, _("LyX View"), lyxview_->form);
+       fl_addto_tabfolder(dialog_->tabFolder, _("LaTeX Size"), size_->form);
        fl_addto_tabfolder(dialog_->tabFolder, _("Bounding Box"), bbox_->form);
        fl_addto_tabfolder(dialog_->tabFolder, _("Extras"), special_->form);
 }
@@ -164,7 +171,67 @@ void FormGraphics::apply()
 {
        // Create the parameters structure and fill the data from the dialog.
        InsetGraphicsParams & igp = controller().params();
+
+       // the file section
        igp.filename = getStringFromInput(file_->input_filename);
+       igp.subcaption = fl_get_button(file_->check_subcaption);
+       igp.subcaptionText = getStringFromInput(file_->input_subcaption);
+       igp.rotate = fl_get_button(file_->check_rotate);
+       igp.rotateAngle =
+               strToDbl(getStringFromInput(file_->input_rotate_angle));
+       while (igp.rotateAngle < 0.0 || igp.rotateAngle > 360.0) {
+               if (igp.rotateAngle < 0.0) {
+                       igp.rotateAngle += 360.0;
+               } else if (igp.rotateAngle > 360.0) {
+                       igp.rotateAngle -= 360.0;
+               }
+       }
+       if (fl_get_choice(file_->choice_origin) > 0)
+           igp.rotateOrigin = fl_get_choice_text(file_->choice_origin);
+       else
+           igp.rotateOrigin = string();
+       igp.draft = fl_get_button(file_->button_draft);
+       igp.noUnzip = fl_get_button(file_->button_nounzip);
+
+       // the lyxview section
+       if (fl_get_button(lyxview_->radio_pref))
+           igp.display = InsetGraphicsParams::DEFAULT;
+       else if (fl_get_button(lyxview_->radio_mono))
+           igp.display = InsetGraphicsParams::MONOCHROME;
+       else if (fl_get_button(lyxview_->radio_gray))
+           igp.display = InsetGraphicsParams::GRAYSCALE;
+       else if (fl_get_button(lyxview_->radio_color))
+           igp.display = InsetGraphicsParams::COLOR;
+       else if (fl_get_button(lyxview_->radio_nodisplay))
+           igp.display = InsetGraphicsParams::NONE;
+
+       if (fl_get_button(lyxview_->button_lyxasis))
+           igp.lyxsize_type = InsetGraphicsParams::DEFAULT_SIZE;
+       else if (fl_get_button(lyxview_->button_lyxwh))
+           igp.lyxsize_type = InsetGraphicsParams::WH;
+       else
+           igp.lyxsize_type = InsetGraphicsParams::SCALE;
+       igp.lyxwidth = LyXLength(getLengthFromWidgets(lyxview_->input_lyxwidth,
+               lyxview_->choice_width_lyxwidth));
+       igp.lyxheight = LyXLength(getLengthFromWidgets(lyxview_->input_lyxheight,
+               lyxview_->choice_width_lyxheight));
+       igp.lyxscale = strToInt(getStringFromInput(lyxview_->input_lyxscale));
+
+       // the size section
+       if (fl_get_button(size_->button_asis))
+           igp.size_type = InsetGraphicsParams::DEFAULT_SIZE;
+       else if (fl_get_button(size_->button_wh))
+           igp.size_type = InsetGraphicsParams::WH;
+       else
+           igp.size_type = InsetGraphicsParams::SCALE;
+       igp.width = LyXLength(getLengthFromWidgets(size_->input_width,
+               size_->choice_width_units));
+       igp.height = LyXLength(getLengthFromWidgets(size_->input_height,
+               size_->choice_height_units));
+       igp.scale = strToInt(getStringFromInput(size_->input_scale));
+       igp.keepAspectRatio = fl_get_button(size_->check_aspectratio);
+
+       // the bb section
        if (!controller().bbChanged)            // different to the original one?
            igp.bb = string();                  // don't write anything     
        else {
@@ -173,149 +240,112 @@ void FormGraphics::apply()
                bb = "0 ";
            else
                bb = getLengthFromWidgets(bbox_->input_bb_x0,
-                       bbox_->choice_bb_x0)+" ";
+                       bbox_->choice_bb_units)+" ";
            if (getStringFromInput(bbox_->input_bb_y0).empty())
                bb += "0 ";
            else
                bb += (getLengthFromWidgets(bbox_->input_bb_y0,
-                       bbox_->choice_bb_y0)+" ");
+                       bbox_->choice_bb_units)+" ");
            if (getStringFromInput(bbox_->input_bb_x1).empty())
                bb += "0 ";
            else
                bb += (getLengthFromWidgets(bbox_->input_bb_x1,
-                       bbox_->choice_bb_x1)+" ");
+                       bbox_->choice_bb_units)+" ");
            if (getStringFromInput(bbox_->input_bb_y1).empty())
                bb += "0 ";
            else
                bb += (getLengthFromWidgets(bbox_->input_bb_y1,
-                       bbox_->choice_bb_y1)+" ");
+                       bbox_->choice_bb_units)+" ");
            igp.bb = bb;
        }
-       igp.draft = fl_get_button(bbox_->button_draft);
        igp.clip = fl_get_button(bbox_->button_clip);
-       igp.subcaption = fl_get_button(file_->check_subcaption);
-       igp.subcaptionText = getStringFromInput(file_->input_subcaption);
-       // use preferences settings if choice is set to default
-       if (fl_get_choice(file_->choice_display) == 1) {
-               if (lyxrc.display_graphics == "mono")
-                       igp.display = InsetGraphicsParams::MONOCHROME;
-               else if (lyxrc.display_graphics == "gray")
-                       igp.display = InsetGraphicsParams::GRAYSCALE;
-               else if (lyxrc.display_graphics == "color")
-                       igp.display = InsetGraphicsParams::COLOR;
-               else if (lyxrc.display_graphics == "no")
-                       igp.display = InsetGraphicsParams::NONE;
-       } else if (fl_get_choice(file_->choice_display) == 2) {
-               igp.display = InsetGraphicsParams::MONOCHROME;
-       } else if (fl_get_choice(file_->choice_display) == 3) {
-               igp.display = InsetGraphicsParams::GRAYSCALE;
-       } else if (fl_get_choice(file_->choice_display) == 4) {
-               igp.display = InsetGraphicsParams::COLOR;
-       } else if (fl_get_choice(file_->choice_display) == 5) {
-               igp.display = InsetGraphicsParams::NONE;
-       }
-       if (fl_get_button(size_->button_default))
-           igp.size_type = InsetGraphicsParams::DEFAULT_SIZE;
-       else if (fl_get_button(size_->button_wh))
-           igp.size_type = InsetGraphicsParams::WH;
-       else
-           igp.size_type = InsetGraphicsParams::SCALE;
-       igp.width = LyXLength(getLengthFromWidgets(size_->input_width,
-               size_->choice_width_units));
-       igp.height = LyXLength(getLengthFromWidgets(size_->input_height,
-               size_->choice_height_units));
-       igp.scale = strToInt(getStringFromInput(size_->input_scale));
-       igp.keepAspectRatio = fl_get_button(size_->check_aspectratio);
-       igp.lyxwidth = LyXLength(getLengthFromWidgets(file_->input_lyxwidth,
-               file_->choice_width_lyxwidth));
-       igp.lyxheight = LyXLength(getLengthFromWidgets(file_->input_lyxheight,
-               file_->choice_width_lyxheight));
 
-       igp.rotateAngle =
-               strToDbl(getStringFromInput(special_->input_rotate_angle));
-       while (igp.rotateAngle < 0.0 || igp.rotateAngle > 360.0) {
-               if (igp.rotateAngle < 0.0) {
-                       igp.rotateAngle += 360.0;
-               } else if (igp.rotateAngle > 360.0) {
-                       igp.rotateAngle -= 360.0;
-               }
-       }
-       if (fl_get_choice(special_->choice_origin) > 0)
-           igp.rotateOrigin = fl_get_choice_text(special_->choice_origin);
-       else
-           igp.rotateOrigin = string();
+       // the special section
        igp.special = getStringFromInput(special_->input_special);
-       igp.testInvariant();
 }
 
 
-void FormGraphics::update()
-{      string unit = "cm";
+void FormGraphics::update() {  
        if (lyxrc.default_papersize < 3)
-           unit = "in";
-       string const defaultUnit = string(unit); 
+           defaultUnit = string("in"); 
        // Update dialog with details from inset
        InsetGraphicsParams & igp = controller().params();
+
+       // the file section
        fl_set_input(file_->input_filename, igp.filename.c_str());
-       // set the bounding box values, if exists. First we need the whole
-       // path, because the controller knows nothing about the doc-dir
-       lyxerr << "GraphicsUpdate::BoundingBox = " << igp.bb << "\n";
-       controller().bbChanged = false;
-       if (igp.bb.empty()) {
-           string const fileWithAbsPath = MakeAbsPath(igp.filename, OnlyPath(igp.filename));   
-           string bb = controller().readBB(fileWithAbsPath);
-           lyxerr << "file::BoundingBox = " << bb << "\n";
-           if (!bb.empty()) {          
-               // get the values from the file
-               // in this case we always have the point-unit
-               fl_set_input(bbox_->input_bb_x0, token(bb,' ',0).c_str());
-               fl_set_input(bbox_->input_bb_y0, token(bb,' ',1).c_str());
-               fl_set_input(bbox_->input_bb_x1, token(bb,' ',2).c_str());
-               fl_set_input(bbox_->input_bb_y1, token(bb,' ',3).c_str());
-           }
-       } else {                                // get the values from the inset
-           controller().bbChanged = true;
-           LyXLength anyLength;
-           anyLength = LyXLength(token(igp.bb,' ',0));
-           updateWidgetsFromLength(bbox_->input_bb_x0,
-                       bbox_->choice_bb_x0,anyLength,"pt");
-           anyLength = LyXLength(token(igp.bb,' ',1));
-           updateWidgetsFromLength(bbox_->input_bb_y0,
-                       bbox_->choice_bb_y0,anyLength,"pt");
-           anyLength = LyXLength(token(igp.bb,' ',2));
-           updateWidgetsFromLength(bbox_->input_bb_x1,
-                       bbox_->choice_bb_x1,anyLength,"pt");
-           anyLength = LyXLength(token(igp.bb,' ',3));
-           updateWidgetsFromLength(bbox_->input_bb_y1,
-                       bbox_->choice_bb_y1,anyLength,"pt");
-       }
-       // Update the draft and clip mode
-       fl_set_button(bbox_->button_draft, igp.draft);
-       fl_set_button(bbox_->button_clip, igp.clip);
-       // Update the subcaption check button and input field
        fl_set_button(file_->check_subcaption, igp.subcaption);
        fl_set_input(file_->input_subcaption, igp.subcaptionText.c_str());
        setEnabled(file_->input_subcaption,
                   fl_get_button(file_->check_subcaption));
-
+       fl_set_button(file_->check_rotate, igp.rotate);
+       fl_set_input(file_->input_rotate_angle,
+                    tostr(igp.rotateAngle).c_str());
+       if (igp.rotateOrigin.empty())
+           fl_set_choice(file_->choice_origin,0);
+       else
+           fl_set_choice_text(file_->choice_origin,igp.rotateOrigin.c_str());
+       setEnabled(file_->input_rotate_angle,
+                  fl_get_button(file_->check_rotate));
+       setEnabled(file_->choice_origin,
+                  fl_get_button(file_->check_rotate));
+       fl_set_button(file_->button_draft, igp.draft);
+       fl_set_button(file_->button_nounzip, igp.noUnzip);
+
+       // the lyxview section
        switch (igp.display) {
-           case InsetGraphicsParams::NONE: {   // dont't display
-               fl_set_choice(file_->choice_display, 5);
+       case InsetGraphicsParams::DEFAULT:
+               fl_set_button(lyxview_->radio_pref, 1);
                break;
-           }
-           case InsetGraphicsParams::MONOCHROME: {
-               fl_set_choice(file_->choice_display, 2);
+       case InsetGraphicsParams::MONOCHROME:
+               fl_set_button(lyxview_->radio_mono, 1);
+               break;
+       case InsetGraphicsParams::GRAYSCALE:
+               fl_set_button(lyxview_->radio_gray, 1);
+               break;
+       case InsetGraphicsParams::COLOR:
+               fl_set_button(lyxview_->radio_color, 1);
+               break;
+       case InsetGraphicsParams::NONE:
+               fl_set_button(lyxview_->radio_nodisplay, 1);
+               break;
+       }
+       updateWidgetsFromLength(lyxview_->input_lyxwidth,
+               lyxview_->choice_width_lyxwidth, igp.lyxwidth,defaultUnit);
+       updateWidgetsFromLength(lyxview_->input_lyxheight,
+               lyxview_->choice_width_lyxheight, igp.lyxheight,defaultUnit);
+       fl_set_input(lyxview_->input_lyxscale, tostr(igp.lyxscale).c_str());
+       switch (igp.lyxsize_type) {
+           case InsetGraphicsParams::DEFAULT_SIZE: {
+               fl_set_button(lyxview_->button_lyxasis,1);
+               setEnabled(lyxview_->input_lyxwidth, 0);
+               setEnabled(lyxview_->choice_width_lyxwidth, 0);
+               setEnabled(lyxview_->input_lyxheight, 0);
+               setEnabled(lyxview_->choice_width_lyxheight, 0);
+               setEnabled(lyxview_->input_lyxscale, 0);
                break;
            }
-           case InsetGraphicsParams::GRAYSCALE: {
-               fl_set_choice(file_->choice_display, 3);
+           case InsetGraphicsParams::WH: {
+               fl_set_button(lyxview_->button_lyxwh, 1);
+               setEnabled(lyxview_->input_lyxwidth, 1);
+               setEnabled(lyxview_->choice_width_lyxwidth, 1);
+               setEnabled(lyxview_->input_lyxheight, 1);
+               setEnabled(lyxview_->choice_width_lyxheight, 1);
+               setEnabled(lyxview_->input_lyxscale, 0);
                break;
            }
-           case InsetGraphicsParams::COLOR: {
-               fl_set_choice(file_->choice_display, 4);
+           case InsetGraphicsParams::SCALE: {
+               fl_set_button(lyxview_->button_lyxscale, 1);
+               setEnabled(lyxview_->input_lyxwidth, 0);
+               setEnabled(lyxview_->choice_width_lyxwidth, 0);
+               setEnabled(lyxview_->input_lyxheight, 0);
+               setEnabled(lyxview_->choice_width_lyxheight, 0);
+               setEnabled(lyxview_->input_lyxscale, 1);
                break;
            }
        }
+
+       // the size section
+       // Update the draft and clip mode
        updateWidgetsFromLength(size_->input_width,
                size_->choice_width_units,igp.width,defaultUnit);
        updateWidgetsFromLength(size_->input_height,
@@ -323,11 +353,12 @@ void FormGraphics::update()
        fl_set_input(size_->input_scale, tostr(igp.scale).c_str());
        switch (igp.size_type) {
            case InsetGraphicsParams::DEFAULT_SIZE: {
-               fl_set_button(size_->button_default,1);
+               fl_set_button(size_->button_asis,1);
                setEnabled(size_->input_width, 0);
                setEnabled(size_->choice_width_units, 0);
                setEnabled(size_->input_height, 0);
                setEnabled(size_->choice_height_units, 0);
+               setEnabled(size_->check_aspectratio, 0);
                setEnabled(size_->input_scale, 0);
                break;
            }
@@ -337,6 +368,7 @@ void FormGraphics::update()
                setEnabled(size_->choice_width_units, 1);
                setEnabled(size_->input_height, 1);
                setEnabled(size_->choice_height_units, 1);
+               setEnabled(size_->check_aspectratio, 1);
                setEnabled(size_->input_scale, 0);
                break;
            }
@@ -346,23 +378,56 @@ void FormGraphics::update()
                setEnabled(size_->choice_width_units, 0);
                setEnabled(size_->input_height, 0);
                setEnabled(size_->choice_height_units, 0);
+               setEnabled(size_->check_aspectratio, 0);
                setEnabled(size_->input_scale, 1);
                break;
            }
        }
        fl_set_button(size_->check_aspectratio, igp.keepAspectRatio);
-       // now the lyx-internally viewsize
-       updateWidgetsFromLength(file_->input_lyxwidth,
-               file_->choice_width_lyxwidth, igp.lyxwidth,defaultUnit);
-       updateWidgetsFromLength(file_->input_lyxheight,
-               file_->choice_width_lyxheight, igp.lyxheight,defaultUnit);
-       // Update the rotate angle and special commands
-       fl_set_input(special_->input_rotate_angle,
-                    tostr(igp.rotateAngle).c_str());
-       if (igp.rotateOrigin.empty())
-           fl_set_choice(special_->choice_origin,0);
-       else
-           fl_set_choice_text(special_->choice_origin,igp.rotateOrigin.c_str());
+       
+       // the bb section               
+       // set the bounding box values, if exists. First we need the whole
+       // path, because the controller knows nothing about the doc-dir
+       controller().bbChanged = false;
+       if (igp.bb.empty()) {
+       lyxerr[Debug::GRAPHICS] << "update:: no BoundingBox" << endl;
+           string const fileWithAbsPath = 
+               MakeAbsPath(igp.filename, OnlyPath(igp.filename));      
+           string const bb = controller().readBB(fileWithAbsPath);         
+           if (!bb.empty()) {          
+               // get the values from the file
+               // in this case we always have the point-unit
+               fl_set_input(bbox_->input_bb_x0, token(bb,' ',0).c_str());
+               fl_set_input(bbox_->input_bb_y0, token(bb,' ',1).c_str());
+               fl_set_input(bbox_->input_bb_x1, token(bb,' ',2).c_str());
+               fl_set_input(bbox_->input_bb_y1, token(bb,' ',3).c_str());
+           } else {            // no bb from file
+                   fl_set_input(bbox_->input_bb_x0, bb.c_str());
+                   fl_set_input(bbox_->input_bb_y0, bb.c_str());
+                   fl_set_input(bbox_->input_bb_x1, bb.c_str());
+                   fl_set_input(bbox_->input_bb_y1, bb.c_str());
+           }
+           fl_set_choice(bbox_->choice_bb_units, 1);   // "pt"
+       } else {                                // get the values from the inset
+           lyxerr[Debug::GRAPHICS] << "update:: igp has BoundingBox" << endl;
+           controller().bbChanged = true;
+           LyXLength anyLength;
+           anyLength = LyXLength(token(igp.bb,' ',0));
+           updateWidgetsFromLength(bbox_->input_bb_x0,
+                       bbox_->choice_bb_units,anyLength,"pt");
+           anyLength = LyXLength(token(igp.bb,' ',1));
+           updateWidgetsFromLength(bbox_->input_bb_y0,
+                       bbox_->choice_bb_units,anyLength,"pt");
+           anyLength = LyXLength(token(igp.bb,' ',2));
+           updateWidgetsFromLength(bbox_->input_bb_x1,
+                       bbox_->choice_bb_units,anyLength,"pt");
+           anyLength = LyXLength(token(igp.bb,' ',3));
+           updateWidgetsFromLength(bbox_->input_bb_y1,
+                       bbox_->choice_bb_units,anyLength,"pt");
+       }
+       fl_set_button(bbox_->button_clip, igp.clip);
+
+       // the special section
        fl_set_input(special_->input_special, igp.special.c_str());
 }
 
@@ -381,6 +446,7 @@ bool isValid(FL_OBJECT * ob)
        
 ButtonPolicy::SMInput FormGraphics::input(FL_OBJECT * ob, long)
 {
+       // the file section
        if (ob == file_->button_browse) {
                // Get the filename from the dialog
                string const in_name = getStringFromInput(file_->input_filename);
@@ -388,49 +454,163 @@ ButtonPolicy::SMInput FormGraphics::input(FL_OBJECT * ob, long)
                if (out_name != in_name && !out_name.empty()) {
                        fl_set_input(file_->input_filename, out_name.c_str());
                }
+       } else if (ob == file_->check_subcaption) {
+               setEnabled(file_->input_subcaption,
+                          fl_get_button(file_->check_subcaption));
+       } else if (ob == file_->check_rotate) {
+               setEnabled(file_->input_rotate_angle,
+                          fl_get_button(file_->check_rotate));
+               setEnabled(file_->choice_origin,
+                          fl_get_button(file_->check_rotate));
+
+       // the lyxview section
+       } else if (ob == lyxview_->button_lyxasis) {
+               setEnabled(lyxview_->input_lyxwidth, 0);
+               setEnabled(lyxview_->choice_width_lyxwidth, 0);
+               setEnabled(lyxview_->input_lyxheight, 0);
+               setEnabled(lyxview_->choice_width_lyxheight, 0);
+               setEnabled(lyxview_->input_lyxscale, 0);
+       } else if (ob == lyxview_->button_lyxwh) {
+               setEnabled(lyxview_->input_lyxwidth, 1);
+               setEnabled(lyxview_->choice_width_lyxwidth, 1);
+               setEnabled(lyxview_->input_lyxheight, 1);
+               setEnabled(lyxview_->choice_width_lyxheight, 1);
+               setEnabled(lyxview_->input_lyxscale, 0);
+       } else if (ob == lyxview_->button_lyxscale) {
+               setEnabled(lyxview_->input_lyxwidth, 0);
+               setEnabled(lyxview_->choice_width_lyxwidth, 0);
+               setEnabled(lyxview_->input_lyxheight, 0);
+               setEnabled(lyxview_->choice_width_lyxheight, 0);
+               setEnabled(lyxview_->input_lyxscale, 1);
+       } else if (ob == lyxview_->button_latex_values) {
+               if (fl_get_choice(size_->choice_width_units) > 3
+                       && fl_get_choice(size_->choice_width_units) < 8
+                       || fl_get_choice(size_->choice_height_units) > 3
+                       && fl_get_choice(size_->choice_height_units) < 8)
+                               Alert::alert(_("Warning!"),
+                               _("The units t%, p%, c% and l% are not allowed here."),
+                               _("Cannot use the values from LaTeX size!"));
+               else {
+                       LyXLength dummy = LyXLength(getLengthFromWidgets(size_->input_width,
+                               size_->choice_width_units));
+                       updateWidgetsFromLength(lyxview_->input_lyxwidth,
+                               lyxview_->choice_width_lyxwidth, dummy, defaultUnit);
+                       dummy = LyXLength(getLengthFromWidgets(size_->input_height,
+                               size_->choice_height_units));
+                       updateWidgetsFromLength(lyxview_->input_lyxheight,
+                               lyxview_->choice_width_lyxheight, dummy, defaultUnit);
+                       string const scale = getStringFromInput(size_->input_scale);
+                       fl_set_input(lyxview_->input_lyxscale, scale.c_str());
+                       if (fl_get_button (size_->button_asis) == 1) {
+                               fl_set_button (lyxview_->button_lyxasis, 1);
+                               setEnabled(lyxview_->input_lyxwidth, 0);
+                               setEnabled(lyxview_->choice_width_lyxwidth, 0);
+                               setEnabled(lyxview_->input_lyxheight, 0);
+                               setEnabled(lyxview_->choice_width_lyxheight, 0);
+                               setEnabled(lyxview_->input_lyxscale, 0);
+                       } else if (fl_get_button (size_->button_wh) == 1) {
+                               fl_set_button (lyxview_->button_lyxwh, 1);
+                               setEnabled(lyxview_->input_lyxwidth, 1);
+                               setEnabled(lyxview_->choice_width_lyxwidth, 1);
+                               setEnabled(lyxview_->input_lyxheight, 1);
+                               setEnabled(lyxview_->choice_width_lyxheight, 1);
+                               setEnabled(lyxview_->input_lyxscale, 0);
+                       } else if (fl_get_button (size_->button_scale) ==1) {
+                               fl_set_button (lyxview_->button_lyxscale, 1);
+                               setEnabled(lyxview_->input_lyxwidth, 0);
+                               setEnabled(lyxview_->choice_width_lyxwidth, 0);
+                               setEnabled(lyxview_->input_lyxheight, 0);
+                               setEnabled(lyxview_->choice_width_lyxheight, 0);
+                               setEnabled(lyxview_->input_lyxscale, 1);
+                       }
+               }
+
+       // the bb section
        } else if (!controller().bbChanged && 
-                   ((ob == bbox_->input_bb_x0) || (ob == bbox_->input_bb_y0) ||
-                   (ob == bbox_->input_bb_x1) || (ob == bbox_->input_bb_y1) ||
-                   (ob == bbox_->choice_bb_x0) || (ob == bbox_->choice_bb_y0) ||
-                   (ob == bbox_->choice_bb_x1) || (ob == bbox_->choice_bb_y1))) {
+                 (ob == bbox_->choice_bb_units ||
+                   ob == bbox_->input_bb_x0 || ob == bbox_->input_bb_y0 ||
+                  ob == bbox_->input_bb_x1 || ob == bbox_->input_bb_y1)) {
            controller().bbChanged = true; 
-       } else if (ob == size_->button_default) {
+       } else if (ob == bbox_->button_getBB) {
+           string const filename = getStringFromInput(file_->input_filename);
+           if (!filename.empty()) {
+               string const fileWithAbsPath = MakeAbsPath(filename, OnlyPath(filename));       
+               string bb = controller().readBB(fileWithAbsPath);
+               if (!bb.empty()) {
+                   fl_set_input(bbox_->input_bb_x0, token(bb,' ',0).c_str());
+                   fl_set_input(bbox_->input_bb_y0, token(bb,' ',1).c_str());
+                   fl_set_input(bbox_->input_bb_x1, token(bb,' ',2).c_str());
+                   fl_set_input(bbox_->input_bb_y1, token(bb,' ',3).c_str());
+                   string const unit("pt");
+                   fl_set_choice_text(bbox_->choice_bb_units, unit.c_str());
+               }
+               controller().bbChanged = false;
+           } else {
+                   fl_set_input(bbox_->input_bb_x0, "");
+                   fl_set_input(bbox_->input_bb_y0, "");
+                   fl_set_input(bbox_->input_bb_x1, "");
+                   fl_set_input(bbox_->input_bb_y1, "");
+                   fl_set_choice_text(bbox_->choice_bb_units, "pt");
+           }
+
+       // the size section
+       } else if (ob == size_->button_asis) {
                setEnabled(size_->input_width, 0);
                setEnabled(size_->choice_width_units, 0);
                setEnabled(size_->input_height, 0);
                setEnabled(size_->choice_height_units, 0);
+               setEnabled(size_->check_aspectratio, 0);
                setEnabled(size_->input_scale, 0);
        } else if (ob == size_->button_wh) {
                setEnabled(size_->input_width, 1);
                setEnabled(size_->choice_width_units, 1);
                setEnabled(size_->input_height, 1);
                setEnabled(size_->choice_height_units, 1);
+               setEnabled(size_->check_aspectratio, 1);
                setEnabled(size_->input_scale, 0);
        } else if (ob == size_->button_scale) {
                setEnabled(size_->input_width, 0);
                setEnabled(size_->choice_width_units, 0);
                setEnabled(size_->input_height, 0);
                setEnabled(size_->choice_height_units, 0);
+               setEnabled(size_->check_aspectratio, 0);
+               setEnabled(size_->input_scale, 1);
+       } else if (ob == size_->button_lyx_values) {
+           LyXLength dummy = LyXLength(getLengthFromWidgets(
+               lyxview_->input_lyxwidth, lyxview_->choice_width_lyxwidth));
+           updateWidgetsFromLength(size_->input_width,
+               size_->choice_width_units, dummy, defaultUnit);
+           dummy = LyXLength(getLengthFromWidgets(lyxview_->input_lyxheight,
+               lyxview_->choice_width_lyxheight));
+           updateWidgetsFromLength(size_->input_height,
+               size_->choice_height_units, dummy, defaultUnit);
+           string const scale = getStringFromInput(lyxview_->input_lyxscale);
+           fl_set_input(size_->input_scale, scale.c_str());
+           if (fl_get_button (lyxview_->button_lyxasis) == 1) {
+               fl_set_button (size_->button_asis, 1);
+               setEnabled(size_->input_width, 0);
+               setEnabled(size_->choice_width_units, 0);
+               setEnabled(size_->input_height, 0);
+               setEnabled(size_->choice_height_units, 0);
+               setEnabled(size_->check_aspectratio, 0);
+               setEnabled(size_->input_scale, 0);
+           } else if (fl_get_button (lyxview_->button_lyxwh) == 1) {
+               fl_set_button (size_->button_wh, 1);
+               setEnabled(size_->input_width, 1);
+               setEnabled(size_->choice_width_units, 1);
+               setEnabled(size_->input_height, 1);
+               setEnabled(size_->choice_height_units, 1);
+               setEnabled(size_->check_aspectratio, 1);
+               setEnabled(size_->input_scale, 0);
+           } else if (fl_get_button (lyxview_->button_lyxscale) ==1) {
+               fl_set_button (size_->button_scale, 1);
+               setEnabled(size_->input_width, 0);
+               setEnabled(size_->choice_width_units, 0);
+               setEnabled(size_->input_height, 0);
+               setEnabled(size_->choice_height_units, 0);
+               setEnabled(size_->check_aspectratio, 0);
                setEnabled(size_->input_scale, 1);
-       } else if (ob == file_->check_subcaption) {
-               setEnabled(file_->input_subcaption,
-                          fl_get_button(file_->check_subcaption));
-       } else if (ob == bbox_->button_getBB) {
-           string const filename = getStringFromInput(file_->input_filename);
-           if (!filename.empty()) {
-               string const fileWithAbsPath = MakeAbsPath(filename, OnlyPath(filename));       
-               string bb = controller().readBB(fileWithAbsPath);
-               lyxerr << "getBB::BoundingBox = " << bb << "\n";
-               if (!bb.empty()) {              
-                   fl_set_input(bbox_->input_bb_x0, token(bb,' ',0).c_str());
-                   fl_set_input(bbox_->input_bb_y0, token(bb,' ',1).c_str());
-                   fl_set_input(bbox_->input_bb_x1, token(bb,' ',2).c_str());
-                   fl_set_input(bbox_->input_bb_y1, token(bb,' ',3).c_str());
-               }
-               controller().bbChanged = false;
            }
-       } else if (ob == dialog_->button_help) {
-           controller().help();
        }
 
        // check if the input is valid
@@ -440,19 +620,15 @@ ButtonPolicy::SMInput FormGraphics::input(FL_OBJECT * ob, long)
        invalid = invalid || !isValid(bbox_->input_bb_y1);
        invalid = invalid || !isValid(size_->input_width);
        invalid = invalid || !isValid(size_->input_height);
-       invalid = invalid || !isValid(file_->input_lyxwidth);
-       invalid = invalid || !isValid(file_->input_lyxheight);
+       invalid = invalid || !isValid(lyxview_->input_lyxwidth);
+       invalid = invalid || !isValid(lyxview_->input_lyxheight);
 
        // deactivate OK/ Apply buttons and
        // spit out warnings if invalid
-       if (ob == bbox_->input_bb_x0
-                       || ob == bbox_->input_bb_x1
-                       || ob == bbox_->input_bb_y0
-                       || ob == bbox_->input_bb_y1
-                       || ob == size_->input_width
-                       || ob == size_->input_height
-                       || ob == file_->input_lyxwidth
-                       || ob == file_->input_lyxheight) {
+       if (ob == bbox_->input_bb_x0 || ob == bbox_->input_bb_x1 || 
+           ob == bbox_->input_bb_y0 || ob == bbox_->input_bb_y1 || 
+           ob == size_->input_width || ob == size_->input_height || 
+           ob == lyxview_->input_lyxwidth || ob == lyxview_->input_lyxheight) {
                if (invalid) {
                        fl_set_object_label(dialog_->text_warning,
                                _("Warning: Invalid Length!"));
@@ -463,25 +639,6 @@ ButtonPolicy::SMInput FormGraphics::input(FL_OBJECT * ob, long)
                }
        }
 
-       return checkInput();
-}
-
-
-ButtonPolicy::SMInput FormGraphics::checkInput()
-{
-       // Put verifications that the dialog shows some sane values,
-       // if not disallow clicking on ok/apply.
-       // Possibly use a label in the bottom of the dialog to give the reason.
-       ButtonPolicy::SMInput activate = ButtonPolicy::SMI_VALID;
-       // We verify now that there is a filename, it exists, it's a file
-       // and it's readable.
-       string filename = getStringFromInput(file_->input_filename);
-       FileInfo file(filename);
-       if (filename.empty() || !file.isOK() || !file.exist()
-                           || !file.isRegular() || !file.readable()
-          )
-               activate = ButtonPolicy::SMI_INVALID;
-
-       return activate;
+       return ButtonPolicy::SMI_VALID;
 }