all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target
@ 2018-12-07 14:46 Drew Adams
  2018-12-07 15:09 ` Eli Zaretskii
  2018-12-09  0:28 ` Stefan Monnier
  0 siblings, 2 replies; 5+ messages in thread
From: Drew Adams @ 2018-12-07 14:46 UTC (permalink / raw)
  To: 33662

This index entry seems a bit wrong, or perhaps the targeted text is a
bit wrong - in this sense: There is no occurrence of "tangible" or
"intangible" in that node.  Using that index entry and arriving at that
node, a user can wonder what happened - why there is nothing there that
is explicitly about "intangible text".  Perhaps the node should make
some mention of intangibility.


In GNU Emacs 26.1 (build 1, x86_64-w64-mingw32)
 of 2018-05-30
Repository revision: 07f8f9bc5a51f5aa94eb099f3e15fbe0c20ea1ea
Windowing system distributor `Microsoft Corp.', version 10.0.16299
Configured using:
 `configure --without-dbus --host=x86_64-w64-mingw32
 --without-compress-install 'CFLAGS=-O2 -static -g3''





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target
  2018-12-07 14:46 bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target Drew Adams
@ 2018-12-07 15:09 ` Eli Zaretskii
  2018-12-09  0:28 ` Stefan Monnier
  1 sibling, 0 replies; 5+ messages in thread
From: Eli Zaretskii @ 2018-12-07 15:09 UTC (permalink / raw)
  To: Drew Adams, Stefan Monnier; +Cc: 33662

> Date: Fri, 7 Dec 2018 14:46:15 +0000 (UTC)
> From: Drew Adams <drew.adams@oracle.com>
> 
> This index entry seems a bit wrong, or perhaps the targeted text is a
> bit wrong - in this sense: There is no occurrence of "tangible" or
> "intangible" in that node.  Using that index entry and arriving at that
> node, a user can wonder what happened - why there is nothing there that
> is explicitly about "intangible text".  Perhaps the node should make
> some mention of intangibility.

The text of that node did mention intangible originally, but Stefan
removed that as part of fixing bug#10222.  I'll let Stefan explain why
he thought that was a good idea.





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target
  2018-12-07 14:46 bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target Drew Adams
  2018-12-07 15:09 ` Eli Zaretskii
@ 2018-12-09  0:28 ` Stefan Monnier
  2018-12-09  4:05   ` Drew Adams
  2018-12-09  6:39   ` Eli Zaretskii
  1 sibling, 2 replies; 5+ messages in thread
From: Stefan Monnier @ 2018-12-09  0:28 UTC (permalink / raw)
  To: Drew Adams; +Cc: 33662

> This index entry seems a bit wrong, or perhaps the targeted text is a
> bit wrong - in this sense: There is no occurrence of "tangible" or
> "intangible" in that node.  Using that index entry and arriving at that
> node, a user can wonder what happened - why there is nothing there that
> is explicitly about "intangible text".  Perhaps the node should make
> some mention of intangibility.

The section is about text that is intangible in the sense that that
point is automatically moved away from such text.

> The text of that node did mention intangible originally, but Stefan
> removed that as part of fixing bug#10222.  I'll let Stefan explain why
> he thought that was a good idea.

I removed mention of the `intangible` property since AFAICT the
corresponding code does not look at the `intangible` property (which is
not handled "After Commands" but directly during execution of the
various point motion Elisp functions).

In my understanding "invisible/intangible" here refers to the concepts
rather to the specific text-properties.

How 'bout the patch below?


        Stefan


diff --git a/doc/lispref/commands.texi b/doc/lispref/commands.texi
index 427379bc79..b6749d0a0c 100644
--- a/doc/lispref/commands.texi
+++ b/doc/lispref/commands.texi
@@ -1012,7 +1012,8 @@ Adjusting Point
 sequence of text that has the @code{display} or @code{composition}
 property, or is invisible.  Therefore, after a command finishes and
 returns to the command loop, if point is within such a sequence, the
-command loop normally moves point to the edge of the sequence.
+command loop normally moves point to the edge of the sequence, making this
+sequence effectively intangible.
 
   A command can inhibit this feature by setting the variable
 @code{disable-point-adjustment}:





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target
  2018-12-09  0:28 ` Stefan Monnier
@ 2018-12-09  4:05   ` Drew Adams
  2018-12-09  6:39   ` Eli Zaretskii
  1 sibling, 0 replies; 5+ messages in thread
From: Drew Adams @ 2018-12-09  4:05 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: 33662

> > This index entry seems a bit wrong, or perhaps the targeted text is a
> > bit wrong - in this sense: There is no occurrence of "tangible" or
> > "intangible" in that node.  Using that index entry and arriving at
> that
> > node, a user can wonder what happened - why there is nothing there
> that
> > is explicitly about "intangible text".  Perhaps the node should make
> > some mention of intangibility.
> 
> The section is about text that is intangible in the sense that that
> point is automatically moved away from such text.
> 
> > The text of that node did mention intangible originally, but Stefan
> > removed that as part of fixing bug#10222.  I'll let Stefan explain
> why
> > he thought that was a good idea.
> 
> I removed mention of the `intangible` property since AFAICT the
> corresponding code does not look at the `intangible` property (which is
> not handled "After Commands" but directly during execution of the
> various point motion Elisp functions).
> 
> In my understanding "invisible/intangible" here refers to the concepts
> rather to the specific text-properties.
> 
> How 'bout the patch below?
>
> if point is within such a sequence, the
> -command loop normally moves point to the edge of the sequence.
> +command loop normally moves point to the edge of the sequence, making
> this
> +sequence effectively intangible.

Thanks for taking a look.  I can't really tell what the
right fix is.  I was looking for info about intangible
text (knowing nothing about it and guessing, wrongly,
that it might be something I could use in a particular
use case).

I used `i', and matched against that long index entry
(using regexp or substring completion matching):
`invisible/intangible text, and point'.

If that node is really about intangible text then your
change is maybe the right fix.  If it is not really
about that (or if it shouldn't really be about it)
then the fix is perhaps to change that index entry.

I'm guessing that that index entry should anyway be
changed somehow (perhaps become multiple entries,
maybe be removed altogether - dunno).  It's a long,
weird index entry.  And without substring matching
for `i' candidates it's unlikely that someone would
be able to use that entry to find that node for info
about "intangibility" anyway.

Since you're here ;-), and since you apparently wrote
`cursor-sensor.el', perhaps you could take a look at
bug #33664 also?  Thx.





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target
  2018-12-09  0:28 ` Stefan Monnier
  2018-12-09  4:05   ` Drew Adams
@ 2018-12-09  6:39   ` Eli Zaretskii
  1 sibling, 0 replies; 5+ messages in thread
From: Eli Zaretskii @ 2018-12-09  6:39 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: 33662

> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Date: Sat, 08 Dec 2018 19:28:05 -0500
> Cc: 33662@debbugs.gnu.org
> 
> The section is about text that is intangible in the sense that that
> point is automatically moved away from such text.
> 
> > The text of that node did mention intangible originally, but Stefan
> > removed that as part of fixing bug#10222.  I'll let Stefan explain why
> > he thought that was a good idea.
> 
> I removed mention of the `intangible` property since AFAICT the
> corresponding code does not look at the `intangible` property (which is
> not handled "After Commands" but directly during execution of the
> various point motion Elisp functions).
> 
> In my understanding "invisible/intangible" here refers to the concepts
> rather to the specific text-properties.
> 
> How 'bout the patch below?

LGTM, thanks.





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-12-09  6:39 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-07 14:46 bug#33662: 26.1; Elisp manual, index entry `invisible/intangible text, and point' and target Drew Adams
2018-12-07 15:09 ` Eli Zaretskii
2018-12-09  0:28 ` Stefan Monnier
2018-12-09  4:05   ` Drew Adams
2018-12-09  6:39   ` Eli Zaretskii

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.