Sorry, got it wrong, missed the special == "none" in the lyx file and the not
is binary instead of logical (too many programing languages in parallel ... )

Anyway this is supposed to be the correct patch (I hope I got the indentation
according to the coding convention).

Index: src/insets/InsetBox.cpp
===================================================================
--- src/insets/InsetBox.cpp     (revision 18229)
+++ src/insets/InsetBox.cpp     (working copy)
@@ -340,16 +340,18 @@
                        os << "\\begin{minipage}";
 
                os << "[" << params_.pos << "]";
-               if (params_.height_special == "none") {
-                       // FIXME UNICODE
-                       os << '[' << from_ascii(params_.height.asLatexString())
-                          << ']';
-               } else {
-                       // Special heights
-                       // FIXME UNICODE
-                       os << "[" << params_.height.value()
-                          << '\\' << from_utf8(params_.height_special)
-                          << ']';
+               if (!params_.height.zero()) {
+                       if (params_.height_special == "none") {
+                               // FIXME UNICODE
+                               os << '[' << 
from_ascii(params_.height.asLatexString())
+                                  << ']';
+                       } else {
+                               // Special heights
+                               // FIXME UNICODE
+                               os << "[" << params_.height.value()
+                                  << '\\' << from_utf8(params_.height_special)
+                                  << ']';
+                       }
                }
                if (params_.inner_pos != params_.pos)
                        os << "[" << params_.inner_pos << "]";


On Mon, 07 May 2007 22:47:06 +0200
Uwe Stöhr <[EMAIL PROTECTED]> wrote:

> Here's a ptch from Macha Feigin:
> 
> Ok, I made a patch, hope that it is the correct solution, if so then it's a
> one liner
> Didn't know where to send it to
> 
> Index: src/insets/InsetBox.cpp
> ===================================================================
> --- src/insets/InsetBox.cpp     (revision 18217)
> +++ src/insets/InsetBox.cpp     (working copy)
> @@ -344,7 +344,7 @@
>                          // FIXME UNICODE
>                          os << '[' <<
> from_ascii(params_.height.asLatexString()) << ']';
> -               } else {
> +               } else if (~params_.height.zero()) {
>                          // Special heights
>                          // FIXME UNICODE
>                          os << "[" << params_.height.value()
> 
> 
> On Mon, 7 May 2007 02:32:57 +0300
> Micha Feigin <[EMAIL PROTECTED]> wrote:
> 
> > I seem to be having a problem with some tex code imported into lyx 1.5
> > It contains an image and only width originally, but lyx insists on inserting
> > height zero which completely messes up the rendering. (image is interleaved
> > with text)
> > 
> > i.e, the following
> > 
> >     \parbox{0.24\textwidth}{
> >       \begin{center}
> >     \includegraphics[width=0.22\textwidth, height=0.33\textwidth]
> >     {eigvec_-0_0009.jpg}\\
> >     $\lambda = -0.0009$
> >       \end{center}
> >     }
> >  
> > is imported as
> > 
> > \parbox[c][0pt]{0.24\textwidth}{%
> > 
> > 
> > \begin{center}
> > \includegraphics[width=0.22\textwidth,height=0.33\textwidth]{images/eigvec_-0_0009}\\
> >  $\lambda=-0.0009$ 
> > \par\end{center}%
> > }%
> > 
> > Am I doing something wrong or is this a bug?
> > 
> > Thanks
> > 
> 

Reply via email to