>>>>> "Dov" == Dov Feldstern <[EMAIL PROTECTED]> writes:

Dov> Please, could someone respond regarding this patch from Elazar?
Dov> It fixes issue (3) of
Dov> http://bugzilla.lyx.org/show_bug.cgi?id=3551, which is a
Dov> regression.

Dov> It works. The concept makes sense. The implementation -- well, we
Dov> don't love it, but we haven't gotten any better suggestions. (The
Dov> question is: how to identify the difference between being "at an
Dov> inset" vs. "inside it".) It's safe --- at the very worst, it will
Dov> only affect bidi users.

I still have the nasty feeling that the correction could be done in
somewhere like coordOffset (bufferview_funcs.cpp), but since I do not
have the energy to actually research it, I think indeed that the patch
could be committed.

Actually, coordOffset is not good because it also handles non-text
insets. 

Dov> So please, can we have it okayed and committed?

José?

JMarc

Reply via email to