On Wed, 2005-02-09 at 10:00, Georg Baum wrote:
> Martin Vermeer wrote:
> 
> > You're even righter than you think: these conditions are not needed when
> > you think about it, so I just removed all of them.
> 
> Why are they not needed? I must be blind, but I don't see it. And if they
> are really not needed, then there is IMHO no need for the new methods at
> all, simply call rowinfo_[row].lines_++; etc. in dispatch().

Ah yes of course. Making things needlessly complicated again.
> 
> >> Another question: What about lines above the first row and below the
> >> last? I believe that this is possible in LaTeX, but how to support it
> >> here? I can't think of a better way than having an addHLineBelow() etc.
> > 
> > No, neither can I. But as there is no support for this in the current
> > mathed infrastructure, implementing this would go much further than just
> > fixing a broken UI. For 1.5?
> 
> This is supported, at least for hlines (OK, the display in LyX is broken,
> but LaTeX output is alright). It is not for vlines, but since this is
> possible in LaTeX and mathed reads and writes (a reasonable subset of)
> LaTeX this qualifies IMO rather as bug and not as new feature.
> I'll have a look wether it is easy to fix.

What I meant is that the lines_ parameter in RowInfo only describes
hlines _above_ the row. If you see a simple fix, great.

> >> And getStatus needs to be updated to disable addHLineAbove if the cursor
> >> is in the first row.
> > 
> > How? If I look at getStatus for math_gridinset, I see a lot of stuff
> > commented out. Is this supposed to work at all, and how?
> 
> I don't understand this one:
>                if (v_align_ == '\0') {
>                         flag.enabled(true);
>                         break;
>                 }
> 
> but the rest looks as if it should be reenabled.

> What I meant is something like in MathHullInset: E.g disable add-hline-above
>         if (row == 0 && s == "add-hline-above") {
>                 flag.message(N_("Can't add a hline above the first row"));
>                 flag.enabled(false);
>                 return true;
>         }

Actually I have no problem putting a hline above the first row, or
outputting it to LaTeX. So no need for this condition. (Or a vline left
of the first column.)

> But the problem is that we don't know row here!
> 
> 
> 
> Georg

- Martin

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to