emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)]
@ 2021-11-15 18:29 Edrie Ddrie
  2021-11-19 14:34 ` Ihor Radchenko
  0 siblings, 1 reply; 4+ messages in thread
From: Edrie Ddrie @ 2021-11-15 18:29 UTC (permalink / raw)
  To: emacs-orgmode

[-- Attachment #1: Type: text/plain, Size: 730 bytes --]

When using `org-fill-paragraph' on a part of a paragraph, e.g. with halve
the lines as active region, it still uses the unselected text around when
filling.
This works different than `fill-paragraph', `fill-region' and
`fill-region-as-paragraph' that fill only the selected part of a paragraph
WITHOUT effecting the text around.
I don't know if this is a real bug but it is confusing, and annoying
because org-mode overrides the default `C-q' key binding for
`fill-paragraph' so I have to invoke that command via M-x.

Emacs  : GNU Emacs 27.1 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.5,
cairo version 1.16.0)
 of 2021-02-09, modified by Debian
Package: Org mode version 9.3 (release_9.3 @
/usr/share/emacs/27.1/lisp/org/)

[-- Attachment #2: Type: text/html, Size: 851 bytes --]

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

* Re: Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)]
  2021-11-15 18:29 Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)] Edrie Ddrie
@ 2021-11-19 14:34 ` Ihor Radchenko
  2022-10-06  8:38   ` Should we make org-fill-element respect the selected region and not expand the selection to the element boundaties? (was: Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)]) Ihor Radchenko
  0 siblings, 1 reply; 4+ messages in thread
From: Ihor Radchenko @ 2021-11-19 14:34 UTC (permalink / raw)
  To: Edrie Ddrie; +Cc: emacs-orgmode

Edrie Ddrie <planete3@gmail.com> writes:

> When using `org-fill-paragraph' on a part of a paragraph, e.g. with halve
> the lines as active region, it still uses the unselected text around when
> filling.
> This works different than `fill-paragraph', `fill-region' and
> `fill-region-as-paragraph' that fill only the selected part of a paragraph
> WITHOUT effecting the text around.
> I don't know if this is a real bug but it is confusing, and annoying
> because org-mode overrides the default `C-q' key binding for
> `fill-paragraph' so I have to invoke that command via M-x.

Confirmed

Best,
Ihor


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

* Should we make org-fill-element respect the selected region and not expand the selection to the element boundaties? (was: Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)])
  2021-11-19 14:34 ` Ihor Radchenko
@ 2022-10-06  8:38   ` Ihor Radchenko
  2022-10-06  9:32     ` Should we make org-fill-element respect the selected region and not expand the selection to the element boundaties? Fraga, Eric
  0 siblings, 1 reply; 4+ messages in thread
From: Ihor Radchenko @ 2022-10-06  8:38 UTC (permalink / raw)
  To: Edrie Ddrie; +Cc: emacs-orgmode

Ihor Radchenko <yantar92@gmail.com> writes:

> Edrie Ddrie <planete3@gmail.com> writes:
>
>> When using `org-fill-paragraph' on a part of a paragraph, e.g. with halve
>> the lines as active region, it still uses the unselected text around when
>> filling.
>> This works different than `fill-paragraph', `fill-region' and
>> `fill-region-as-paragraph' that fill only the selected part of a paragraph
>> WITHOUT effecting the text around.
>> I don't know if this is a real bug but it is confusing, and annoying
>> because org-mode overrides the default `C-q' key binding for
>> `fill-paragraph' so I have to invoke that command via M-x.
>
> Confirmed

While I understand that the current behaviour is unexpected, it is
internally consistent within Org for different kind of Org elements:
paragraphs, source blocks, comments, tables etc. For all the elements,
Org effectively expands the region boundaries to the beginning/end of
the first/last element in the selected region.

If we do decide to change expanding the selection to the element
boundaries, it should be done across all filled element types.

It will also be a breaking change.

Should we go for it?

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>


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

* Re: Should we make org-fill-element respect the selected region and not expand the selection to the element boundaties?
  2022-10-06  8:38   ` Should we make org-fill-element respect the selected region and not expand the selection to the element boundaties? (was: Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)]) Ihor Radchenko
@ 2022-10-06  9:32     ` Fraga, Eric
  0 siblings, 0 replies; 4+ messages in thread
From: Fraga, Eric @ 2022-10-06  9:32 UTC (permalink / raw)
  To: Ihor Radchenko; +Cc: Edrie Ddrie, emacs-orgmode@gnu.org

On Thursday,  6 Oct 2022 at 16:38, Ihor Radchenko wrote:
> Should we go for it?

It would mean better alignment with the rest of the text modes in Emacs
so I would be supportive.  But I only tend to use filling for paragraphs
in any case so the breaking change would not affect me much.

-- 
: Eric S Fraga, with org release_9.5.5-853-g7b9d8e in Emacs 29.0.50

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

end of thread, other threads:[~2022-10-06  9:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-15 18:29 Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)] Edrie Ddrie
2021-11-19 14:34 ` Ihor Radchenko
2022-10-06  8:38   ` Should we make org-fill-element respect the selected region and not expand the selection to the element boundaties? (was: Bug: org-fill-paragraph [9.3 (release_9.3 @ /usr/share/emacs/27.1/lisp/org/)]) Ihor Radchenko
2022-10-06  9:32     ` Should we make org-fill-element respect the selected region and not expand the selection to the element boundaties? Fraga, Eric

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.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).