José Matos wrote:
Hi all,
        feedback is welcome for this message.
[...]
Thanks to Uwe and Bennet I have compiled this list of bugs. Is there any issue missing here? Which bugs do you expect to tackle during the release candidates stage?


I'm still having some stability issues with Hebrew. I pretty sure they're mostly encoding-related, because they only appear with certain encoding options. I think, but am not sure, that solving bug 1820 (which is not trivial) will solve some of the issues. Anyone up to the challenge?

http://bugzilla.lyx.org/show_bug.cgi?id=1820 (Footnotes starting in English of a Hebrew paragraph come out as Hebrew gibberish)

Also, there are a few regressions with regard to Bidi which are not on the list:

http://bugzilla.lyx.org/show_bug.cgi?id=3550 (Selection in mixed RTL-LTR paragraphs is broken)

http://bugzilla.lyx.org/show_bug.cgi?id=3551 (Only issue (3) is a regression, (1) and (2) are not; but they are solved by the patches I submitted tonight, assuming they are accepted)

If possible, could the patch I submitted here http://thread.gmane.org/gmane.editors.lyx.devel/80783/focus=81153 (and attached, after adjusting to renaming changes) be checked and committed? It's only the beginning of a solution for the problems I raised in that thread, but it's standalone, so there's no reason why it should not go in even though the whole issue isn't resolved yet.

I'll try over the weekend to make a more organized summary of the situation with regard to Hebrew/Bidi.

Thanks!
Dov
Index: src/rowpainter.cpp
===================================================================
--- src/rowpainter.cpp	2007-05-08 23:02:46.000000000 +0300
+++ src/rowpainter.cpp	2007-05-08 23:02:53.000000000 +0300
@@ -818,10 +818,13 @@
 			x_ += 2;
 			++vpos;
 		} else if (par_.isSeparator(pos)) {
+			Font orig_font = text_.getFont(*bv_.buffer(), par_, pos);
+			double const orig_x = x_;
 			x_ += width_pos;
 			if (pos >= body_pos)
 				x_ += separator_;
 			++vpos;
+			paintForeignMark(orig_x, orig_font);
 		} else {
 			paintFromPos(vpos);
 		}

Reply via email to