{"action":"reopened","issue":{"url":"https://api.github.com/repos/magit/magit/issues/1771","labels_url":"https://api.github.com/repos/magit/magit/issues/1771/labels{/name}","comments_url":"https://api.github.com/repos/magit/magit/issues/1771/comments","events_url":"https://api.github.com/repos/magit/magit/issues/1771/events","html_url":"https://github.com/magit/magit/issues/1771","id":62491547,"number":1771,"title":"Point lands before invisible section instead of next visible one","user":{"login":"uu1101","id":552356,"avatar_url":"https://avatars.githubusercontent.com/u/552356?v=3","gravatar_id":"","url":"https://api.github.com/users/uu1101","html_url":"https://github.com/uu1101","followers_url":"https://api.github.com/users/uu1101/followers","following_url":"https://api.github.com/users/uu1101/following{/other_user}","gists_url":"https://api.github.com/users/uu1101/gists{/gist_id}","starred_url":"https://api.github.com/users/uu1101/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/uu1101/subscriptions","organizations_url":"https://api.github.com/users/uu1101/orgs","repos_url":"https://api.github.com/users/uu1101/repos","events_url":"https://api.github.com/users/uu1101/events{/privacy}","received_events_url":"https://api.github.com/users/uu1101/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/magit/magit/labels/01%20unconfirmed","name":"01 unconfirmed","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/magit/magit/milestones/5","html_url":"https://github.com/magit/magit/milestones/2.1.0","labels_url":"https://api.github.com/repos/magit/magit/milestones/5/labels","id":336392,"number":5,"title":"2.1.0","description":"","creator":{"login":"tarsius","id":25046,"avatar_url":"https://avatars.githubusercontent.com/u/25046?v=3","gravatar_id":"","url":"https://api.github.com/users/tarsius","html_url":"https://github.com/tarsius","followers_url":"https://api.github.com/users/tarsius/followers","following_url":"https://api.github.com/users/tarsius/following{/other_user}","gists_url":"https://api.github.com/users/tarsius/gists{/gist_id}","starred_url":"https://api.github.com/users/tarsius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tarsius/subscriptions","organizations_url":"https://api.github.com/users/tarsius/orgs","repos_url":"https://api.github.com/users/tarsius/repos","events_url":"https://api.github.com/users/tarsius/events{/privacy}","received_events_url":"https://api.github.com/users/tarsius/received_events","type":"User","site_admin":false},"open_issues":3,"closed_issues":1210,"state":"open","created_at":"2013-05-18T00:36:03Z","updated_at":"2015-05-07T11:04:31Z","due_on":null,"closed_at":null},"comments":8,"created_at":"2015-03-17T19:14:28Z","updated_at":"2015-05-07T11:04:31Z","closed_at":null,"body":"When `line-move-visual` is `nil`, issuing `next-line` on the `magit-status` buffer does not move point to the next visible header, it moves it just before the following invisible header.  This causes the cursor to look positioned at the next visible heading, when actually`(point)` is just before the next hidden header. [*] \r\n\r\nThe following function can help in reproducing the issue:\r\n~~~{.elisp}\r\n(defun next-line-non-visual (&optional arg)\r\n  (interactive)\r\n  (or arg (setq arg 1))\r\n  (let ((line-move-visual nil))\r\n    (next-line arg)))\r\n(global-set-key (kbd \"C-S-n\") #'next-line-non-visual)\r\n~~~\r\n\r\nThe change below fixes the issue /on my machine/, but I admit that I do not understand what's going on on 9d55bcaf53627a06e4c149d6b42272509ad51f66:\r\n\r\n~~~{.patch}\r\ndiff --git i/magit-mode.el w/magit-mode.el\r\nindex a7f16e7..d3b5254 100644\r\n--- i/magit-mode.el\r\n+++ w/magit-mode.el\r\n@@ -264,9 +264,7 @@ Magit is documented in info node `(magit)'.\"\r\n \r\n (defun magit-post-command-adjust-point ()\r\n   (when (and (get-text-property (point) 'invisible)\r\n-             (not (if (fboundp 'get-pos-property) ; since 24.4, see #1671\r\n-                      (get-pos-property (point) 'invisible)\r\n-                    (get-text-property (1+ (point)) 'invisible))))\r\n+             (get-text-property (1+ (point)) 'invisible))\r\n     (goto-char (next-single-char-property-change (point) 'invisible))))\r\n \r\n (defvar-local magit-refresh-function nil)\r\n~~~\r\n\r\nThis hapens to me as a user of the evil package: it is common to rebind `j` and `k` to `evil-{next,previous}-line`.  These let-bind `line-move-visual` to `nil` before invoking `{next,previous}-line`.\r\n\r\nI know you are working hard on getting the release ready; rest  assured that this can wait for the next one.  As always, thank you very much for your effort.\r\n\r\nBest regards,\r\nuu1101.\r\n\r\nRelated to #1592.\r\n\r\n[*]: This is quite difficult for me to explain, please ask for clarifications."}}