[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#64696: 30.0.50; indent-to inherits preceding text properties, includ
From: |
Eli Zaretskii |
Subject: |
bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible |
Date: |
Wed, 19 Jul 2023 16:10:33 +0300 |
> From: Ihor Radchenko <yantar92@posteo.net>
> Cc: Eli Zaretskii <eliz@gnu.org>, 64696@debbugs.gnu.org
> Date: Wed, 19 Jul 2023 08:42:45 +0000
>
> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
> >>> FWIW, I can't see any good reason to inherit properties here, other than
> >>> "we've always done that", but I'd be surprised if making this change
> >>> won't affect some other use elsewhere :-(
> >>
> >> ??? That's a non-starter, from where I stand.
> >
> > I formulated it differently, but you're just agreeing with the "but ..."
> > quoted above :-)
>
> Note that unlike `indent-to', `insert' does not inherit properties.
> Not to say that changing `indent-to' is safe, but there is slight
> historical inconsistency here.
I don't see any inconsistency. 'insert' is a much more
general-purpose primitive than indent-to. It would be dead wrong for
each insertion to inherit the properties.
indent-to is different: when you indent text, you supposed want the
whitespace to use the same face, for example. Think about it.
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, (continued)
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Eli Zaretskii, 2023/07/23
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Ihor Radchenko, 2023/07/24
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Eli Zaretskii, 2023/07/24
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Ihor Radchenko, 2023/07/25
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Eli Zaretskii, 2023/07/25
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Stefan Monnier, 2023/07/22
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Stefan Monnier, 2023/07/18
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Eli Zaretskii, 2023/07/18
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Stefan Monnier, 2023/07/18
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Ihor Radchenko, 2023/07/19
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible,
Eli Zaretskii <=
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Ihor Radchenko, 2023/07/19
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Eli Zaretskii, 2023/07/19
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Ihor Radchenko, 2023/07/19
- bug#64696: 30.0.50; indent-to inherits preceding text properties, including 'invisible, Stefan Monnier, 2023/07/19