From: Samuel Wales <samologist@gmail.com>
To: Carsten Dominik <carsten.dominik@gmail.com>
Cc: emacs-orgmode@gnu.org
Subject: Re: org-show-following-heading shows even when nil
Date: Fri, 9 Jul 2010 13:27:54 -0700 [thread overview]
Message-ID: <AANLkTil5XReoo9ncxRGe9vU0xYcVhCDpWgyg2T2PbNJt@mail.gmail.com> (raw)
In-Reply-To: <E41E2B0A-3468-4660-B5F8-8E9D36BA3EF1@gmail.com>
Hi Carsten,
Thank you for the patch. This patch works for what it does,
but does not handle the situations that I normally encounter
the bug in.
To expand its scope, I made the the same change to the other
of org-agenda-goto and org-agenda-switch-to and have used
the combined patch (yours and mine) for quite a while now.
I just have git rebase it every pull.
It works for headlines but does not work for body text.
Body text does not stay canonical. That is, you cannot
achieve the state using motion and TAB in the buffer.
Here is a test case. Search for applebanana and RET on it.
Thanks.
Samuel
***** example: my major headline test
SCHEDULED: <2010-07-06 Tue>
some body text.
go up to the headline and fold the whole thing with tab,
then search for apple.banana without the dot. you should
notice that this entire body text is folded.
does another variable control this?
******* my minor headline test applebanana
SCHEDULED: <2010-07-06 Tue>
********* something underneath
********* again
******* another
********* under
********* under
******* more
On 2010-05-14, Carsten Dominik <carsten.dominik@gmail.com> wrote:
>
> On May 7, 2010, at 7:32 AM, Samuel Wales wrote:
>
>> When I go to a location from the agenda, the next headline
>> is revealed if it was hidden, even if it is a child.
>>
>> Setting org-show-following-heading to nil or leaving it at
>> its default value does not prevent this. Should it? The
>> documentation seems to suggest that it should. What I want
>> is for hidden children to remain hidden.
>>
>> Rationale: I like the buffer to be in a form that can be
>> obtained by navigation and cycling. For lack of a better
>> term, I call that "canonical visibility".
>
> Hi Samuel,
>
> for some reason I do not remember, I enforce showing the next headline
> in this special case. Please try the following patch and run with if
> for a while, to see if it causes any problems.
>
> - Carsten
>
>
> Modified lisp/org-agenda.el
> diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
> index 19ea87c..d7bad10 100644
> --- a/lisp/org-agenda.el
> +++ b/lisp/org-agenda.el
> @@ -6124,10 +6124,7 @@ and by additional input from the age of a
> schedules or deadline entry."
> (push-mark)
> (goto-char pos)
> (when (org-mode-p)
> - (org-show-context 'agenda)
> - (save-excursion
> - (and (outline-next-heading)
> - (org-flag-heading nil)))) ; show the next heading
> + (org-show-context 'agenda))
> (recenter (/ (window-height) 2))
> (run-hooks 'org-agenda-after-show-hook)
> (and highlight (org-highlight (point-at-bol) (point-at-eol)))))
>
>
>
>>
>> Thanks.
>>
>> Samuel
>>
>> --
>> Q: How many CDC "scientists" does it take to change a lightbulb?
>> A: "You only think it's dark." [CDC has denied a deadly disease for
>> 25 years]
>> ==========
>> Retrovirus: http://www.wpinstitute.org/xmrv/index.html
>>
>>
>> _______________________________________________
>> Emacs-orgmode mailing list
>> Please use `Reply All' to send replies to the list.
>> Emacs-orgmode@gnu.org
>> http://lists.gnu.org/mailman/listinfo/emacs-orgmode
>
> - Carsten
>
>
>
>
--
Q: How many CDC "scientists" does it take to change a lightbulb?
A: "You only think it's dark." [CDC has denied a deadly disease for 25 years]
==========
Retrovirus: http://www.wpinstitute.org/xmrv/index.html -- PLEASE DONATE
prev parent reply other threads:[~2010-07-09 20:28 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-07 5:32 org-show-following-heading shows even when nil Samuel Wales
2010-05-14 22:45 ` Samuel Wales
2010-05-15 6:41 ` Carsten Dominik
2010-05-18 4:17 ` Samuel Wales
2010-05-19 9:56 ` Carsten Dominik
2010-05-22 18:13 ` Samuel Wales
2010-05-24 1:58 ` Samuel Wales
2010-07-09 20:27 ` Samuel Wales [this message]
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=AANLkTil5XReoo9ncxRGe9vU0xYcVhCDpWgyg2T2PbNJt@mail.gmail.com \
--to=samologist@gmail.com \
--cc=carsten.dominik@gmail.com \
--cc=emacs-orgmode@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 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.