* Moving from org-mode 6.33/7.9.3 to 8.3.2
@ 2015-11-27 16:56 Steve Moreau
2015-11-27 17:32 ` John Hendy
0 siblings, 1 reply; 11+ messages in thread
From: Steve Moreau @ 2015-11-27 16:56 UTC (permalink / raw)
To: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 2895 bytes --]
Hi,
Moving from old org-mode versions (6.33 or 7.9.3f-17-g7524ef) to the latest
stable one currently (8.3.2), I noticed two changes which made me out of my
comfort zone.
1. Demoting/promoting a header now changes the indentation of text below.
I reverted to the previous mode with '(setf org-adapt-indentation nil)'
in ~/.emacs, and that's fine but I mentioned it in case it has an impact on
point#2.
2. 'M-x fill-region' does not fill custom lists as before. In the
previous version, I could select and fill several mixed paragraphs/lists
directly. For example, filling the following block (without --- separator
added as a delimiter in this message)
---------------
# a very looooooooooooooooooooooooooooooooooooooooooooooooooooong comment
a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence that
should be filled over 2 lines
- a looooooooooooooooooooooooooooooooooong item that should be filled
over 2 lines
- a looooooooooooooooooooooooooooooooooong item that should be filled
over 2 lines
a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence that
should be filled over 2 lines
---------------
resulted in (version 6.33 or 7.9.3):
---------------
# a very looooooooooooooooooooooooooooooooooooooooooooooooooooong comment
a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
that should be filled over 2 lines
- a looooooooooooooooooooooooooooooooooong item that should be filled
over 2 lines
- a looooooooooooooooooooooooooooooooooong item that should be
filled over 2 lines
a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
that should be filled over 2 lines
---------------
and now results in (version 8.3.2):
---------------
# a very looooooooooooooooooooooooooooooooooooooooooooooooooooong comment
a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
that should be filled over 2 lines
- a looooooooooooooooooooooooooooooooooong item that should be filled
over 2 lines
- a looooooooooooooooooooooooooooooooooong item that should be filled
over 2 lines
a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
that should be filled over 2 lines
---------------
Lists are not filled anymore and I have to select each list entry and
fill it individually to get the same output.
So my questions are:
- Is there any simple way to revert to the previous mode with a variable?
- Else, could you explain briefly where I should look if I want to
understand why the filling is skipped in this case, and eventually change
it if I cannot change my mind and adapt to this different behavior? I had a
look at function (org.el:23249 org-setup-filling) but I am not sure this is
the right location.
Thanks, and have a nice day.
[-- Attachment #2: Type: text/html, Size: 3241 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-11-27 16:56 Moving from org-mode 6.33/7.9.3 to 8.3.2 Steve Moreau
@ 2015-11-27 17:32 ` John Hendy
2015-11-27 18:13 ` John Hendy
[not found] ` <CAD9K4eCt9Gha63gaWagEsihPSJzttE5aBh_BFV0aoG0DXPs--A@mail.gmail.com>
0 siblings, 2 replies; 11+ messages in thread
From: John Hendy @ 2015-11-27 17:32 UTC (permalink / raw)
To: Steve Moreau; +Cc: emacs-orgmode
On Fri, Nov 27, 2015 at 10:56 AM, Steve Moreau <moreau.steve@free.fr> wrote:
> Hi,
>
> Moving from old org-mode versions (6.33 or 7.9.3f-17-g7524ef) to the latest
> stable one currently (8.3.2), I noticed two changes which made me out of my
> comfort zone.
>
> Demoting/promoting a header now changes the indentation of text below.
> I reverted to the previous mode with '(setf org-adapt-indentation nil)' in
> ~/.emacs, and that's fine but I mentioned it in case it has an impact on
> point#2.
>
> 'M-x fill-region' does not fill custom lists as before. In the previous
> version, I could select and fill several mixed paragraphs/lists directly.
> For example, filling the following block (without --- separator added as a
> delimiter in this message)
> ---------------
> # a very looooooooooooooooooooooooooooooooooooooooooooooooooooong comment
>
> a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence that
> should be filled over 2 lines
>
> - a looooooooooooooooooooooooooooooooooong item that should be filled over 2
> lines
>
> - a looooooooooooooooooooooooooooooooooong item that should be filled over
> 2 lines
>
> a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence that
> should be filled over 2 lines
[snip]
> Lists are not filled anymore and I have to select each list entry and fill
> it individually to get the same output.
>
> So my questions are:
>
> Is there any simple way to revert to the previous mode with a variable?
> Else, could you explain briefly where I should look if I want to understand
> why the filling is skipped in this case, and eventually change it if I
> cannot change my mind and adapt to this different behavior? I had a look at
> function (org.el:23249 org-setup-filling) but I am not sure this is the
> right location.
I don't really have an answer regarding *why*, but I have this in my
.emacs that seems to work with either spaces at the end of your
example above (fills the current line) or with M-x fill-region. Want
to try that? Maybe the new org requires manually setting fill options?
#+begin_src .emacs
;; change 80 to whatever you prefer
(setq-default fill-column 80)
(add-hook 'text-mode-hook 'turn-on-auto-fill)
#+end_src
Hope that helps!
John
>
> Thanks, and have a nice day.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-11-27 17:32 ` John Hendy
@ 2015-11-27 18:13 ` John Hendy
[not found] ` <CAD9K4eCt9Gha63gaWagEsihPSJzttE5aBh_BFV0aoG0DXPs--A@mail.gmail.com>
1 sibling, 0 replies; 11+ messages in thread
From: John Hendy @ 2015-11-27 18:13 UTC (permalink / raw)
To: Steve Moreau; +Cc: emacs-orgmode
On Fri, Nov 27, 2015 at 11:32 AM, John Hendy <jw.hendy@gmail.com> wrote:
> On Fri, Nov 27, 2015 at 10:56 AM, Steve Moreau <moreau.steve@free.fr> wrote:
>> Hi,
>>
>> Moving from old org-mode versions (6.33 or 7.9.3f-17-g7524ef) to the latest
>> stable one currently (8.3.2), I noticed two changes which made me out of my
>> comfort zone.
>>
>> Demoting/promoting a header now changes the indentation of text below.
>> I reverted to the previous mode with '(setf org-adapt-indentation nil)' in
>> ~/.emacs, and that's fine but I mentioned it in case it has an impact on
>> point#2.
>>
>> 'M-x fill-region' does not fill custom lists as before. In the previous
>> version, I could select and fill several mixed paragraphs/lists directly.
>> For example, filling the following block (without --- separator added as a
>> delimiter in this message)
>> ---------------
>> # a very looooooooooooooooooooooooooooooooooooooooooooooooooooong comment
>>
>> a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence that
>> should be filled over 2 lines
>>
>> - a looooooooooooooooooooooooooooooooooong item that should be filled over 2
>> lines
>>
>> - a looooooooooooooooooooooooooooooooooong item that should be filled over
>> 2 lines
>>
>> a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence that
>> should be filled over 2 lines
>
> [snip]
>
>> Lists are not filled anymore and I have to select each list entry and fill
>> it individually to get the same output.
>>
>> So my questions are:
>>
>> Is there any simple way to revert to the previous mode with a variable?
>> Else, could you explain briefly where I should look if I want to understand
>> why the filling is skipped in this case, and eventually change it if I
>> cannot change my mind and adapt to this different behavior? I had a look at
>> function (org.el:23249 org-setup-filling) but I am not sure this is the
>> right location.
>
> I don't really have an answer regarding *why*, but I have this in my
> .emacs that seems to work with either spaces at the end of your
> example above (fills the current line) or with M-x fill-region. Want
> to try that? Maybe the new org requires manually setting fill options?
>
> #+begin_src .emacs
>
> ;; change 80 to whatever you prefer
> (setq-default fill-column 80)
> (add-hook 'text-mode-hook 'turn-on-auto-fill)
>
> #+end_src
I take that back. It didn't work on the unordered list after all.
Googling around, turns out someone asked about this sort of thing
before (me!):
- http://comments.gmane.org/gmane.emacs.orgmode/76593
In following the thread, it does't seem like there was a
fill-paragraph equivalent that worked on lists. Perhaps someone else
can chime in here on why Org doesn't fill on lists. I get no filling
when running =M-x fill-paragraph= on your example, and it turns it
into a paragraph (ignoring the current "-" bullet structure) using
=M-x fill-region-as-paragraph=.
John
>
>
> Hope that helps!
> John
>
>>
>> Thanks, and have a nice day.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
[not found] ` <CA+M2ft9ApGTz8ogGaCxCfejAnhWW2Hwuj1X7ZwWg_VvMkmzAmA@mail.gmail.com>
@ 2015-11-28 17:15 ` Steve Moreau
2015-11-29 14:44 ` Steve Moreau
0 siblings, 1 reply; 11+ messages in thread
From: Steve Moreau @ 2015-11-28 17:15 UTC (permalink / raw)
To: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 4062 bytes --]
To anyone interested in tracking this behavior, here's what I can say for
now.
Filling list in the given example changes* from tag 'release_8.0.7' to tag
'release_8.1'*.
I keep you updated. Have a nice day,
2015-11-27 19:15 GMT+01:00 John Hendy <jw.hendy@gmail.com>:
> Whoops! I swore when I tested that it filled the lists, but you're
> right! Hmm. Just sent a follow-up to the list clarifying that this did
> *not* work, as well as including a link where it turns out I actually
> asked the same thing some 2 years ago! Hope someone chimes in to help
> you out. I definitely would like to use something like this, as when I
> edit lists in my work reports/presentations, I mess up the nicely
> filled original wrapping behavior and I'd like to tidy it back up.
>
>
> John
>
> On Fri, Nov 27, 2015 at 11:54 AM, Steve Moreau <moreau.steve@free.fr>
> wrote:
> > Hi John,
> >
> > Thanks for your answer.
> > You made me discover the auto-fill hook and it works fine both in
> paragraphs
> > and list whatever the number of nested level (list entries).
> > But, if I select all the block and run fill-region (let say after a
> > copy/paste), only the paragraphs are properly filled in my case.
> > I guess it's worth getting a deeper look at the source code. Keep you
> > updated.
> >
> >
> >
> > 2015-11-27 18:32 GMT+01:00 John Hendy <jw.hendy@gmail.com>:
> >>
> >> On Fri, Nov 27, 2015 at 10:56 AM, Steve Moreau <moreau.steve@free.fr>
> >> wrote:
> >> > Hi,
> >> >
> >> > Moving from old org-mode versions (6.33 or 7.9.3f-17-g7524ef) to the
> >> > latest
> >> > stable one currently (8.3.2), I noticed two changes which made me out
> of
> >> > my
> >> > comfort zone.
> >> >
> >> > Demoting/promoting a header now changes the indentation of text below.
> >> > I reverted to the previous mode with '(setf org-adapt-indentation
> nil)'
> >> > in
> >> > ~/.emacs, and that's fine but I mentioned it in case it has an impact
> on
> >> > point#2.
> >> >
> >> > 'M-x fill-region' does not fill custom lists as before. In the
> previous
> >> > version, I could select and fill several mixed paragraphs/lists
> >> > directly.
> >> > For example, filling the following block (without --- separator added
> as
> >> > a
> >> > delimiter in this message)
> >> > ---------------
> >> > # a very looooooooooooooooooooooooooooooooooooooooooooooooooooong
> >> > comment
> >> >
> >> > a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
> that
> >> > should be filled over 2 lines
> >> >
> >> > - a looooooooooooooooooooooooooooooooooong item that should be filled
> >> > over 2
> >> > lines
> >> >
> >> > - a looooooooooooooooooooooooooooooooooong item that should be
> filled
> >> > over
> >> > 2 lines
> >> >
> >> > a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
> that
> >> > should be filled over 2 lines
> >>
> >> [snip]
> >>
> >> > Lists are not filled anymore and I have to select each list entry and
> >> > fill
> >> > it individually to get the same output.
> >> >
> >> > So my questions are:
> >> >
> >> > Is there any simple way to revert to the previous mode with a
> variable?
> >> > Else, could you explain briefly where I should look if I want to
> >> > understand
> >> > why the filling is skipped in this case, and eventually change it if I
> >> > cannot change my mind and adapt to this different behavior? I had a
> look
> >> > at
> >> > function (org.el:23249 org-setup-filling) but I am not sure this is
> the
> >> > right location.
> >>
> >> I don't really have an answer regarding *why*, but I have this in my
> >> .emacs that seems to work with either spaces at the end of your
> >> example above (fills the current line) or with M-x fill-region. Want
> >> to try that? Maybe the new org requires manually setting fill options?
> >>
> >> #+begin_src .emacs
> >>
> >> ;; change 80 to whatever you prefer
> >> (setq-default fill-column 80)
> >> (add-hook 'text-mode-hook 'turn-on-auto-fill)
> >>
> >> #+end_src
> >>
> >>
> >> Hope that helps!
> >> John
> >>
> >> >
> >> > Thanks, and have a nice day.
> >>
> >
>
>
[-- Attachment #2: Type: text/html, Size: 5700 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-11-28 17:15 ` Steve Moreau
@ 2015-11-29 14:44 ` Steve Moreau
2015-11-29 21:43 ` Nicolas Goaziou
0 siblings, 1 reply; 11+ messages in thread
From: Steve Moreau @ 2015-11-29 14:44 UTC (permalink / raw)
To: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 5766 bytes --]
Hi,
First, I have been struggling with git since I have no methodology, and I
got lost several times.
But I converged anyway and, as far as I understand, the issue here should
have been introduced in the following commit:
Revision: f3955d5367bd44e98824e3a5bd4b36e06483f4c2
Author: Nicolas Goaziou <n.goaziou@gmail.com>
Date: 2013-06-02 11:12:02 AM
Message:
org.el: Slight change to filling mechanism
* lisp/org.el (org-setup-filling): Set `paragraph-start' and
`paragraph-separate'.
(org-fill-paragraph-separate-nobreak-p): Remove function.
(org-mode): Do not set `paragraph-start'.
----
Modified: lisp/org.el
Indeed, the org-setup-filling was concerned.
*Before:*
(append fill-nobreak-predicate
'(org-fill-paragraph-separate-nobreak-p
...
(defvar org-element-paragraph-separate) ; org-element.el
(defun org-fill-paragraph-separate-nobreak-p ()
"Non-nil when a new line at point would end current paragraph."
(looking-at (substring org-element-paragraph-separate 1)))
*After:*
'org-fill-paragraph-separate-nobreak-p' predicate removed and the previous
function has been deleted
The following code has been added:
(let ((paragraph-ending (substring org-element-paragraph-separate 1)))
(org-set-local 'paragraph-start paragraph-ending)
(org-set-local 'paragraph-separate paragraph-ending))
Could someone explain to me why the function has been removed?
Thanks a lot for the great job on this wonderful org-mode.
Have a nice day,
Steve
2015-11-28 18:15 GMT+01:00 Steve Moreau <moreau.steve@free.fr>:
> To anyone interested in tracking this behavior, here's what I can say for
> now.
>
> Filling list in the given example changes* from tag 'release_8.0.7' to
> tag 'release_8.1'*.
>
> I keep you updated. Have a nice day,
>
> 2015-11-27 19:15 GMT+01:00 John Hendy <jw.hendy@gmail.com>:
>
>> Whoops! I swore when I tested that it filled the lists, but you're
>> right! Hmm. Just sent a follow-up to the list clarifying that this did
>> *not* work, as well as including a link where it turns out I actually
>> asked the same thing some 2 years ago! Hope someone chimes in to help
>> you out. I definitely would like to use something like this, as when I
>> edit lists in my work reports/presentations, I mess up the nicely
>> filled original wrapping behavior and I'd like to tidy it back up.
>>
>>
>> John
>>
>> On Fri, Nov 27, 2015 at 11:54 AM, Steve Moreau <moreau.steve@free.fr>
>> wrote:
>> > Hi John,
>> >
>> > Thanks for your answer.
>> > You made me discover the auto-fill hook and it works fine both in
>> paragraphs
>> > and list whatever the number of nested level (list entries).
>> > But, if I select all the block and run fill-region (let say after a
>> > copy/paste), only the paragraphs are properly filled in my case.
>> > I guess it's worth getting a deeper look at the source code. Keep you
>> > updated.
>> >
>> >
>> >
>> > 2015-11-27 18:32 GMT+01:00 John Hendy <jw.hendy@gmail.com>:
>> >>
>> >> On Fri, Nov 27, 2015 at 10:56 AM, Steve Moreau <moreau.steve@free.fr>
>> >> wrote:
>> >> > Hi,
>> >> >
>> >> > Moving from old org-mode versions (6.33 or 7.9.3f-17-g7524ef) to the
>> >> > latest
>> >> > stable one currently (8.3.2), I noticed two changes which made me
>> out of
>> >> > my
>> >> > comfort zone.
>> >> >
>> >> > Demoting/promoting a header now changes the indentation of text
>> below.
>> >> > I reverted to the previous mode with '(setf org-adapt-indentation
>> nil)'
>> >> > in
>> >> > ~/.emacs, and that's fine but I mentioned it in case it has an
>> impact on
>> >> > point#2.
>> >> >
>> >> > 'M-x fill-region' does not fill custom lists as before. In the
>> previous
>> >> > version, I could select and fill several mixed paragraphs/lists
>> >> > directly.
>> >> > For example, filling the following block (without --- separator
>> added as
>> >> > a
>> >> > delimiter in this message)
>> >> > ---------------
>> >> > # a very looooooooooooooooooooooooooooooooooooooooooooooooooooong
>> >> > comment
>> >> >
>> >> > a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
>> that
>> >> > should be filled over 2 lines
>> >> >
>> >> > - a looooooooooooooooooooooooooooooooooong item that should be filled
>> >> > over 2
>> >> > lines
>> >> >
>> >> > - a looooooooooooooooooooooooooooooooooong item that should be
>> filled
>> >> > over
>> >> > 2 lines
>> >> >
>> >> > a looooooooooooooooooooooooooooooooooooooooooooooooooooong sentence
>> that
>> >> > should be filled over 2 lines
>> >>
>> >> [snip]
>> >>
>> >> > Lists are not filled anymore and I have to select each list entry and
>> >> > fill
>> >> > it individually to get the same output.
>> >> >
>> >> > So my questions are:
>> >> >
>> >> > Is there any simple way to revert to the previous mode with a
>> variable?
>> >> > Else, could you explain briefly where I should look if I want to
>> >> > understand
>> >> > why the filling is skipped in this case, and eventually change it if
>> I
>> >> > cannot change my mind and adapt to this different behavior? I had a
>> look
>> >> > at
>> >> > function (org.el:23249 org-setup-filling) but I am not sure this is
>> the
>> >> > right location.
>> >>
>> >> I don't really have an answer regarding *why*, but I have this in my
>> >> .emacs that seems to work with either spaces at the end of your
>> >> example above (fills the current line) or with M-x fill-region. Want
>> >> to try that? Maybe the new org requires manually setting fill options?
>> >>
>> >> #+begin_src .emacs
>> >>
>> >> ;; change 80 to whatever you prefer
>> >> (setq-default fill-column 80)
>> >> (add-hook 'text-mode-hook 'turn-on-auto-fill)
>> >>
>> >> #+end_src
>> >>
>> >>
>> >> Hope that helps!
>> >> John
>> >>
>> >> >
>> >> > Thanks, and have a nice day.
>> >>
>> >
>>
>>
>
[-- Attachment #2: Type: text/html, Size: 8163 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-11-29 14:44 ` Steve Moreau
@ 2015-11-29 21:43 ` Nicolas Goaziou
2015-11-30 7:05 ` Steve Moreau
0 siblings, 1 reply; 11+ messages in thread
From: Nicolas Goaziou @ 2015-11-29 21:43 UTC (permalink / raw)
To: Steve Moreau; +Cc: emacs-orgmode
Hello,
Steve Moreau <moreau.steve@free.fr> writes:
> Indeed, the org-setup-filling was concerned.
>
> *Before:*
>
> (append fill-nobreak-predicate
> '(org-fill-paragraph-separate-nobreak-p
> ...
>
> (defvar org-element-paragraph-separate) ; org-element.el
> (defun org-fill-paragraph-separate-nobreak-p ()
> "Non-nil when a new line at point would end current paragraph."
> (looking-at (substring org-element-paragraph-separate 1)))
>
> *After:*
>
> 'org-fill-paragraph-separate-nobreak-p' predicate removed and the previous
> function has been deleted
>
> The following code has been added:
>
> (let ((paragraph-ending (substring org-element-paragraph-separate 1)))
> (org-set-local 'paragraph-start paragraph-ending)
> (org-set-local 'paragraph-separate paragraph-ending))
>
>
> Could someone explain to me why the function has been removed?
It was removed because `org-element-paragraph-separate' is not enough to
find the boundaries of a paragraph, e.g.,
- item1
- item2
This is a new paragraph
It is usually not a problem except for `fill-region', as you noticed.
However, I do not know how to plug our own functions into `fill-region'.
We could replace `fill-region' with `org-fill-region', tho.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-11-29 21:43 ` Nicolas Goaziou
@ 2015-11-30 7:05 ` Steve Moreau
2015-11-30 14:53 ` Nicolas Goaziou
0 siblings, 1 reply; 11+ messages in thread
From: Steve Moreau @ 2015-11-30 7:05 UTC (permalink / raw)
To: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 2136 bytes --]
Hi Nicolas,
Thanks for you answer.
OK, I understand. That explains why I needed to add blank lines between
list items before to fill region, such as:
- item1
- item2
This is a new paragraph
I have 2 more questions for you please.
Let say I would like to give a hand and try to improve the org-fill-region
to deal with this case.
- Would it be local to fill-region only, as I would suspect, or would it
put a mess around because it is used in some other contexts I am not aware
of, and would it be something interesting for the org-mode according to you?
- If you wanted to develop it, would you surround it with a when condition
so that user can disable it?
Thanks,
Steve
2015-11-29 22:43 GMT+01:00 Nicolas Goaziou <mail@nicolasgoaziou.fr>:
> Hello,
>
> Steve Moreau <moreau.steve@free.fr> writes:
>
> > Indeed, the org-setup-filling was concerned.
> >
> > *Before:*
> >
> > (append fill-nobreak-predicate
> > '(org-fill-paragraph-separate-nobreak-p
> > ...
> >
> > (defvar org-element-paragraph-separate) ; org-element.el
> > (defun org-fill-paragraph-separate-nobreak-p ()
> > "Non-nil when a new line at point would end current paragraph."
> > (looking-at (substring org-element-paragraph-separate 1)))
> >
> > *After:*
> >
> > 'org-fill-paragraph-separate-nobreak-p' predicate removed and the
> previous
> > function has been deleted
> >
> > The following code has been added:
> >
> > (let ((paragraph-ending (substring org-element-paragraph-separate 1)))
> > (org-set-local 'paragraph-start paragraph-ending)
> > (org-set-local 'paragraph-separate paragraph-ending))
> >
> >
> > Could someone explain to me why the function has been removed?
>
> It was removed because `org-element-paragraph-separate' is not enough to
> find the boundaries of a paragraph, e.g.,
>
> - item1
> - item2
> This is a new paragraph
>
> It is usually not a problem except for `fill-region', as you noticed.
> However, I do not know how to plug our own functions into `fill-region'.
>
> We could replace `fill-region' with `org-fill-region', tho.
>
>
> Regards,
>
> --
> Nicolas Goaziou
>
>
[-- Attachment #2: Type: text/html, Size: 3059 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-11-30 7:05 ` Steve Moreau
@ 2015-11-30 14:53 ` Nicolas Goaziou
2015-12-01 7:33 ` Steve Moreau
0 siblings, 1 reply; 11+ messages in thread
From: Nicolas Goaziou @ 2015-11-30 14:53 UTC (permalink / raw)
To: Steve Moreau; +Cc: emacs-orgmode
Hello,
Steve Moreau <moreau.steve@free.fr> writes:
> Thanks for you answer.
> OK, I understand. That explains why I needed to add blank lines between
> list items before to fill region, such as:
>
> - item1
>
> - item2
>
> This is a new paragraph
>
> I have 2 more questions for you please.
> Let say I would like to give a hand and try to improve the org-fill-region
> to deal with this case.
Actually, there is nothing to improve. It needs to be built from
scratch. There is no function in Org to fill a region ATM.
> - Would it be local to fill-region only, as I would suspect, or would it
> put a mess around because it is used in some other contexts I am not aware
> of, and would it be something interesting for the org-mode according
> to you?
I think we should bind M-q to org-fill-paragraph. This function could
delegate the work to `org-fill-region' when a region is active.
> - If you wanted to develop it, would you surround it with a when condition
> so that user can disable it?
Why would someone want to disable it?
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-11-30 14:53 ` Nicolas Goaziou
@ 2015-12-01 7:33 ` Steve Moreau
2015-12-01 19:09 ` Samuel Wales
2015-12-01 22:31 ` Nicolas Goaziou
0 siblings, 2 replies; 11+ messages in thread
From: Steve Moreau @ 2015-12-01 7:33 UTC (permalink / raw)
To: Steve Moreau, emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 2072 bytes --]
Hello,
Well, actually, it seems that I did not understand anything at all :-)
Sorry about that, but I am new to the source code, and it is not very clear
to me.
So if I try to summarize what I have got:
- In the past, what I could see with the standard 'M-x fill-region' over
paragraphs or lists with empty lines between each inner items, was probably
performed thanks to several calls to org-fill-paragraph. Removing the
org-fill-paragraph-separate-nobreak-p function simply highlights that I was
fortunate to use it this way before, and there is just no org-fill-region.
- There are no unit tests, nor past discussions about how different text
block contents should be filled with this new function
Please tell if I am still wrong, and otherwise I gonna try to implement
some unit tests over different cases I may think about.
Have a nice day,
2015-11-30 15:53 GMT+01:00 Nicolas Goaziou <mail@nicolasgoaziou.fr>:
> Hello,
>
> Steve Moreau <moreau.steve@free.fr> writes:
>
> > Thanks for you answer.
> > OK, I understand. That explains why I needed to add blank lines between
> > list items before to fill region, such as:
> >
> > - item1
> >
> > - item2
> >
> > This is a new paragraph
> >
> > I have 2 more questions for you please.
> > Let say I would like to give a hand and try to improve the
> org-fill-region
> > to deal with this case.
>
> Actually, there is nothing to improve. It needs to be built from
> scratch. There is no function in Org to fill a region ATM.
>
> > - Would it be local to fill-region only, as I would suspect, or would it
> > put a mess around because it is used in some other contexts I am not
> aware
> > of, and would it be something interesting for the org-mode according
> > to you?
>
> I think we should bind M-q to org-fill-paragraph. This function could
> delegate the work to `org-fill-region' when a region is active.
>
> > - If you wanted to develop it, would you surround it with a when
> condition
> > so that user can disable it?
>
> Why would someone want to disable it?
>
> Regards,
>
> --
> Nicolas Goaziou
>
>
[-- Attachment #2: Type: text/html, Size: 2892 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-12-01 7:33 ` Steve Moreau
@ 2015-12-01 19:09 ` Samuel Wales
2015-12-01 22:31 ` Nicolas Goaziou
1 sibling, 0 replies; 11+ messages in thread
From: Samuel Wales @ 2015-12-01 19:09 UTC (permalink / raw)
To: Steve Moreau; +Cc: emacs-orgmode
whenever something doesn't fill, i enable filladapt.el.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Moving from org-mode 6.33/7.9.3 to 8.3.2
2015-12-01 7:33 ` Steve Moreau
2015-12-01 19:09 ` Samuel Wales
@ 2015-12-01 22:31 ` Nicolas Goaziou
1 sibling, 0 replies; 11+ messages in thread
From: Nicolas Goaziou @ 2015-12-01 22:31 UTC (permalink / raw)
To: Steve Moreau; +Cc: emacs-orgmode
Hello,
Steve Moreau <moreau.steve@free.fr> writes:
> - In the past, what I could see with the standard 'M-x fill-region' over
> paragraphs or lists with empty lines between each inner items, was probably
> performed thanks to several calls to org-fill-paragraph.
I don't think so. `fill-region' uses `fill-region-as-paragraph', which
does its own thing. AFACT, it never calls `fill-paragraph-function',
i.e., `org-fill-paragraph'.
> - There are no unit tests, nor past discussions about how different text
> block contents should be filled with this new function
`org-fill-paragraph' is heavily tested in `test-org/fill-paragraph'.
`org-fill-region' is expected to behave as if `org-fill-paragraph' is
called on every element in the region.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2015-12-01 22:29 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-27 16:56 Moving from org-mode 6.33/7.9.3 to 8.3.2 Steve Moreau
2015-11-27 17:32 ` John Hendy
2015-11-27 18:13 ` John Hendy
[not found] ` <CAD9K4eCt9Gha63gaWagEsihPSJzttE5aBh_BFV0aoG0DXPs--A@mail.gmail.com>
[not found] ` <CA+M2ft9ApGTz8ogGaCxCfejAnhWW2Hwuj1X7ZwWg_VvMkmzAmA@mail.gmail.com>
2015-11-28 17:15 ` Steve Moreau
2015-11-29 14:44 ` Steve Moreau
2015-11-29 21:43 ` Nicolas Goaziou
2015-11-30 7:05 ` Steve Moreau
2015-11-30 14:53 ` Nicolas Goaziou
2015-12-01 7:33 ` Steve Moreau
2015-12-01 19:09 ` Samuel Wales
2015-12-01 22:31 ` Nicolas Goaziou
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).