org: Reset saved scroll position on new cycling sequence

* lisp/org.el (org-optimize-window-after-visibility-change): Reset
org-scroll-position-to-restore if it is not a repeated call.

The org-scroll-position-to-restore variable is used to save the
position, but the position may be from a cycling sequence other than
the current one.  Reset it at the start of a sequence.
This commit is contained in:
Kyle Meyer 2020-12-10 00:40:05 -05:00
parent 4f8ac0a7a3
commit cf02219e72
1 changed files with 17 additions and 14 deletions

View File

@ -6615,20 +6615,23 @@ With numerical argument N, show content up to level N."
"Adjust the window after a change in outline visibility.
This function is the default value of the hook `org-cycle-hook'."
(when (get-buffer-window (current-buffer))
(cond
((eq state 'content) nil)
((eq state 'all) nil)
((and org-scroll-position-to-restore
(eq state 'folded) (eq last-command this-command))
(set-window-start nil org-scroll-position-to-restore))
((eq state 'folded) nil)
((eq state 'children)
(setq org-scroll-position-to-restore (window-start))
(or (org-subtree-end-visible-p) (recenter 1)))
((eq state 'subtree)
(when (not (eq last-command this-command))
(setq org-scroll-position-to-restore (window-start)))
(or (org-subtree-end-visible-p) (recenter 1))))))
(let ((repeat (eq last-command this-command)))
(unless repeat
(setq org-scroll-position-to-restore nil))
(cond
((eq state 'content) nil)
((eq state 'all) nil)
((and org-scroll-position-to-restore repeat
(eq state 'folded))
(set-window-start nil org-scroll-position-to-restore))
((eq state 'folded) nil)
((eq state 'children)
(setq org-scroll-position-to-restore (window-start))
(or (org-subtree-end-visible-p) (recenter 1)))
((eq state 'subtree)
(unless repeat
(setq org-scroll-position-to-restore (window-start)))
(or (org-subtree-end-visible-p) (recenter 1)))))))
(defun org-clean-visibility-after-subtree-move ()
"Fix visibility issues after moving a subtree."