Uwe Stöhr wrote:
The math delimiter dialog is still buggy. Attached is a patch that fixes bug 3457 and bug 3458:

- the sequence "\\" must not be used as math delimiters, so currently we produce LaTeX-errors
 (bug 3458)
  I fixed this by using \backslash instead

I think you can commit this part.


- the shown TeX-Code of the selected delimiters is wrong (bug 3457). In the fix is a problem I
  cannot solve:
  This line makes the dialog crashing when you call it:

  string stri_left = fromqstr(leftLW->currentItem()->toolTip());

  What is wrong with this?

The row is in the process of being changed so currentItem() is not valid, you should not use it.

Please don't commit this part.


- the "\" charaacter of the cmsy font has whitespace below. As we currently don't use the same font for "/" too, the delimiters are not vertically in sync, see the attached screenshot. I fixed this
  by using the cmsy font for "/" too. Perhaps you have a better idea.

Your idea is fine. I think you can commit this part.

Abdel.

Reply via email to