Abdelrazak Younes wrote: > Alfredo Braunstein wrote: >> Andre Poenitz wrote: >> >>>> It seems pretty much a mess to me. >>> whatEverDraw(x, y) should draw the baseline of the first embedded line >>> on y. Everything else should be changed to adhere to that scheme. >> >> Yes, this is what I was trying to convince Abdel to (I was trying to use >> actual arguments though...) > > OK, I may have gone too far in this. My rationale was that it doesn't > make sense that the ascent of a multirow text is the ascent of the first
Agree, that doesn't make sense. Shouldn't be like this. In particular since multirow cells normally have a button at the top... It *may* make sense that the baseline of the inset is the baseline of the first row though. OTOH, maybe in multirow cells the baseline is probably not used at all (because there is nothing else in the row...) > row. I would have put the baseline in the middle of the text. Of course > when you have only one line your rationale is probably better... > > I don't have time to act on this until saturday so feel free to change > that if you want. I don't have time either unfortunately. I'll be a rather busy (read traveling) in the next 30 days or so... IMHO, (if you agree) we should just revert the full patch series if possible. Another thing I don't like of the recent changes is the fact that InsetCollapsable now accesses the TextMetrics directly, that should be done by InsetText or better yet restrict it to Text. A/
