Stefan Schimanski wrote:
OK, got it from bugzilla.

Sorry, forgot to attach it to the mail as well.

Good detective work! I feel stupid for not seeing that at the first glance :-(
Do you have SVN access?

No, no write access.

OK, I'll commit the patch for you. Please consider asking for it.


There's still some drawing problem though but I think this is orthogonal to this patch: the 'a' and '1' are drawn at the same position. I guess there is some mis-calculation of the x-position somewhere.

In InsetMathMacro::draw, the following lines:

  string t = "#1: ";
  mathed_string_dim(font, name(), ldim);

The latter should take t instead of name(), and add some pixels because of the +3 some lines below for the #1 label.

OK, I'll see what I can do, thanks for the tips.


@edit: I meant the InsetMathNest::edit method which I tried to overwrite in InsetMathMacro. But it is not called when the cursor enters the macro inset. Instead only idxLast and idxFirst are called by the dispatcher method of InsetMathNest which got the key command in my test case.

OK, thanks for the clarification. Andre' should be able to answer your question about why InsetMathNest::edit() is not used. Andre'?

But your patch is good anyway, thanks.

Abdel.

Reply via email to