John Levon wrote:

 > On Wed, Jun 26, 2002 at 12:56:33AM +0100, Jan Warnking wrote:
 >
 >
 >>Basically, when using the "get Latex size" button in the graphics dialog,
 >>a custom Latex Size of 90 % of text height gets converted to a 90cm 
height
 >>for Lyx view.
 >>
 >>The above can be verified inserting a graphic in an empty document (no
 >>image file needed).
 >>
 >
 > Confirmed. Can you open a bugzilla bug on this please ?


attached a patch against cvs, should go into 1.2.1, too

there is still the problem with the hiding gui. John,
can you have a look??
when the alert-window with the message appears and you
click on ok, the open graphics-gui is behind the main lyx-window.

Herbert

-- 
http://www.lyx.org/help/

Index: ChangeLog
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/frontends/xforms/ChangeLog,v
retrieving revision 1.453
diff -u -r1.453 ChangeLog
--- ChangeLog   10 Jul 2002 15:03:25 -0000      1.453
+++ ChangeLog   11 Jul 2002 06:24:25 -0000
@@ -1,3 +1,8 @@
+2002-07-11  Herbert Voss  <[EMAIL PROTECTED]>
+
+       * FormGraphics.C (input): test also the height for %-value, when
+       importing the latex-value into the lyx-view
+
 2002-07-10  Andrew Zabolotny  <[EMAIL PROTECTED]>
 
        * XMiniBuffer.C (peek_event): 
Index: FormGraphics.C
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/frontends/xforms/FormGraphics.C,v
retrieving revision 1.73
diff -u -r1.73 FormGraphics.C
--- FormGraphics.C      2 Jul 2002 19:23:09 -0000       1.73
+++ FormGraphics.C      11 Jul 2002 06:24:26 -0000
@@ -544,9 +544,10 @@
                setEnabled(lyxview_->choice_lyxheight, 0);
                setEnabled(lyxview_->input_lyxscale, 1);
        } else if (ob == lyxview_->button_latex_values) {
-               if (contains(fl_get_choice_text(size_->choice_width),'%'))
+               if (contains(fl_get_choice_text(size_->choice_width),'%') || 
+                       contains(fl_get_choice_text(size_->choice_height),'%'))
                        Alert::alert(_("Warning!"),
-                                    _("The units text%, page%, col% and line% are not 
allowed here."),
+                                    _("The %-units are not allowed here."),
                                     _("Cannot use the values from LaTeX size!"));
                else {
                        LyXLength dummy =

Reply via email to