unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: David Kastrup <dak@gnu.org>
Cc: emacs-bidi@gnu.org, emacs-devel@gnu.org
Subject: Re: Re: Column numbering in bidirectional display
Date: Fri, 21 May 2010 14:17:54 +0300	[thread overview]
Message-ID: <83zkztzeot.fsf@gnu.org> (raw)
In-Reply-To: <87y6fdppp7.fsf@lola.goethe.zz>

> From: David Kastrup <dak@gnu.org>
> Date: Fri, 21 May 2010 11:30:12 +0200
> Cc: emacs-devel@gnu.org
> 
> It is my opinion that bidi reordering should be kept strictly a display
> feature.

Just so I'm sure I understand what you are saying: do you agree that
current-column should return a logical-order column number it does
today?

> A command like vertical-motion acts on a display text presentation
> rather than a logical representation: it would heed bidi (where
> applicable).

This already works, as long as all paragraphs have the same direction,
either L2R or R2L.  The cursor is placed on characters whose visual
distance from the window margin is the same (as far as the line's
length allows that).  That's because the display engine internally
keeps the correct horizontal position of each glyph, after reordering,
and the various routines that move in ``display line'' use bidi
iteration.

I will probably need to fix this for when paragraph direction changes;
currently, Emacs puts the cursor at the same distance from the other
edge of the window, which is not terribly wrong, but I think users
will not expect that.  However, note that if this is fixed, the value
of current-column will change when point moves from a L2R paragraph to
a R2L one or vice versa.

> Programmatically, text manipulation should keep as far away from those
> display-oriented functions as possible (except where indeed the display
> representation should be manipulated).  And all basic text manipulation
> should stay

Hmm, looks unfinished.

Anyway, thanks for the feedback.

  reply	other threads:[~2010-05-21 11:17 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-21  9:08 Column numbering in bidirectional display Eli Zaretskii
2010-05-21  9:30 ` David Kastrup
2010-05-21 11:17   ` Eli Zaretskii [this message]
2010-05-21 13:20 ` Yair F
2010-05-21 14:07   ` David Kastrup
2010-05-22  0:34 ` Stefan Monnier
2010-06-30  4:30 ` "Martin J. Dürst"
2010-06-30 17:22   ` Eli Zaretskii
2010-07-01  1:36     ` "Martin J. Dürst"

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83zkztzeot.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=dak@gnu.org \
    --cc=emacs-bidi@gnu.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).